Closed Bug 1194985 Opened 9 years ago Closed 9 years ago

avoid shift overflows in devtools server tests

Categories

(DevTools :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED DUPLICATE of bug 1198980

People

(Reporter: froydnj, Assigned: froydnj)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

      No description provided.
clang complains about shifting quantities beyond the width of their
type, since that is undefined behavior.  Let's fix the problematic
occurrences by adding some casts prior to shifting.
Attachment #8648376 - Flags: review?(nfitzgerald)
Comment on attachment 8648376 [details] [diff] [review]
avoid shift overflows in devtools gtests

Review of attachment 8648376 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks!
Attachment #8648376 - Flags: review?(nfitzgerald) → review+
Backed out because the test was apparently depending on truncation of the values, or something?

https://hg.mozilla.org/integration/mozilla-inbound/rev/a30421f03bc5
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → DUPLICATE
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: