Closed Bug 1577066 Opened 3 months ago Closed 3 months ago

Nursery.cpp contains an implicit integer-to-float conversion that changes value, triggering a warning with new-enough clang

Categories

(Core :: JavaScript: GC, task)

task
Not set

Tracking

()

RESOLVED FIXED
mozilla70
Tracking Status
firefox70 --- fixed

People

(Reporter: Waldo, Assigned: Waldo)

Details

Attachments

(1 file)

The problem and fix here are a little fugly, but I don't know there's much clean to do about it.

Pushed by jwalden@mit.edu:
https://hg.mozilla.org/integration/autoland/rev/fcaf8571985f
Fix an implicit-conversion-changes-value warning with new-enough clang in Nursery.cpp by doing a less-than comparison with the actually-representable next higher value.  r=jonco
Pushed by jwalden@mit.edu:
https://hg.mozilla.org/integration/autoland/rev/0948113e9229
Fix an implicit-conversion-changes-value warning with new-enough clang in Nursery.cpp by doing a less-than comparison with the actually-representable next higher value.  r=jonco
Status: ASSIGNED → RESOLVED
Closed: 3 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla70
Flags: needinfo?(jwalden)
You need to log in before you can comment on or make changes to this bug.