-
Notifications
You must be signed in to change notification settings - Fork 688
Add copyWithin and slice functions to TypedArray #2984
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
Add copyWithin and slice functions to TypedArray #2984
Conversation
jerry-core/ecma/builtin-objects/typedarray/ecma-builtin-typedarray-prototype.c
Outdated
Show resolved
Hide resolved
jerry-core/ecma/builtin-objects/typedarray/ecma-builtin-typedarray-prototype.c
Outdated
Show resolved
Hide resolved
jerry-core/ecma/builtin-objects/typedarray/ecma-builtin-typedarray-prototype.c
Outdated
Show resolved
Hide resolved
jerry-core/ecma/builtin-objects/typedarray/ecma-builtin-typedarray-prototype.c
Outdated
Show resolved
Hide resolved
jerry-core/ecma/builtin-objects/typedarray/ecma-builtin-typedarray-prototype.c
Outdated
Show resolved
Hide resolved
jerry-core/ecma/builtin-objects/typedarray/ecma-builtin-typedarray-prototype.c
Outdated
Show resolved
Hide resolved
jerry-core/ecma/builtin-objects/typedarray/ecma-builtin-typedarray-prototype.c
Outdated
Show resolved
Hide resolved
jerry-core/ecma/builtin-objects/typedarray/ecma-builtin-typedarray-prototype.c
Outdated
Show resolved
Hide resolved
jerry-core/ecma/builtin-objects/typedarray/ecma-builtin-typedarray-prototype.c
Outdated
Show resolved
Hide resolved
c76d123 to
50ed1d2
Compare
|
Did some update, but im currently working on the other requests. |
215fa1f to
e874b63
Compare
d232756 to
01c5d10
Compare
jerry-core/ecma/builtin-objects/typedarray/ecma-builtin-typedarray-prototype.c
Outdated
Show resolved
Hide resolved
jerry-core/ecma/builtin-objects/typedarray/ecma-builtin-typedarray-prototype.c
Outdated
Show resolved
Hide resolved
jerry-core/ecma/builtin-objects/typedarray/ecma-builtin-typedarray-prototype.c
Show resolved
Hide resolved
jerry-core/ecma/builtin-objects/typedarray/ecma-builtin-typedarray-prototype.c
Outdated
Show resolved
Hide resolved
jerry-core/ecma/builtin-objects/typedarray/ecma-builtin-typedarray-prototype.c
Outdated
Show resolved
Hide resolved
01c5d10 to
024b804
Compare
|
Did the missing changes. |
jerry-core/ecma/builtin-objects/typedarray/ecma-builtin-typedarray-prototype.c
Outdated
Show resolved
Hide resolved
jerry-core/ecma/builtin-objects/typedarray/ecma-builtin-typedarray-prototype.c
Outdated
Show resolved
Hide resolved
024b804 to
d2acc07
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, after this minor style issue has been fixed.
jerry-core/ecma/builtin-objects/typedarray/ecma-builtin-typedarray-prototype.c
Outdated
Show resolved
Hide resolved
jerry-core/ecma/builtin-objects/typedarray/ecma-builtin-typedarray-prototype.c
Outdated
Show resolved
Hide resolved
jerry-core/ecma/builtin-objects/typedarray/ecma-builtin-typedarray-prototype.c
Outdated
Show resolved
Hide resolved
jerry-core/ecma/builtin-objects/typedarray/ecma-builtin-typedarray-prototype.c
Outdated
Show resolved
Hide resolved
jerry-core/ecma/builtin-objects/typedarray/ecma-builtin-typedarray-prototype.c
Outdated
Show resolved
Hide resolved
jerry-core/ecma/builtin-objects/typedarray/ecma-builtin-typedarray-prototype.c
Outdated
Show resolved
Hide resolved
d2acc07 to
0dcce15
Compare
jerry-core/ecma/builtin-objects/typedarray/ecma-builtin-typedarray-prototype.c
Outdated
Show resolved
Hide resolved
jerry-core/ecma/builtin-objects/typedarray/ecma-builtin-typedarray-prototype.c
Outdated
Show resolved
Hide resolved
jerry-core/ecma/builtin-objects/typedarray/ecma-builtin-typedarray-prototype.c
Outdated
Show resolved
Hide resolved
The algorithm's are based on ECMA-262 v6, 22.2.3.5 and 22.2.3.23 Co-authored-by: Tibor Dusnoki tdusnoki@inf.u-szeged.hu JerryScript-DCO-1.0-Signed-off-by: Adam Szilagyi aszilagy@inf.u-szeged.hu
0dcce15 to
84fdfcd
Compare
galpeter
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
dbatyai
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
The algorithm's are based on ECMA-262 v6, 22.2.3.5 and 22.2.3.23
Co-authored-by: Tibor Dusnoki tdusnoki@inf.u-szeged.hu
JerryScript-DCO-1.0-Signed-off-by: Adam Szilagyi aszilagy@inf.u-szeged.hu