dri/swrast: minor cosmetic

diff --git a/src/mesa/drivers/dri/swrast/swrast.c b/src/mesa/drivers/dri/swrast/swrast.c
index c4dba591987de19d17fd933a0619ac35650cedd9..f41e331809497b9da580fadaa6e9de2bdb896940 100644 (file)
--- a/src/mesa/drivers/dri/swrast/swrast.c
+++ b/src/mesa/drivers/dri/swrast/swrast.c
@@ -326,7 +326,7 @@ swrast_alloc_back_storage(GLcontext *ctx, struct gl_renderbuffer *rb,
 
     _mesa_free(rb->Data);
 
-    (void) swrast_alloc_front_storage(ctx, rb, internalFormat, width, height);
+    swrast_alloc_front_storage(ctx, rb, internalFormat, width, height);
 
     rb->Data = _mesa_malloc(height * xrb->pitch);
 
@@ -341,59 +341,61 @@ swrast_new_renderbuffer(const GLvisual *visual, GLboolean front)
 
     TRACE;
 
-    if (xrb) {
-       _mesa_init_renderbuffer(&xrb->Base, 0);
+    if (!xrb)
+       return NULL;
 
-       pixel_format = choose_pixel_format(visual);
+    _mesa_init_renderbuffer(&xrb->Base, 0);
 
-       xrb->Base.Delete = swrast_delete_renderbuffer;
-       if (front) {
-           xrb->Base.AllocStorage = swrast_alloc_front_storage;
-           swrast_set_span_funcs_front(xrb, pixel_format);
-       }
-       else {
-           xrb->Base.AllocStorage = swrast_alloc_back_storage;
-           swrast_set_span_funcs_back(xrb, pixel_format);
-       }
+    pixel_format = choose_pixel_format(visual);
 
-       switch (pixel_format) {
-       case PF_A8R8G8B8:
-           xrb->Base.InternalFormat = GL_RGBA;
-           xrb->Base._BaseFormat = GL_RGBA;
-           xrb->Base.DataType = GL_UNSIGNED_BYTE;
-           xrb->Base.RedBits   = 8 * sizeof(GLubyte);
-           xrb->Base.GreenBits = 8 * sizeof(GLubyte);
-           xrb->Base.BlueBits  = 8 * sizeof(GLubyte);
-           xrb->Base.AlphaBits = 8 * sizeof(GLubyte);
-           break;
-       case PF_R5G6B5:
-           xrb->Base.InternalFormat = GL_RGB;
-           xrb->Base._BaseFormat = GL_RGB;
-           xrb->Base.DataType = GL_UNSIGNED_BYTE;
-           xrb->Base.RedBits   = 5 * sizeof(GLubyte);
-           xrb->Base.GreenBits = 6 * sizeof(GLubyte);
-           xrb->Base.BlueBits  = 5 * sizeof(GLubyte);
-           xrb->Base.AlphaBits = 0;
-           break;
-       case PF_R3G3B2:
-           xrb->Base.InternalFormat = GL_RGB;
-           xrb->Base._BaseFormat = GL_RGB;
-           xrb->Base.DataType = GL_UNSIGNED_BYTE;
-           xrb->Base.RedBits   = 3 * sizeof(GLubyte);
-           xrb->Base.GreenBits = 3 * sizeof(GLubyte);
-           xrb->Base.BlueBits  = 2 * sizeof(GLubyte);
-           xrb->Base.AlphaBits = 0;
-           break;
-       case PF_CI8:
-           xrb->Base.InternalFormat = GL_COLOR_INDEX8_EXT;
-           xrb->Base._BaseFormat = GL_COLOR_INDEX;
-           xrb->Base.DataType = GL_UNSIGNED_BYTE;
-           xrb->Base.IndexBits = 8 * sizeof(GLubyte);
-           break;
-       default:
-           return NULL;
-       }
+    xrb->Base.Delete = swrast_delete_renderbuffer;
+    if (front) {
+       xrb->Base.AllocStorage = swrast_alloc_front_storage;
+       swrast_set_span_funcs_front(xrb, pixel_format);
+    }
+    else {
+       xrb->Base.AllocStorage = swrast_alloc_back_storage;
+       swrast_set_span_funcs_back(xrb, pixel_format);
     }
+
+    switch (pixel_format) {
+    case PF_A8R8G8B8:
+       xrb->Base.InternalFormat = GL_RGBA;
+       xrb->Base._BaseFormat = GL_RGBA;
+       xrb->Base.DataType = GL_UNSIGNED_BYTE;
+       xrb->Base.RedBits   = 8 * sizeof(GLubyte);
+       xrb->Base.GreenBits = 8 * sizeof(GLubyte);
+       xrb->Base.BlueBits  = 8 * sizeof(GLubyte);
+       xrb->Base.AlphaBits = 8 * sizeof(GLubyte);
+       break;
+    case PF_R5G6B5:
+       xrb->Base.InternalFormat = GL_RGB;
+       xrb->Base._BaseFormat = GL_RGB;
+       xrb->Base.DataType = GL_UNSIGNED_BYTE;
+       xrb->Base.RedBits   = 5 * sizeof(GLubyte);
+       xrb->Base.GreenBits = 6 * sizeof(GLubyte);
+       xrb->Base.BlueBits  = 5 * sizeof(GLubyte);
+       xrb->Base.AlphaBits = 0;
+       break;
+    case PF_R3G3B2:
+       xrb->Base.InternalFormat = GL_RGB;
+       xrb->Base._BaseFormat = GL_RGB;
+       xrb->Base.DataType = GL_UNSIGNED_BYTE;
+       xrb->Base.RedBits   = 3 * sizeof(GLubyte);
+       xrb->Base.GreenBits = 3 * sizeof(GLubyte);
+       xrb->Base.BlueBits  = 2 * sizeof(GLubyte);
+       xrb->Base.AlphaBits = 0;
+       break;
+    case PF_CI8:
+       xrb->Base.InternalFormat = GL_COLOR_INDEX8_EXT;
+       xrb->Base._BaseFormat = GL_COLOR_INDEX;
+       xrb->Base.DataType = GL_UNSIGNED_BYTE;
+       xrb->Base.IndexBits = 8 * sizeof(GLubyte);
+       break;
+    default:
+       return NULL;
+    }
+
     return xrb;
 }