Closed Bug 598055 Opened 15 years ago Closed 14 years ago

MSVC jsscope.h(211) : warning C4307: '+' : integral constant overflow

Categories

(Core :: JavaScript Engine, defect)

Product:
Core
Component:
JavaScript Engine
Platform:
x86
Windows 2000
Type:
defect
Priority:
Not set
Severity:
trivial

Tracking

()

Status:
VERIFIED FIXED
Milestone:
mozilla2.0b8

People

(Reporter: sayrer, Assigned: jorendorff)

References

Details

(Whiteboard: [fixed by bug 604276])

> JS_STATIC_ASSERT(uint32(SHAPE_INVALID_SLOT + 1) == uint32(0)); Looks intentional. Should I disable C4307 using a pragma?
Could rewrite -- would JS_STATIC_ASSERT(uint32(SHAPE_INVALID_SLOT) == ~uint32(0)); pass without warning? Two's complement is here to stay! :-P /be
yeah, that should pass without warning.
OS: Mac OS X → Windows 2000
Version: unspecified → Trunk
Assigning to myself, I hate the warning.
Assignee: general → bugspam.Callek
Landed a patch silencing the warning in bug 604276.
See: http://hg.mozilla.org/tracemonkey/rev/9bda4099205e
Assignee: bugspam.Callek → jorendorff
Depends on: 604276
Whiteboard: [fixed-in-tracemonkey]
Severity: normal → trivial
Status: NEW → ASSIGNED
Flags: in-testsuite-
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1287267977.1287280239.25512.gz&fulltext=1 WINNT 5.2 mozilla-central build on 2010/10/16 15:26:17 had this warning. http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1289688821.1289696696.24803.gz&fulltext=1 WINNT 5.2 mozilla-central build on 2010/11/13 14:53:41 doesn't have it anymore. V.Fixed
Status: RESOLVED → VERIFIED
Whiteboard: [fixed-in-tracemonkey] → [fixed by bug 604276]
Target Milestone: --- → mozilla2.0b8
You need to log in before you can comment on or make changes to this bug.