Closed Bug 953145 Opened 11 years ago Closed 10 years ago

js/src/jit/FixedList.h: warning: field 'length_' will be initialized after field 'list_' [-Wreorder]

Categories

(Core :: JavaScript Engine: JIT, defect)

Product:
Core
Component:
JavaScript Engine: JIT
Platform:
x86
macOS
Type:
defect
Priority:
Not set
Severity:
trivial

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla29

People

(Reporter: cpeterson, Assigned: cpeterson)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

fix-Wreorder-warning.patch
844 bytes, patch
nbp
: review+
Details | Diff | Splinter Review 
Bug 950716 introduced a -Wreorder warning for clang on OS X:

Warning: -Wreorder in mozilla/inbound/js/src/jit/FixedList.h: field 'length_' will be initialized after field 'list_'
mozilla/inbound/js/src/jit/FixedList.h:31:9: warning: field 'length_' will be initialized after field 'list_' [-Wreorder]
       : length_(0), list_(nullptr)
Attachment #8351468 - Flags: review?(nicolas.b.pierron)
Attachment #8351468 - Flags: review?(nicolas.b.pierron) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/07f7597000b7
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Fixed = m-c
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Yeah, but it was fixed by a no-bug push, so unless I happened to see firebot mentioning it being marked fixed and was curious, I wouldn't have known to mark it when I merged the no-bug push...
https://hg.mozilla.org/mozilla-central/rev/07f7597000b7
Status: REOPENED → RESOLVED
Closed: 10 years ago → 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla29
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: