Closed
Bug 676465
Opened 13 years ago
Closed 13 years ago
Replace PRBool with boolean in IDL files
Categories
(Core :: General, defect)
Core
General
Tracking
()
RESOLVED
FIXED
mozilla9
People
(Reporter: mwu, Assigned: mwu)
References
Details
Attachments
(1 file)
30.62 KB,
patch
|
benjamin
:
review+
|
Details | Diff | Splinter Review |
I ran: find . ! -wholename "*nsprpub*" ! -wholename "*security/nss*" ! -wholename "*/.hg*" -type f -iname "*.idl" | xargs -n 1 sed -i "s/\([\ \t]\)PRBool/\1boolean/g" and then adjusted the resulting patch by hand to eliminate changes in c++ code and interfaces that actually want to use PRBool. (mostly some test interfaces)
Attachment #550603 -
Flags: review?(benjamin)
Updated•13 years ago
|
Attachment #550603 -
Flags: review?(benjamin) → review+
Assignee | ||
Comment 1•13 years ago
|
||
http://hg.mozilla.org/integration/mozilla-inbound/rev/f7dc94e1b75c
Whiteboard: [inbound]
http://hg.mozilla.org/mozilla-central/rev/f7dc94e1b75c
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla8
Assignee | ||
Comment 3•13 years ago
|
||
One hunk in this got lost after bug 672536 merged interfaces. I put it back in. http://hg.mozilla.org/integration/mozilla-inbound/rev/870b53893fd6
Comment 4•13 years ago
|
||
The missing hunk: http://hg.mozilla.org/mozilla-central/rev/870b53893fd6
Whiteboard: [inbound]
Target Milestone: mozilla8 → mozilla9
You need to log in
before you can comment on or make changes to this bug.
Description
•