Skip to content

Implement DataView builtin #2804

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Apr 16, 2019

Conversation

rerobika
Copy link
Member

@rerobika rerobika commented Mar 26, 2019

New API functions:

  • jerry_create_dataview
  • jerry_value_is_dataview
  • jerry_get_dataview_buffer

JerryScript-DCO-1.0-Signed-off-by: Robert Fancsik [email protected]

@rerobika rerobika force-pushed the data_view branch 2 times, most recently from 9d05318 to 717c0e9 Compare April 2, 2019 09:38
@rerobika rerobika added the ES2015 Related to ES2015 features label Apr 2, 2019
@rerobika rerobika added this to the Release 2.0 milestone Apr 2, 2019
@rerobika rerobika added the ecma builtins Related to ECMA built-in routines label Apr 2, 2019
@rerobika rerobika force-pushed the data_view branch 2 times, most recently from b1b42dd to 7b3e4d8 Compare April 2, 2019 10:09
@rerobika rerobika marked this pull request as ready for review April 2, 2019 11:45
@rerobika
Copy link
Member Author

rerobika commented Apr 8, 2019

@galpeter @LaszloLango Thanks for the reviews I've updated the patch according to them.

@rerobika
Copy link
Member Author

rerobika commented Apr 9, 2019

Rebased after #2793 has landed.

New API functions:
 - jerry_create_dataview
 - jerry_value_is_dataview
 - jerry_get_dataview_buffer

JerryScript-DCO-1.0-Signed-off-by: Robert Fancsik [email protected]
Copy link
Contributor

@LaszloLango LaszloLango left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@galpeter galpeter left a comment

Choose a reason for hiding this comment

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

lgtm

@LaszloLango LaszloLango merged commit 5c72d99 into jerryscript-project:master Apr 16, 2019
@rerobika rerobika deleted the data_view branch June 16, 2020 08:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ecma builtins Related to ECMA built-in routines ES2015 Related to ES2015 features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants