Closed
Bug 529980
Opened 16 years ago
Closed 16 years ago
Compiler errors in WebGLContextGL.cpp
Categories
(Core :: Graphics: CanvasWebGL, defect)
Tracking
()
RESOLVED
WORKSFORME
People
(Reporter: RyanVM, Unassigned)
Details
Compiling in VC8 SP1:
content\canvas\src\webglcontextgl.cpp(3238) : warning C4700: uninitialized local variable 'argPixelsLen' used
content\canvas\src\webglcontextgl.cpp(3053) : warning C4700: uninitialized local variable 'argPixelsLen' used
Regression from bug 526185?
http://hg.mozilla.org/mozilla-central/rev/e0a6e7510974
|
Reporter | |
Comment 1•16 years ago
|
||
Daniel, would reverting bug 518894 take care of this?
|
|
Reporter | |
Comment 2•16 years ago
|
||
Actually, bug 518894 was already reverted by bug 524042.
http://hg.mozilla.org/mozilla-central/rev/2fa0a16c7af7
So I guess the |!::JS_GetArrayLength(js.ctx, argPixelsObj, &argPixelsLen))| in the old code was how argPixelsLen was getting a value prior to bug 526185?
Forgive my c++ newbness, but would changing the variable declaration to |jsuint argPixelsLen = &argPixelsLen| fix this?
|
|
Reporter | |
Comment 3•16 years ago
|
||
This WFM now after today's WebGL landings.
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → WORKSFORME
You need to log in
before you can comment on or make changes to this bug.
Description
•