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...
Product: Core
Classification: Components
Component: IPC (show other bugs)
: Trunk
: x86_64 Linux
: -- normal (vote)
: mozilla12
Assigned To: Daniel Holbert [:dholbert]
: [PTO to Dec5] Bill McCloskey (:billm)
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:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

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
Comment 3 Ed Morley [:emorley] 2012-01-31 07:03:11 PST

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