Bug 1313541 - ClearBuffer style fixes. - r=ethlin draft
authorJeff Gilbert (:jgilbert) <jgilbert@mozilla.com>
Thu, 13 Oct 2016 16:25:11 -0700
changeset 430651 75da32f0c212fe4a1463d403cb6244dcd5f3d359
parent 430650 a7f739b1087ca5013899c3aef521e121e8a429c1
child 430652 37a99342024fa252c385939505c3d210d8f82f85
push id33862
push userbmo:jgilbert@mozilla.com
push dateFri, 28 Oct 2016 02:14:30 +0000
reviewersethlin
bugs1313541
milestone52.0a1
Bug 1313541 - ClearBuffer style fixes. - r=ethlin MozReview-Commit-ID: CZbWDAepxlX
dom/canvas/WebGL2ContextMRTs.cpp
--- a/dom/canvas/WebGL2ContextMRTs.cpp
+++ b/dom/canvas/WebGL2ContextMRTs.cpp
@@ -5,53 +5,56 @@
 
 #include "WebGL2Context.h"
 
 #include "GLContext.h"
 #include "WebGLFramebuffer.h"
 
 namespace mozilla {
 
-bool WebGL2Context::ValidateClearBuffer(const char* info, GLenum buffer, GLint drawbuffer, size_t elemCount)
+bool
+WebGL2Context::ValidateClearBuffer(const char* info, GLenum buffer, GLint drawbuffer,
+                                   size_t elemCount)
 {
-    size_t requiredElements = -1;
-    GLint maxDrawbuffer = -1;
+    size_t requiredElements;
+    GLint maxDrawbuffer;
     switch (buffer) {
-      case LOCAL_GL_COLOR:
-      case LOCAL_GL_FRONT:
-      case LOCAL_GL_BACK:
-      case LOCAL_GL_LEFT:
-      case LOCAL_GL_RIGHT:
-      case LOCAL_GL_FRONT_AND_BACK:
+    case LOCAL_GL_COLOR:
+    case LOCAL_GL_FRONT:
+    case LOCAL_GL_BACK:
+    case LOCAL_GL_LEFT:
+    case LOCAL_GL_RIGHT:
+    case LOCAL_GL_FRONT_AND_BACK:
           requiredElements = 4;
           maxDrawbuffer = mGLMaxDrawBuffers - 1;
           break;
 
-      case LOCAL_GL_DEPTH:
-      case LOCAL_GL_STENCIL:
+    case LOCAL_GL_DEPTH:
+    case LOCAL_GL_STENCIL:
           requiredElements = 1;
           maxDrawbuffer = 0;
           break;
 
-      default:
+    default:
           ErrorInvalidEnumInfo(info, buffer);
           return false;
     }
 
     if (drawbuffer < 0 || drawbuffer > maxDrawbuffer) {
         ErrorInvalidValue("%s: invalid drawbuffer %d. This buffer only supports drawbuffer values between 0 and %d",
                           info, drawbuffer, maxDrawbuffer);
         return false;
     }
 
     if (elemCount < requiredElements) {
         ErrorInvalidValue("%s: Not enough elements. Require %u. Given %u.",
                           info, requiredElements, elemCount);
         return false;
     }
+
     return true;
 }
 
 // Common base functionality. These a good candidates to be moved into WebGLContextUnchecked.cpp
 void
 WebGL2Context::ClearBufferiv_base(GLenum buffer, GLint drawbuffer, const GLint* value)
 {
     const char funcName[] = "clearBufferiv";