Closed Bug 743813 Opened 8 years ago Closed 8 years ago

Fix the warnings generated by WebGL activeTexture

Categories

(Core :: Canvas: WebGL, defect)

Product:
Core
Component:
Canvas: WebGL
Type:
defect
Priority:
Not set

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla14

People

(Reporter: bjacob, Assigned: bjacob)

Details

Attachments

(1 file)

fix the warning
1.22 KB, patch
jgilbert
: review+
Details | Diff | Splinter Review
Attached patch fix the warning — — Splinter Review 
Current warnings are bogus/misleading.

+ some minor style/code enhancements.
Attachment #613389 - Flags: review?(jgilbert)
Comment on attachment 613389 [details] [diff] [review]
fix the warning

Review of attachment 613389 [details] [diff] [review]:
-----------------------------------------------------------------

::: content/canvas/src/WebGLContextGL.cpp
@@ +137,5 @@
> +    if (texture < LOCAL_GL_TEXTURE0 ||
> +        texture >= LOCAL_GL_TEXTURE0 + mGLMaxTextureUnits)
> +    {
> +        return ErrorInvalidEnum(
> +          "ActiveTexture: texture unit %d out of range. "

Should be 4-space not 2-space.
Attachment #613389 - Flags: review?(jgilbert) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/8a59c745c563
Assignee: nobody → bjacob
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla14
fix warning causing error due to WARNINGS_AS_ERRORS:

http://hg.mozilla.org/integration/mozilla-inbound/rev/b1bed3ae0476
You need to log in before you can comment on or make changes to this bug.