Closed
Bug 670003
Opened 13 years ago
Closed 13 years ago
[OS/2] update Yarr files
Categories
(Core :: JavaScript Engine, defect)
Tracking
()
RESOLVED
FIXED
mozilla8
People
(Reporter: wuno, Assigned: dave.r.yeo)
References
Details
(Whiteboard: tracemonkey)
Attachments
(1 file, 3 obsolete files)
5.93 KB,
patch
|
dmandelin
:
review+
|
Details | Diff | Splinter Review |
Dave has made an initial patch to update Yarr for OS/2
Reporter | ||
Updated•13 years ago
|
Attachment #544617 -
Attachment is patch: true
Attachment #544617 -
Attachment mime type: message/rfc822 → text/plain
Reporter | ||
Updated•13 years ago
|
Assignee: general → daveryeo
Status: NEW → ASSIGNED
Comment 1•13 years ago
|
||
Is the patch ready for review?
(In reply to comment #1) > Is the patch ready for review? Testing an updated patch, will ask you for review after testing
Blocks: 625600
Attachment #544617 -
Attachment is obsolete: true
Attachment #545007 -
Flags: review?(dmandelin)
After some feedback from Rich patch updated with better commit and decommit plus a simplification of protection()
Attachment #545007 -
Attachment is obsolete: true
Attachment #545089 -
Flags: review?(dmandelin)
Attachment #545007 -
Flags: review?(dmandelin)
Comment 5•13 years ago
|
||
Comment on attachment 545089 [details] [diff] [review] Update Yarr to work on OS/2 Review of attachment 545089 [details] [diff] [review]: ----------------------------------------------------------------- ::: js/src/yarr/PageBlock.cpp @@ +58,5 @@ > +{ > + return getpagesize(); > +} > + > +#elif WTF_OS_OS2 One question: any reason not to just use #if (WTF_OS_UNIX && !WTF_OS_SYMBIAN) || WTF_OS_OS2 above?
(In reply to comment #5) > Comment on attachment 545089 [details] [diff] [review] [review] > Update Yarr to work on OS/2 > > Review of attachment 545089 [details] [diff] [review] [review]: > ----------------------------------------------------------------- > > ::: js/src/yarr/PageBlock.cpp > @@ +58,5 @@ > > +{ > > + return getpagesize(); > > +} > > + > > +#elif WTF_OS_OS2 > > One question: any reason not to just use > > #if (WTF_OS_UNIX && !WTF_OS_SYMBIAN) || WTF_OS_OS2 > > above? I meant to fix this and forgot. Updated patch attached
Attachment #545089 -
Attachment is obsolete: true
Attachment #545319 -
Flags: review?(dmandelin)
Attachment #545089 -
Flags: review?(dmandelin)
Comment 7•13 years ago
|
||
Comment on attachment 545319 [details] [diff] [review] Update Yarr to work on OS/2 Review of attachment 545319 [details] [diff] [review]: -----------------------------------------------------------------
Attachment #545319 -
Flags: review?(dmandelin) → review+
Reporter | ||
Updated•13 years ago
|
Keywords: checkin-needed
Whiteboard: tracemonkey
Comment 8•13 years ago
|
||
http://hg.mozilla.org/integration/mozilla-inbound/rev/e9245c104f52
Keywords: checkin-needed
Whiteboard: tracemonkey → tracemonkey [inbound]
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Comment 9•13 years ago
|
||
(Technically, bugs aren't FIXED until their patches make it into mozilla-central. That should happen for this bug in the next day or so, at the next merge from mozilla-inbound. Whoever merges mozilla-inbound to mozilla-central will resolve this bug.)
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Updated•13 years ago
|
Status: REOPENED → ASSIGNED
Comment 10•13 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/e9245c104f52
Status: ASSIGNED → RESOLVED
Closed: 13 years ago → 13 years ago
Resolution: --- → FIXED
Whiteboard: tracemonkey [inbound] → tracemonkey
Target Milestone: --- → mozilla8
You need to log in
before you can comment on or make changes to this bug.
Description
•