Skip to content

Conversation

@galpeter
Copy link
Contributor

As a few methods were removed and replaced it is a good idea to describe
which method(s) can reproduce the same functionality. In addition add a
bit of extra comments for various methods/structs.

JerryScript-DCO-1.0-Signed-off-by: Peter Gal pgal.usz@partner.samsung.com

@galpeter galpeter added the documentation Related to documentation label Aug 14, 2021
Copy link
Member

@zherczeg zherczeg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM with minor change

@galpeter
Copy link
Contributor Author

Update done and also added an extra comment for the parse option bits.

*New in version 2.4*.

*Changed in version [[NEXT_RELEASE]]*: Added `JERRY_OBJECT_TYPE_SCRIPT`, `JERRY_OBJECT_TYPE_MODULE` values.
*Changed in version [[NEXT_RELEASE]]*: Added `JERRY_OBJECT_TYPE_WEAKREF`, `JERRY_OBJECT_TYPE_SCRIPT`, `JERRY_OBJECT_TYPE_MODULE` values.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How about JERRY_OBJECT_TYPE_SHARED_ARRAYBUFFER? I think it need to be mentioned as well.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

True.

@rerobika
Copy link
Member

There were a lot of api doc changes nowdays, please rebase.

As a few methods were removed and replaced it is a good idea to describe
which method(s) can reproduce the same functionality. In addition add a
bit of extra comments for various methods/structs.

JerryScript-DCO-1.0-Signed-off-by: Peter Gal pgal.usz@partner.samsung.com
@galpeter
Copy link
Contributor Author

Rebased, as per request.

@rerobika rerobika merged commit 1c6b18e into jerryscript-project:master Aug 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Related to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants