Skip to content

Implement BigInt asIntN and asUintN methods#5165

Merged
LaszloLango merged 1 commit into
jerryscript-project:masterfrom
gergocs:bigintFix
Nov 22, 2024
Merged

Implement BigInt asIntN and asUintN methods#5165
LaszloLango merged 1 commit into
jerryscript-project:masterfrom
gergocs:bigintFix

Conversation

@gergocs

@gergocs gergocs commented Nov 15, 2024

Copy link
Copy Markdown
Contributor

The following methods were implemented:

  • BigInt.asIntN
  • BigInt.asUintN

Custom dispatcher also added to builtin_bigint.

The implementation is based on PR #4736, only applied the requested changes.

Co-authored-by: Daniel Batiz batizjob@gmail.com
Signed-off-by: Gergo Csizi gergocs@inf.u-szeged.hu

@gergocs
gergocs force-pushed the bigintFix branch 2 times, most recently from d4ba173 to 4a2983c Compare November 15, 2024 07:31
@ossy-szeged

Copy link
Copy Markdown
Contributor

Please add this line before the signed off:
Co-authored-by: Daniel Batiz batizjob@gmail.com

@gergocs
gergocs force-pushed the bigintFix branch 2 times, most recently from c812ef9 to 84a067c Compare November 15, 2024 08:29
@ossy-szeged ossy-szeged added this to the Release 3.0.0 milestone Nov 15, 2024
Comment thread jerry-core/ecma/builtin-objects/ecma-builtin-bigint.c Outdated
Comment thread jerry-core/ecma/builtin-objects/ecma-builtin-bigint.c Outdated
Comment thread jerry-core/lit/lit-magic-strings.inc.h
Comment thread jerry-core/ecma/builtin-objects/ecma-builtin-bigint.c Outdated
Comment thread jerry-core/ecma/builtin-objects/ecma-builtin-bigint.c
Comment thread jerry-core/ecma/builtin-objects/ecma-builtin-bigint.c Outdated
The following methods were implemented:
- BigInt.asIntN
- BigInt.asUintN

Custom dispatcher also added to builtin_bigint.

The implementation is based on PR jerryscript-project#4736, only applied the requested changes.

Co-authored-by: Daniel Batiz batizjob@gmail.com
JerryScript-DCO-1.0-Signed-off-by: Gergo Csizi gergocs@inf.u-szeged.hu

@LaszloLango LaszloLango left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@robertsipka robertsipka left a comment

Copy link
Copy Markdown
Contributor

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 00d12c0 into jerryscript-project:master Nov 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants