Skip to content
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

Rename Js(Copy|Create)PropertyIdUtf8 to Js(Copy|Create)PropertyId #42

Merged
merged 2 commits into from
Aug 22, 2017
Merged

Rename Js(Copy|Create)PropertyIdUtf8 to Js(Copy|Create)PropertyId #42

merged 2 commits into from
Aug 22, 2017

Conversation

xiaoyinl
Copy link
Contributor

There is no exported function called JsCopyPropertyIdUtf8, but there is JsCopyPropertyId,
declared in ChakraCore.h. So update the doc to match the source code.

There is no exported function called JsCopyPropertyIdUtf8, but there is JsCopyPropertyId,
declared in ChakraCore.h. So update the doc to match the source code.
@msftclas
Copy link

@xiaoyinl,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by Microsoft. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

There is no exported function called JsCreatePropertyIdUtf8, but there is JsCreatePropertyId,
declared in ChakraCore.h. So update the doc to match the source code.
Copy link
Member

@liminzhu liminzhu left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks!

@dilijev dilijev changed the title Rename JsCopyPropertyIdUtf8 to JsCopyPropertyId Rename Js(Copy|Create)PropertyIdUtf8 to Js(Copy|Create)PropertyId Aug 22, 2017
@dilijev dilijev merged commit 188d6b2 into microsoft:master Aug 22, 2017
@xiaoyinl xiaoyinl deleted the JsCopyPropertyId branch August 22, 2017 19:32
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