Replace PRBool with boolean in IDL files

RESOLVED FIXED in mozilla9

Status

()

Core
General
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: mwu, Assigned: mwu)

Tracking

Trunk
mozilla9
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
Created attachment 550603 [details] [diff] [review]
Replace PRBools with boolean in IDLs

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

6 years ago
Attachment #550603 - Flags: review?(benjamin) → review+
(Assignee)

Comment 1

6 years ago
http://hg.mozilla.org/integration/mozilla-inbound/rev/f7dc94e1b75c
Whiteboard: [inbound]
http://hg.mozilla.org/mozilla-central/rev/f7dc94e1b75c
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla8
(Assignee)

Comment 3

6 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

6 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.