Fix PRBool misuses found in a debug build

RESOLVED FIXED in mozilla8

Status

()

Core
Rewriting and Analysis
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: mwu, Assigned: mwu)

Tracking

(Depends on: 1 bug)

Trunk
mozilla8
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [inbound])

Attachments

(1 attachment)

Fix
1.71 KB, patch
(dormant account)
: review+
Details | Diff | Splinter Review
(Assignee)

Description

6 years ago
Created attachment 549634 [details] [diff] [review]
Fix

Found by boolcheck. Some of these were missed the first time due to my mozconfig not matching the tinderbox debug mozconfig.

bz:
 layout/generic/nsFrame.cpp                           |    4 ++--
 tools/trace-malloc/leaksoup.cpp                      |    2 +-

taras:
 startupcache/StartupCache.cpp                        |    2 +-
Attachment #549634 - Flags: review?(tglek)
Attachment #549634 - Flags: review?(bzbarsky)

Updated

6 years ago
Attachment #549634 - Flags: review?(tglek) → review+

Comment 1

6 years ago
Comment on attachment 549634 [details] [diff] [review]
Fix

r=me
Attachment #549634 - Flags: review?(bzbarsky) → review+
(Assignee)

Comment 2

6 years ago
http://hg.mozilla.org/integration/mozilla-inbound/rev/d11aaaf51059
Whiteboard: [inbound]
http://hg.mozilla.org/mozilla-central/rev/d11aaaf51059
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla8
You need to log in before you can comment on or make changes to this bug.