Closed Bug 59677 Opened 19 years ago Closed 18 years ago

Occurances of uninitialized variables being used before being set.

Categories

(Core :: DOM: Core & HTML, defect, P3)

defect

Tracking

()

RESOLVED FIXED
mozilla0.9.9

People

(Reporter: rich.burridge, Assigned: jst)

References

Details

(Keywords: dom1, Whiteboard: [HAVE FIX])

Attachments

(1 file)

For more details on this problem, see:

http://bugzilla.mozilla.org/show_bug.cgi?id=59652

This bug is just for the warnings in various source files in the DOM 
modules:

/dom/src/base/nsJSEnvironment.cpp
nsJSEnvironment.cpp:454: warning: `enum JSVersion newVersion' might be used
uninitialized in this function
nsJSEnvironment.cpp:462: warning: `enum JSVersion oldVersion' might be used
uninitialized in this function
nsJSEnvironment.cpp:570: warning: `enum JSVersion newVersion' might be used
uninitialized in this function
nsJSEnvironment.cpp:578: warning: `enum JSVersion oldVersion' might be used
uninitialized in this function
nsJSEnvironment.cpp:655: warning: `enum JSVersion newVersion' might be used
uninitialized in this function
nsJSEnvironment.cpp:662: warning: `enum JSVersion oldVersion' might be used
uninitialized in this function
----
Blocks: 59652
Moving DOM viewer bugs to future. I do not expect to get to these bugs any time
soon, so feel free to work on them.
Status: NEW → ASSIGNED
Target Milestone: --- → Future
wait just a second - this bug is supposed to be for the DOM, not the DOM viewer.
Assignee: alecf → jst
Status: ASSIGNED → NEW
Component: DOM Viewer → DOM Level 1
QA Contact: sspitzer → janc
Sorry, that's my fault. I wasn't sure which of the DOM modules these 
warning refered to.
Accepting...
Status: NEW → ASSIGNED
Keywords: dom1
Component: DOM Level 1 → DOM HTML
QA contact Update
QA Contact: janc → desale
Johnny, is the only thing to do here to use 
|JSVersion oldVersion = nsnull;|
or some such instead of 
|JSVersion oldVersion;|
?
It it's that simple, let me know and I'll whip up a patch.
JSVersion oldVersion = JSVERSION_DEFAULT;

Should shut down the warnings (which are compltely harmless)...
Updating QA contact to Shivakiran Tummala.
QA Contact: desale → stummala
Ohhh.. I had completely forgotten about this patch... r=theworld sr=theuniverse?
(goal: reduce the number of bugs you own by 0.5%)
What happened to this bug? It has an 8 months old patch and seems completely
forgotten...

BTW, currently (http://tinderbox.mozilla.org/SeaMonkey/warn1010759460.5770.html)
the warnings are:

dom/src/base/nsJSEnvironment.cpp:537
 `enum JSVersion newVersion' might be used uninitialized in this function

dom/src/base/nsJSEnvironment.cpp:545
 `enum JSVersion oldVersion' might be used uninitialized in this function

dom/src/base/nsJSEnvironment.cpp:655
 `enum JSVersion newVersion' might be used uninitialized in this function

dom/src/base/nsJSEnvironment.cpp:663
 `enum JSVersion oldVersion' might be used uninitialized in this function

dom/src/base/nsJSEnvironment.cpp:750
 `enum JSVersion oldVersion' might be used uninitialized in this function

jband, wanna stamp this with an sr=? :-)
Component: DOM HTML → DOM Other
Whiteboard: [HAVE FIX]
Target Milestone: Future → mozilla0.9.8
Target Milestone: mozilla0.9.8 → mozilla0.9.9
I no longer see any of these warnings.

http://tinderbox.mozilla.org/SeaMonkey/warn1013026020.23864.html (Wed, 06 Feb
2002 15:07 EST).
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.