Last Comment Bug 678256 - Init list order build warning for nsBuiltinDecoderStateMachine.h / nsBuiltinDecoderStateMachine.cpp
: Init list order build warning for nsBuiltinDecoderStateMachine.h / nsBuiltinD...
Status: RESOLVED FIXED
[build_warning]
:
Product: Core
Classification: Components
Component: Audio/Video (show other bugs)
: Trunk
: All All
: -- normal (vote)
: mozilla8
Assigned To: Daniel Holbert [:dholbert]
:
: Maire Reavy [:mreavy]
Mentors:
http://tinderbox.mozilla.org/showlog....
Depends on:
Blocks: buildwarning
  Show dependency treegraph
 
Reported: 2011-08-11 11:30 PDT by Daniel Holbert [:dholbert]
Modified: 2011-08-12 07:59 PDT (History)
1 user (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
fix (1.29 KB, patch)
2011-08-11 11:32 PDT, Daniel Holbert [:dholbert]
cpearce: review+
Details | Diff | Splinter Review

Description Daniel Holbert [:dholbert] 2011-08-11 11:30:47 PDT
Filing bug on this build warning:
{
nsBuiltinDecoderStateMachine.h: In constructor 'nsBuiltinDecoderStateMachine::nsBuiltinDecoderStateMachine(nsBuiltinDecoder*, nsBuiltinDecoderReader*)':
nsBuiltinDecoderStateMachine.h:622:32: warning: 'nsBuiltinDecoderStateMachine::mEventManager' will be initialized after
nsBuiltinDecoderStateMachine.h:600:16: warning:   'PRPackedBool nsBuiltinDecoderStateMachine::mIsRunning'
nsBuiltinDecoderStateMachine.cpp:196:1: warning:   when initialized here
}

Patch coming up.
Comment 1 Daniel Holbert [:dholbert] 2011-08-11 11:32:12 PDT
Created attachment 552448 [details] [diff] [review]
fix
Comment 2 Daniel Holbert [:dholbert] 2011-08-11 15:24:18 PDT
http://hg.mozilla.org/integration/mozilla-inbound/rev/84a485705173
Comment 3 Matt Brubeck (:mbrubeck) 2011-08-12 07:59:36 PDT
https://hg.mozilla.org/mozilla-central/rev/84a485705173

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