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

Fix accidental shadowing of js_GetErrorMessage #69

Merged
merged 1 commit into from Apr 11, 2014
Merged

Conversation

@mbrubeck
Copy link
Contributor

mbrubeck commented Apr 11, 2014

jsglue.cpp defines a function with the same name of the jsapi function it tries to wrap, causing a probable infinite loop.

@jdm

This comment has been minimized.

Copy link

jdm commented on 85bfc63 Apr 11, 2014

r+

jdm added a commit that referenced this pull request Apr 11, 2014
Fix accidental shadowing of js_GetErrorMessage
@jdm jdm merged commit b3fa520 into servo:master Apr 11, 2014
mmatyas pushed a commit to mmatyas/rust-mozjs that referenced this pull request Jul 30, 2015
tschneidereit pushed a commit to tschneidereit/rust-mozjs that referenced this pull request Aug 26, 2017
Backport fix for bug 1039986

https://bugzilla.mozilla.org/show_bug.cgi?id=1039986

Needed to fix the prototype inheritance chain in Servo.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/mozjs/69)
<!-- Reviewable:end -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.