Skip to content

Update macro definitions in Curie-BSP target #1525

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
Jan 16, 2017

Conversation

jiangzidong
Copy link
Contributor

CurieBSP target should change a little because of the recent patches in jerry-core

  • add JERRY_JS_PARSER
  • add CONFIG_DISABLE_ARRAYBUFFER_BUILTIN
  • add CONFIG_DISABLE_TYPEDARRAY_BUILTIN

BTW, change the compiler warning when CONFIG_ECMA_PROPERTY_HASHMAP_DISABLE

there is no variable name_p in ecma_property_hashmap_delete, it should be name_cp

JerryScript-DCO-1.0-Signed-off-by: Zidong Jiang [email protected]

* add JERRY_JS_PARSER
* add CONFIG_DISABLE_ARRAYBUFFER_BUILTIN
* add CONFIG_DISABLE_TYPEDARRAY_BUILTIN
JerryScript-DCO-1.0-Signed-off-by: Zidong Jiang [email protected]
@robertsipka
Copy link
Contributor

LGTM (informally)

@LaszloLango LaszloLango added bug Undesired behaviour jerry-port Related to the port API or the default port implementation labels Jan 16, 2017
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

@tilmannOSG tilmannOSG changed the title update macros define in Curie-BSP target Update macro definitions in Curie-BSP target Jan 16, 2017
Copy link

@tilmannOSG tilmannOSG left a comment

Choose a reason for hiding this comment

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

LGTM!

@tilmannOSG tilmannOSG merged commit 4f3dc2f into jerryscript-project:master Jan 16, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Undesired behaviour jerry-port Related to the port API or the default port implementation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants