Closed Bug 575708 Opened 14 years ago Closed 14 years ago

new member-variable booleans (e.g. mFinished) in bug 567426 are out of order in init list

Categories

(Core :: Graphics: ImageLib, defect)

defect
Not set
normal

Tracking

()

RESOLVED DUPLICATE of bug 577229

People

(Reporter: dholbert, Unassigned)

References

Details

Filing bug on these build warnings from bug 567426:
/scratch/work/builds/mozilla-central/mozilla-central.09-09-18.12-43/mozilla/modules/libpr0n/encoders/png/nsPNGEncoder.h: In constructor ‘nsPNGEncoder::nsPNGEncoder()’:
/scratch/work/builds/mozilla-central/mozilla-central.09-09-18.12-43/mozilla/modules/libpr0n/encoders/png/nsPNGEncoder.h:101: warning: ‘nsPNGEncoder::mImageBufferReadPoint’ will be initialized after
/scratch/work/builds/mozilla-central/mozilla-central.09-09-18.12-43/mozilla/modules/libpr0n/encoders/png/nsPNGEncoder.h:94: warning:   ‘PRPackedBool nsPNGEncoder::mFinished’
/scratch/work/builds/mozilla-central/mozilla-central.09-09-18.12-43/mozilla/modules/libpr0n/encoders/png/nsPNGEncoder.cpp:51: warning:   when initialized here
nsJPEGEncoder.cpp
/scratch/work/builds/mozilla-central/mozilla-central.09-09-18.12-43/mozilla/modules/libpr0n/encoders/jpeg/nsJPEGEncoder.h: In constructor ‘nsJPEGEncoder::nsJPEGEncoder()’:
/scratch/work/builds/mozilla-central/mozilla-central.09-09-18.12-43/mozilla/modules/libpr0n/encoders/jpeg/nsJPEGEncoder.h:98: warning: ‘nsJPEGEncoder::mImageBufferReadPoint’ will be initialized after
/scratch/work/builds/mozilla-central/mozilla-central.09-09-18.12-43/mozilla/modules/libpr0n/encoders/jpeg/nsJPEGEncoder.h:91: warning:   ‘PRPackedBool nsJPEGEncoder::mFinished’
/scratch/work/builds/mozilla-central/mozilla-central.09-09-18.12-43/mozilla/modules/libpr0n/encoders/jpeg/nsJPEGEncoder.cpp:56: warning:   when initialized here

khuey, mind fixing?  (just need to reorder the constructors' init lists so that they match the ordering of member variables in the class definition)
Yeah, I'll take this.
Status: NEW → ASSIGNED
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.