Last Comment Bug 722424 - Build warning: ‘mozilla::ipc::SyncChannel::mTimeoutMs’ will be initialized after...‘bool mozilla::ipc::SyncChannel::mInTimeoutSecondHalf’
: Build warning: ‘mozilla::ipc::SyncChannel::mTimeoutMs’ will be initialized af...
Status: RESOLVED FIXED
:
Product: Core
Classification: Components
Component: IPC (show other bugs)
: Trunk
: x86_64 Linux
: -- normal (vote)
: mozilla12
Assigned To: Daniel Holbert [:dholbert]
:
:
Mentors:
Depends on:
Blocks: buildwarning 679240
  Show dependency treegraph
 
Reported: 2012-01-30 12:05 PST by Daniel Holbert [:dholbert]
Modified: 2012-01-31 07:03 PST (History)
1 user (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
fix v1: reorder init list (890 bytes, patch)
2012-01-30 12:06 PST, Daniel Holbert [:dholbert]
jmathies: review+
Details | Diff | Splinter Review

Description Daniel Holbert [:dholbert] 2012-01-30 12:05:01 PST
Build warning:
{
SyncChannel.h: In constructor ‘mozilla::ipc::SyncChannel::SyncChannel(mozilla::ipc::SyncChannel::SyncListener*)’:
SyncChannel.h:198:11: warning: ‘mozilla::ipc::SyncChannel::mTimeoutMs’ will be initialized after [-Wreorder]
SyncChannel.h:197:10: warning:   ‘bool mozilla::ipc::SyncChannel::mInTimeoutSecondHalf’ [-Wreorder]
SyncChannel.cpp:59:1: warning:   when initialized here [-Wreorder]
}
Comment 1 Daniel Holbert [:dholbert] 2012-01-30 12:06:42 PST
Created attachment 592813 [details] [diff] [review]
fix v1: reorder init list
Comment 2 Daniel Holbert [:dholbert] 2012-01-30 13:08:02 PST
https://hg.mozilla.org/integration/mozilla-inbound/rev/9f162faf97d2
Comment 3 Ed Morley [:emorley] 2012-01-31 07:03:11 PST
https://hg.mozilla.org/mozilla-central/rev/9f162faf97d2

Note You need to log in before you can comment on or make changes to this bug.