vbo: replace assert(0) with unreachable()

Reviewed-by: Roland Scheidegger <sroland@vmware.com>

diff --git a/src/mesa/vbo/vbo_context.h b/src/mesa/vbo/vbo_context.h
index 04079b7d3299b58342a078ace6997735ff17cb9e..cd1cbd9641bca5cfa25396b78a40f1753b504110 100644 (file)
--- a/src/mesa/vbo/vbo_context.h
+++ b/src/mesa/vbo/vbo_context.h
@@ -154,7 +154,7 @@ vbo_draw_method(struct vbo_context *vbo, gl_draw_method method)
          ctx->Array._DrawArrays = vbo->save.inputs;
          break;
       default:
-         assert(0);
+         unreachable("Bad VBO drawing method");
       }
 
       ctx->NewDriverState |= ctx->DriverFlags.NewArray;
@@ -178,7 +178,7 @@ vbo_attrtype_to_integer_flag(GLenum format)
    case GL_UNSIGNED_INT64_ARB:
       return GL_TRUE;
    default:
-      assert(0);
+      unreachable("Bad vertex attribute type");
       return GL_FALSE;
    }
 }
@@ -195,7 +195,7 @@ vbo_attrtype_to_double_flag(GLenum format)
    case GL_DOUBLE:
       return GL_TRUE;
    default:
-      assert(0);
+      unreachable("Bad vertex attribute type");
       return GL_FALSE;
    }
 }
@@ -218,7 +218,7 @@ vbo_get_default_vals_as_union(GLenum format)
    case GL_UNSIGNED_INT:
       return (fi_type *)default_int;
    default:
-      assert(0);
+      unreachable("Bad vertex format");
       return NULL;
    }
 }

diff --git a/src/mesa/vbo/vbo_exec_array.c b/src/mesa/vbo/vbo_exec_array.c
index 024d4d6c80ebc5d1ba12e8db0a0e98386d2ebc36..16521ff46d74354ca0886ba6576f2f3abdf8b8e0 100644 (file)
--- a/src/mesa/vbo/vbo_exec_array.c
+++ b/src/mesa/vbo/vbo_exec_array.c
@@ -150,7 +150,7 @@ check_draw_elements_data(struct gl_context *ctx, GLsizei count,
          j = ((const GLuint *) elements)[i];
          break;
       default:
-         assert(0);
+         unreachable("Unexpected index buffer type");
       }
 
       /* check element j of each enabled array */

diff --git a/src/mesa/vbo/vbo_exec_draw.c b/src/mesa/vbo/vbo_exec_draw.c
index 3aff97e6f119fd579233f2616d524839225c5c0e..080d50c10e9659d13a05f539325f4ff7824132aa 100644 (file)
--- a/src/mesa/vbo/vbo_exec_draw.c
+++ b/src/mesa/vbo/vbo_exec_draw.c
@@ -159,7 +159,7 @@ vbo_copy_vertices(struct vbo_exec_context *exec)
    case PRIM_OUTSIDE_BEGIN_END:
       return 0;
    default:
-      assert(0);
+      unreachable("Unexpected primitive type");
       return 0;
    }
 }
@@ -220,7 +220,7 @@ vbo_exec_bind_arrays(struct gl_context *ctx)
       }
       break;
    default:
-      assert(0);
+      unreachable("Bad vertex program mode");
    }
 
    for (attr = 0; attr < VERT_ATTRIB_MAX ; attr++) {

diff --git a/src/mesa/vbo/vbo_save_api.c b/src/mesa/vbo/vbo_save_api.c
index 1c575448773df6bfd541bbd37ae4a69c0f0a8c19..49939ed3cef7cfa26b09ae7f31503af503ca634d 100644 (file)
--- a/src/mesa/vbo/vbo_save_api.c
+++ b/src/mesa/vbo/vbo_save_api.c
@@ -174,7 +174,7 @@ copy_vertices(struct gl_context *ctx,
                 sz * sizeof(GLfloat));
       return i;
    default:
-      assert(0);
+      unreachable("Unexpected primitive type");
       return 0;
    }
 }
@@ -675,8 +675,7 @@ copy_from_current(struct gl_context *ctx)
          save->attrptr[i][0] = save->current[i][0];
          break;
       case 0:
-         assert(0);
-         break;
+         unreachable("Unexpected vertex attribute size");
       }
    }
 }

diff --git a/src/mesa/vbo/vbo_save_draw.c b/src/mesa/vbo/vbo_save_draw.c
index ddf911700f86014e7c28c0d6cfe04d5efc1e7790..a63e0674d898047510b7b85750651ef66564efc0 100644 (file)
--- a/src/mesa/vbo/vbo_save_draw.c
+++ b/src/mesa/vbo/vbo_save_draw.c
@@ -194,7 +194,7 @@ bind_vertex_list(struct gl_context *ctx,
       }
       break;
    default:
-      assert(0);
+      unreachable("Bad vertex program mode");
    }
 
    for (attr = 0; attr < VERT_ATTRIB_MAX; attr++) {