clang-cl warning: fields initialized out of order in platform-win32.cc

RESOLVED FIXED in Firefox 51

Status

()

RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: froydnj, Assigned: froydnj)

Tracking

unspecified
mozilla51
All
Windows
Points:
---

Firefox Tracking Flags

(firefox51 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

2 years ago
clang-cl complains thusly:

17:05.88 c:/m-c/tools/profiler/core/platform-win32.cc(88,9):  warning: field 'interval_' will be initialized after field 'sampler_' [-Wreorder]
17:05.88       , interval_(interval)
17:05.88         ^
(Assignee)

Comment 1

2 years ago
Created attachment 8789762 [details] [diff] [review]
fix member initialization order in platform-win32.cc
Attachment #8789762 - Flags: review?(bgirard)
(Assignee)

Updated

2 years ago
Assignee: nobody → nfroyd
Attachment #8789762 - Flags: review?(bgirard) → review+

Comment 2

2 years ago
Pushed by nfroyd@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/9c33c831031f
fix member initialization order in platform-win32.cc; r=BenWa

Comment 3

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/9c33c831031f
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox51: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla51
You need to log in before you can comment on or make changes to this bug.