Closed Bug 711611 Opened 8 years ago Closed 8 years ago

NSPR's obsolete int32, int64, etc. types get defined without an opt-out #define

Categories

(Core :: JavaScript Engine, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla11

People

(Reporter: Waldo, Assigned: Waldo)

Details

Attachments

(1 file)

"These definitions are scheduled to be eliminated at the earliest possible time."

Yeah.  Right.  By the way, are you in the market for a bridge?

This leaks in through jslock.h, which in threadsafe builds includes various NSPR headers.  Le sigh.  Before this patch, an int32 i; in jsinterp.cpp would build.  After, it fails to build.

I'll wait for try results on this, since it's possible this could have hidden other failures in Gecko, but the approach is reviewable without further ado.
Attachment #582387 - Flags: review?(bhackett1024)
Attachment #582387 - Flags: review?(bhackett1024) → review+
Built on try, pushed:

https://hg.mozilla.org/integration/mozilla-inbound/rev/020e3a98f85e
Target Milestone: --- → mozilla11
https://hg.mozilla.org/mozilla-central/rev/020e3a98f85e
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.