Skip to content

Move jerry_port functions into jerry-core #824

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

Closed
wants to merge 1 commit into from

Conversation

rtakacs
Copy link
Contributor

@rtakacs rtakacs commented Jan 27, 2016

Avoid linking errors when building Jerry as a library (eg. for IoT.js).

@LaszloLango
Copy link
Contributor

LGTM

@LaszloLango LaszloLango added the bug Undesired behaviour label Jan 27, 2016
@zherczeg
Copy link
Member

zherczeg commented Feb 1, 2016

Please update this patch to the latest jerry.

@rtakacs
Copy link
Contributor Author

rtakacs commented Feb 1, 2016

Done.

@@ -13,8 +13,8 @@
* limitations under the License.
Copy link
Contributor

Choose a reason for hiding this comment

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

Copyright years need to be updated.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed.

JerryScript-DCO-1.0-Signed-off-by: Roland Takacs rtakacs.u-szeged@partner.samsung.com
@rtakacs
Copy link
Contributor Author

rtakacs commented Feb 5, 2016

I have rebased the patch to the master.

@LaszloLango
Copy link
Contributor

@rtakacs, thanks. Still looks good to me.

@LaszloLango LaszloLango added the critical Raises security concerns label Feb 8, 2016
@LaszloLango
Copy link
Contributor

Landed (6608cc4)

@LaszloLango LaszloLango closed this Feb 9, 2016
@rtakacs rtakacs deleted the jerry-port-funcs branch January 18, 2019 14:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Undesired behaviour critical Raises security concerns
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants