Closed Bug 695324 Opened 8 years ago Closed 8 years ago

Fix precedence problem found with clang warning

Categories

(Core :: DOM: Core & HTML, defect)

Product:
Core
Component:
DOM: Core & HTML
Platform:
x86
macOS
Type:
defect
Priority:
Not set

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla10

People

(Reporter: jrmuizel, Assigned: jrmuizel)

Details

Attachments

(1 file)

Fix precedence problem found with clang warning
877 bytes, patch
mounir
: review+
Details | Diff | Splinter Review 
'*' has higher precedence than '?' so bracket the expression
so we get what we want.
Attachment #567743 - Flags: review?(mounir)
Attachment #567743 - Flags: review?(mounir) → review+
Version: unspecified → Trunk
https://hg.mozilla.org/mozilla-central/rev/344e0e0cabe4
Assignee: nobody → jmuizelaar
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla10
Ouch.  Another reason why using moz_malloc_usable_size is better than computing a size!  Bug 698968 FTW!
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.