Remove JS_TRUE/JS_FALSE from xpconnect

RESOLVED FIXED in mozilla11

Status

()

Core
XPConnect
--
minor
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: Ms2ger, Assigned: Ms2ger)

Tracking

Trunk
mozilla11
Points:
---
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(3 attachments)

(Assignee)

Description

6 years ago
As requested
(Assignee)

Comment 1

6 years ago
Created attachment 573593 [details]
Script
(Assignee)

Comment 2

6 years ago
Created attachment 573594 [details] [diff] [review]
Part a: replace JS_TRUE/JS_FALSE with true/false

Generated with attachment 573593 [details].
Attachment #573594 - Flags: review?(bobbyholley+bmo)
(Assignee)

Comment 3

6 years ago
Created attachment 573595 [details] [diff] [review]
Part b: Reindent macro continuations
Attachment #573595 - Flags: review?(bobbyholley+bmo)
Comment on attachment 573594 [details] [diff] [review]
Part a: replace JS_TRUE/JS_FALSE with true/false

Looks good!

Please wait a few days before landing this so that tracer removal (bug 698201) can land, as so that we can make sure that write barriers doesn't need a backout.

r=bholley on this patch and any minor rebases required given the above.
Attachment #573594 - Flags: review?(bobbyholley+bmo) → review+
Comment on attachment 573595 [details] [diff] [review]
Part b: Reindent macro continuations

r=bholley.
Attachment #573595 - Flags: review?(bobbyholley+bmo) → review+
(Assignee)

Comment 6

6 years ago
Landed the two of them together:

https://hg.mozilla.org/mozilla-central/rev/3258ba3a269d
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Target Milestone: --- → mozilla11
You need to log in before you can comment on or make changes to this bug.