Closed Bug 136878 Opened 18 years ago Closed 18 years ago

Occurances of uninitialized variables being used before being set (in extensions/universalchardet)

Categories

(Core :: Internationalization, defect)

x86
Linux
defect
Not set

Tracking

()

RESOLVED FIXED

People

(Reporter: mozilla-bugs, Assigned: shanjian)

References

Details

(Keywords: intl)

Attachments

(1 file)

This bug is just for the warning in
extensions/universalchardet/src/nsUniversalDetector.cpp:

extensions/universalchardet/src/nsUniversalDetector.cpp:230
 `PRInt32 maxProber' might be used uninitialized in this function

In this case it seems that it's just the compiler not being smart enough, but in
any case it's nice not to have too many "xxx might be used uninitialized"
warnings - often they are pretty nasty bugs.
Bug 59652 is the meta-bug tracking the fight against these warnings.
Blocks: 59652
charset detection->shanjian
Assignee: yokoyama → shanjian
Attached patch patchSplinter Review
though it is just a false alarm, I will get rid of the warning. 
Status: NEW → ASSIGNED
Keywords: intl
QA Contact: ruixu → ylong
Comment on attachment 78732 [details] [diff] [review]
patch

r=brade
Attachment #78732 - Flags: review+
This was fixed by today's checkin (to bug 92806).
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Change QA contact to Frank.

Frank or Shanjian: can you please verify this code change? although I can verify
bug 92806.  Thanks!
QA Contact: ylong → ftang
You need to log in before you can comment on or make changes to this bug.