| Keith Whitwell | cab974c | 2000-12-26 05:09:27 +0000 | [diff] [blame] | 1 | /* |
| 2 | * Mesa 3-D graphics library |
| Brian | a3c3bc9 | 2007-08-20 12:55:34 +0100 | [diff] [blame] | 3 | * Version: 7.1 |
| Keith Whitwell | cab974c | 2000-12-26 05:09:27 +0000 | [diff] [blame] | 4 | * |
| Brian | 37ece4d | 2007-07-08 09:20:42 -0600 | [diff] [blame] | 5 | * Copyright (C) 1999-2007 Brian Paul All Rights Reserved. |
| Keith Whitwell | cab974c | 2000-12-26 05:09:27 +0000 | [diff] [blame] | 6 | * |
| 7 | * Permission is hereby granted, free of charge, to any person obtaining a |
| 8 | * copy of this software and associated documentation files (the "Software"), |
| 9 | * to deal in the Software without restriction, including without limitation |
| 10 | * the rights to use, copy, modify, merge, publish, distribute, sublicense, |
| 11 | * and/or sell copies of the Software, and to permit persons to whom the |
| 12 | * Software is furnished to do so, subject to the following conditions: |
| 13 | * |
| 14 | * The above copyright notice and this permission notice shall be included |
| 15 | * in all copies or substantial portions of the Software. |
| 16 | * |
| 17 | * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS |
| 18 | * OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, |
| 19 | * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL |
| 20 | * BRIAN PAUL BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN |
| 21 | * AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN |
| 22 | * CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE. |
| 23 | */ |
| 24 | |
| 25 | #include "glheader.h" |
| Brian Paul | 8446d1b | 2001-01-02 21:40:57 +0000 | [diff] [blame] | 26 | #include "api_validate.h" |
| Brian Paul | 434ec3a | 2009-08-12 13:46:16 -0600 | [diff] [blame] | 27 | #include "bufferobj.h" |
| Keith Whitwell | cab974c | 2000-12-26 05:09:27 +0000 | [diff] [blame] | 28 | #include "context.h" |
| Brian Paul | 3c63452 | 2002-10-24 23:57:19 +0000 | [diff] [blame] | 29 | #include "imports.h" |
| Keith Whitwell | cab974c | 2000-12-26 05:09:27 +0000 | [diff] [blame] | 30 | #include "mtypes.h" |
| Brian Paul | 8446d1b | 2001-01-02 21:40:57 +0000 | [diff] [blame] | 31 | #include "state.h" |
| Keith Whitwell | cab974c | 2000-12-26 05:09:27 +0000 | [diff] [blame] | 32 | |
| Keith Whitwell | 58e9917 | 2001-01-05 02:26:48 +0000 | [diff] [blame] | 33 | |
| Brian Paul | bb1fb2a | 2009-05-06 12:37:10 -0600 | [diff] [blame] | 34 | |
| 35 | /** |
| 36 | * \return number of bytes in array [count] of type. |
| 37 | */ |
| 38 | static GLsizei |
| 39 | index_bytes(GLenum type, GLsizei count) |
| 40 | { |
| 41 | if (type == GL_UNSIGNED_INT) { |
| 42 | return count * sizeof(GLuint); |
| 43 | } |
| 44 | else if (type == GL_UNSIGNED_BYTE) { |
| 45 | return count * sizeof(GLubyte); |
| 46 | } |
| 47 | else { |
| 48 | ASSERT(type == GL_UNSIGNED_SHORT); |
| 49 | return count * sizeof(GLushort); |
| 50 | } |
| 51 | } |
| 52 | |
| 53 | |
| Brian | d8c6719 | 2007-08-20 13:12:20 +0100 | [diff] [blame] | 54 | /** |
| 55 | * Find the max index in the given element/index buffer |
| 56 | */ |
| 57 | static GLuint |
| 58 | max_buffer_index(GLcontext *ctx, GLuint count, GLenum type, |
| 59 | const void *indices, |
| 60 | struct gl_buffer_object *elementBuf) |
| 61 | { |
| 62 | const GLubyte *map = NULL; |
| 63 | GLuint max = 0; |
| José Fonseca | 37f2117 | 2009-06-15 19:17:07 +0100 | [diff] [blame] | 64 | GLuint i; |
| Brian | d8c6719 | 2007-08-20 13:12:20 +0100 | [diff] [blame] | 65 | |
| Brian Paul | 434ec3a | 2009-08-12 13:46:16 -0600 | [diff] [blame] | 66 | if (_mesa_is_bufferobj(elementBuf)) { |
| Brian | d8c6719 | 2007-08-20 13:12:20 +0100 | [diff] [blame] | 67 | /* elements are in a user-defined buffer object. need to map it */ |
| Brian Paul | 88af3f8 | 2009-05-06 12:27:38 -0600 | [diff] [blame] | 68 | map = ctx->Driver.MapBuffer(ctx, GL_ELEMENT_ARRAY_BUFFER, |
| 69 | GL_READ_ONLY, elementBuf); |
| Brian | d8c6719 | 2007-08-20 13:12:20 +0100 | [diff] [blame] | 70 | /* Actual address is the sum of pointers */ |
| 71 | indices = (const GLvoid *) ADD_POINTERS(map, (const GLubyte *) indices); |
| 72 | } |
| 73 | |
| 74 | if (type == GL_UNSIGNED_INT) { |
| 75 | for (i = 0; i < count; i++) |
| 76 | if (((GLuint *) indices)[i] > max) |
| 77 | max = ((GLuint *) indices)[i]; |
| 78 | } |
| 79 | else if (type == GL_UNSIGNED_SHORT) { |
| 80 | for (i = 0; i < count; i++) |
| 81 | if (((GLushort *) indices)[i] > max) |
| 82 | max = ((GLushort *) indices)[i]; |
| 83 | } |
| 84 | else { |
| 85 | ASSERT(type == GL_UNSIGNED_BYTE); |
| 86 | for (i = 0; i < count; i++) |
| 87 | if (((GLubyte *) indices)[i] > max) |
| 88 | max = ((GLubyte *) indices)[i]; |
| 89 | } |
| 90 | |
| 91 | if (map) { |
| Brian Paul | 88af3f8 | 2009-05-06 12:27:38 -0600 | [diff] [blame] | 92 | ctx->Driver.UnmapBuffer(ctx, GL_ELEMENT_ARRAY_BUFFER_ARB, elementBuf); |
| Brian | d8c6719 | 2007-08-20 13:12:20 +0100 | [diff] [blame] | 93 | } |
| 94 | |
| 95 | return max; |
| 96 | } |
| 97 | |
| Brian Paul | 88af3f8 | 2009-05-06 12:27:38 -0600 | [diff] [blame] | 98 | |
| 99 | /** |
| 100 | * Check if OK to render by examining framebuffer status and vertex arrays. |
| 101 | */ |
| Alan Hourihane | 263b96e | 2009-01-15 11:51:39 +0000 | [diff] [blame] | 102 | static GLboolean |
| 103 | check_valid_to_render(GLcontext *ctx, char *function) |
| 104 | { |
| 105 | if (ctx->DrawBuffer->_Status != GL_FRAMEBUFFER_COMPLETE_EXT) { |
| 106 | _mesa_error(ctx, GL_INVALID_FRAMEBUFFER_OPERATION_EXT, |
| 107 | "glDraw%s(incomplete framebuffer)", function); |
| 108 | return GL_FALSE; |
| 109 | } |
| 110 | |
| Brian Paul | 97dd2dd | 2009-03-02 12:27:16 -0700 | [diff] [blame] | 111 | #if FEATURE_es2_glsl |
| 112 | /* For ES2, we can draw if any vertex array is enabled (and we should |
| 113 | * always have a vertex program/shader). |
| 114 | */ |
| 115 | if (ctx->Array.ArrayObj->_Enabled == 0x0 || !ctx->VertexProgram._Current) |
| 116 | return GL_FALSE; |
| 117 | #else |
| 118 | /* For regular OpenGL, only draw if we have vertex positions (regardless |
| 119 | * of whether or not we have a vertex program/shader). |
| 120 | */ |
| 121 | if (!ctx->Array.ArrayObj->Vertex.Enabled && |
| Alan Hourihane | 263b96e | 2009-01-15 11:51:39 +0000 | [diff] [blame] | 122 | !ctx->Array.ArrayObj->VertexAttrib[0].Enabled) |
| 123 | return GL_FALSE; |
| Brian Paul | 97dd2dd | 2009-03-02 12:27:16 -0700 | [diff] [blame] | 124 | #endif |
| Alan Hourihane | 263b96e | 2009-01-15 11:51:39 +0000 | [diff] [blame] | 125 | |
| 126 | return GL_TRUE; |
| 127 | } |
| Brian | d8c6719 | 2007-08-20 13:12:20 +0100 | [diff] [blame] | 128 | |
| Brian Paul | 88af3f8 | 2009-05-06 12:27:38 -0600 | [diff] [blame] | 129 | |
| 130 | /** |
| 131 | * Error checking for glDrawElements(). Includes parameter checking |
| 132 | * and VBO bounds checking. |
| 133 | * \return GL_TRUE if OK to render, GL_FALSE if error found |
| 134 | */ |
| Keith Whitwell | cab974c | 2000-12-26 05:09:27 +0000 | [diff] [blame] | 135 | GLboolean |
| 136 | _mesa_validate_DrawElements(GLcontext *ctx, |
| Gareth Hughes | 22144ab | 2001-03-12 00:48:37 +0000 | [diff] [blame] | 137 | GLenum mode, GLsizei count, GLenum type, |
| Keith Whitwell | cab974c | 2000-12-26 05:09:27 +0000 | [diff] [blame] | 138 | const GLvoid *indices) |
| 139 | { |
| Gareth Hughes | 22144ab | 2001-03-12 00:48:37 +0000 | [diff] [blame] | 140 | ASSERT_OUTSIDE_BEGIN_END_WITH_RETVAL(ctx, GL_FALSE); |
| Keith Whitwell | cab974c | 2000-12-26 05:09:27 +0000 | [diff] [blame] | 141 | |
| 142 | if (count <= 0) { |
| 143 | if (count < 0) |
| Brian Paul | 0883634 | 2001-03-03 20:33:27 +0000 | [diff] [blame] | 144 | _mesa_error(ctx, GL_INVALID_VALUE, "glDrawElements(count)" ); |
| Keith Whitwell | cab974c | 2000-12-26 05:09:27 +0000 | [diff] [blame] | 145 | return GL_FALSE; |
| 146 | } |
| 147 | |
| Brian Paul | 464bc3b | 2003-04-21 15:04:30 +0000 | [diff] [blame] | 148 | if (mode > GL_POLYGON) { |
| Brian Paul | a2b9bad | 2003-11-10 19:08:37 +0000 | [diff] [blame] | 149 | _mesa_error(ctx, GL_INVALID_ENUM, "glDrawElements(mode)" ); |
| Keith Whitwell | cab974c | 2000-12-26 05:09:27 +0000 | [diff] [blame] | 150 | return GL_FALSE; |
| 151 | } |
| 152 | |
| Gareth Hughes | 22144ab | 2001-03-12 00:48:37 +0000 | [diff] [blame] | 153 | if (type != GL_UNSIGNED_INT && |
| 154 | type != GL_UNSIGNED_BYTE && |
| Keith Whitwell | cab974c | 2000-12-26 05:09:27 +0000 | [diff] [blame] | 155 | type != GL_UNSIGNED_SHORT) |
| 156 | { |
| Brian Paul | 0883634 | 2001-03-03 20:33:27 +0000 | [diff] [blame] | 157 | _mesa_error(ctx, GL_INVALID_ENUM, "glDrawElements(type)" ); |
| Keith Whitwell | cab974c | 2000-12-26 05:09:27 +0000 | [diff] [blame] | 158 | return GL_FALSE; |
| 159 | } |
| 160 | |
| 161 | if (ctx->NewState) |
| Brian Paul | a2b9bad | 2003-11-10 19:08:37 +0000 | [diff] [blame] | 162 | _mesa_update_state(ctx); |
| Keith Whitwell | cab974c | 2000-12-26 05:09:27 +0000 | [diff] [blame] | 163 | |
| Alan Hourihane | 263b96e | 2009-01-15 11:51:39 +0000 | [diff] [blame] | 164 | if (!check_valid_to_render(ctx, "Elements")) |
| Keith Whitwell | cab974c | 2000-12-26 05:09:27 +0000 | [diff] [blame] | 165 | return GL_FALSE; |
| 166 | |
| Brian Paul | 03e29a5 | 2003-12-04 03:16:27 +0000 | [diff] [blame] | 167 | /* Vertex buffer object tests */ |
| Brian Paul | 434ec3a | 2009-08-12 13:46:16 -0600 | [diff] [blame] | 168 | if (_mesa_is_bufferobj(ctx->Array.ElementArrayBufferObj)) { |
| Brian | d8c6719 | 2007-08-20 13:12:20 +0100 | [diff] [blame] | 169 | /* use indices in the buffer object */ |
| Brian | d8c6719 | 2007-08-20 13:12:20 +0100 | [diff] [blame] | 170 | /* make sure count doesn't go outside buffer bounds */ |
| Brian Paul | bb1fb2a | 2009-05-06 12:37:10 -0600 | [diff] [blame] | 171 | if (index_bytes(type, count) > ctx->Array.ElementArrayBufferObj->Size) { |
| Brian Paul | 03e29a5 | 2003-12-04 03:16:27 +0000 | [diff] [blame] | 172 | _mesa_warning(ctx, "glDrawElements index out of buffer bounds"); |
| 173 | return GL_FALSE; |
| 174 | } |
| Brian Paul | 03e29a5 | 2003-12-04 03:16:27 +0000 | [diff] [blame] | 175 | } |
| Brian | 37ece4d | 2007-07-08 09:20:42 -0600 | [diff] [blame] | 176 | else { |
| 177 | /* not using a VBO */ |
| 178 | if (!indices) |
| 179 | return GL_FALSE; |
| 180 | } |
| Brian Paul | 03e29a5 | 2003-12-04 03:16:27 +0000 | [diff] [blame] | 181 | |
| Brian Paul | a2b9bad | 2003-11-10 19:08:37 +0000 | [diff] [blame] | 182 | if (ctx->Const.CheckArrayBounds) { |
| 183 | /* find max array index */ |
| Brian | d8c6719 | 2007-08-20 13:12:20 +0100 | [diff] [blame] | 184 | GLuint max = max_buffer_index(ctx, count, type, indices, |
| 185 | ctx->Array.ElementArrayBufferObj); |
| Brian Paul | a185bcb | 2009-05-14 13:24:24 -0600 | [diff] [blame] | 186 | if (max >= ctx->Array.ArrayObj->_MaxElement) { |
| Brian | a3c3bc9 | 2007-08-20 12:55:34 +0100 | [diff] [blame] | 187 | /* the max element is out of bounds of one or more enabled arrays */ |
| Brian Paul | 87fbc9a | 2009-05-08 12:44:38 -0600 | [diff] [blame] | 188 | _mesa_warning(ctx, "glDrawElements() index=%u is " |
| Brian Paul | a185bcb | 2009-05-14 13:24:24 -0600 | [diff] [blame] | 189 | "out of bounds (max=%u)", max, ctx->Array.ArrayObj->_MaxElement); |
| Brian | a3c3bc9 | 2007-08-20 12:55:34 +0100 | [diff] [blame] | 190 | return GL_FALSE; |
| 191 | } |
| Xiang, Haihao | 2394d20 | 2007-07-30 23:50:52 +0800 | [diff] [blame] | 192 | } |
| 193 | |
| Keith Whitwell | cab974c | 2000-12-26 05:09:27 +0000 | [diff] [blame] | 194 | return GL_TRUE; |
| 195 | } |
| 196 | |
| Brian Paul | 88af3f8 | 2009-05-06 12:27:38 -0600 | [diff] [blame] | 197 | |
| 198 | /** |
| 199 | * Error checking for glDrawRangeElements(). Includes parameter checking |
| 200 | * and VBO bounds checking. |
| 201 | * \return GL_TRUE if OK to render, GL_FALSE if error found |
| 202 | */ |
| Keith Whitwell | cab974c | 2000-12-26 05:09:27 +0000 | [diff] [blame] | 203 | GLboolean |
| Gareth Hughes | 22144ab | 2001-03-12 00:48:37 +0000 | [diff] [blame] | 204 | _mesa_validate_DrawRangeElements(GLcontext *ctx, GLenum mode, |
| 205 | GLuint start, GLuint end, |
| 206 | GLsizei count, GLenum type, |
| Keith Whitwell | cab974c | 2000-12-26 05:09:27 +0000 | [diff] [blame] | 207 | const GLvoid *indices) |
| 208 | { |
| Gareth Hughes | 22144ab | 2001-03-12 00:48:37 +0000 | [diff] [blame] | 209 | ASSERT_OUTSIDE_BEGIN_END_WITH_RETVAL(ctx, GL_FALSE); |
| Keith Whitwell | cab974c | 2000-12-26 05:09:27 +0000 | [diff] [blame] | 210 | |
| 211 | if (count <= 0) { |
| 212 | if (count < 0) |
| Brian Paul | a2b9bad | 2003-11-10 19:08:37 +0000 | [diff] [blame] | 213 | _mesa_error(ctx, GL_INVALID_VALUE, "glDrawRangeElements(count)" ); |
| Keith Whitwell | cab974c | 2000-12-26 05:09:27 +0000 | [diff] [blame] | 214 | return GL_FALSE; |
| 215 | } |
| 216 | |
| Brian Paul | 464bc3b | 2003-04-21 15:04:30 +0000 | [diff] [blame] | 217 | if (mode > GL_POLYGON) { |
| Brian Paul | a2b9bad | 2003-11-10 19:08:37 +0000 | [diff] [blame] | 218 | _mesa_error(ctx, GL_INVALID_ENUM, "glDrawRangeElements(mode)" ); |
| Keith Whitwell | cab974c | 2000-12-26 05:09:27 +0000 | [diff] [blame] | 219 | return GL_FALSE; |
| 220 | } |
| 221 | |
| 222 | if (end < start) { |
| Brian Paul | 0883634 | 2001-03-03 20:33:27 +0000 | [diff] [blame] | 223 | _mesa_error(ctx, GL_INVALID_VALUE, "glDrawRangeElements(end<start)"); |
| Keith Whitwell | cab974c | 2000-12-26 05:09:27 +0000 | [diff] [blame] | 224 | return GL_FALSE; |
| 225 | } |
| 226 | |
| Gareth Hughes | 22144ab | 2001-03-12 00:48:37 +0000 | [diff] [blame] | 227 | if (type != GL_UNSIGNED_INT && |
| 228 | type != GL_UNSIGNED_BYTE && |
| Brian Paul | 61bac60 | 2002-04-21 21:04:46 +0000 | [diff] [blame] | 229 | type != GL_UNSIGNED_SHORT) { |
| Brian Paul | a2b9bad | 2003-11-10 19:08:37 +0000 | [diff] [blame] | 230 | _mesa_error(ctx, GL_INVALID_ENUM, "glDrawRangeElements(type)" ); |
| Keith Whitwell | cab974c | 2000-12-26 05:09:27 +0000 | [diff] [blame] | 231 | return GL_FALSE; |
| 232 | } |
| 233 | |
| 234 | if (ctx->NewState) |
| Brian Paul | a2b9bad | 2003-11-10 19:08:37 +0000 | [diff] [blame] | 235 | _mesa_update_state(ctx); |
| Keith Whitwell | cab974c | 2000-12-26 05:09:27 +0000 | [diff] [blame] | 236 | |
| Alan Hourihane | 263b96e | 2009-01-15 11:51:39 +0000 | [diff] [blame] | 237 | if (!check_valid_to_render(ctx, "RangeElements")) |
| Keith Whitwell | cab974c | 2000-12-26 05:09:27 +0000 | [diff] [blame] | 238 | return GL_FALSE; |
| Keith Whitwell | cab974c | 2000-12-26 05:09:27 +0000 | [diff] [blame] | 239 | |
| Brian | 37ece4d | 2007-07-08 09:20:42 -0600 | [diff] [blame] | 240 | /* Vertex buffer object tests */ |
| Brian Paul | 434ec3a | 2009-08-12 13:46:16 -0600 | [diff] [blame] | 241 | if (_mesa_is_bufferobj(ctx->Array.ElementArrayBufferObj)) { |
| Brian | d8c6719 | 2007-08-20 13:12:20 +0100 | [diff] [blame] | 242 | /* use indices in the buffer object */ |
| Brian | d8c6719 | 2007-08-20 13:12:20 +0100 | [diff] [blame] | 243 | /* make sure count doesn't go outside buffer bounds */ |
| Brian Paul | bb1fb2a | 2009-05-06 12:37:10 -0600 | [diff] [blame] | 244 | if (index_bytes(type, count) > ctx->Array.ElementArrayBufferObj->Size) { |
| Brian | ef5935b | 2007-09-23 13:56:46 -0600 | [diff] [blame] | 245 | _mesa_warning(ctx, "glDrawRangeElements index out of buffer bounds"); |
| Brian | d8c6719 | 2007-08-20 13:12:20 +0100 | [diff] [blame] | 246 | return GL_FALSE; |
| 247 | } |
| Brian | 37ece4d | 2007-07-08 09:20:42 -0600 | [diff] [blame] | 248 | } |
| 249 | else { |
| Brian | d8c6719 | 2007-08-20 13:12:20 +0100 | [diff] [blame] | 250 | /* not using a VBO */ |
| Brian | 37ece4d | 2007-07-08 09:20:42 -0600 | [diff] [blame] | 251 | if (!indices) |
| 252 | return GL_FALSE; |
| 253 | } |
| 254 | |
| Brian Paul | a2b9bad | 2003-11-10 19:08:37 +0000 | [diff] [blame] | 255 | if (ctx->Const.CheckArrayBounds) { |
| Brian | d8c6719 | 2007-08-20 13:12:20 +0100 | [diff] [blame] | 256 | GLuint max = max_buffer_index(ctx, count, type, indices, |
| 257 | ctx->Array.ElementArrayBufferObj); |
| Brian Paul | a185bcb | 2009-05-14 13:24:24 -0600 | [diff] [blame] | 258 | if (max >= ctx->Array.ArrayObj->_MaxElement) { |
| Brian Paul | a2b9bad | 2003-11-10 19:08:37 +0000 | [diff] [blame] | 259 | /* the max element is out of bounds of one or more enabled arrays */ |
| 260 | return GL_FALSE; |
| 261 | } |
| 262 | } |
| Keith Whitwell | cab974c | 2000-12-26 05:09:27 +0000 | [diff] [blame] | 263 | |
| Brian Paul | c5b1e81 | 2003-10-22 22:59:07 +0000 | [diff] [blame] | 264 | return GL_TRUE; |
| 265 | } |
| Keith Whitwell | cab974c | 2000-12-26 05:09:27 +0000 | [diff] [blame] | 266 | |
| Brian Paul | c5b1e81 | 2003-10-22 22:59:07 +0000 | [diff] [blame] | 267 | |
| 268 | /** |
| 269 | * Called from the tnl module to error check the function parameters and |
| 270 | * verify that we really can draw something. |
| Brian Paul | 88af3f8 | 2009-05-06 12:27:38 -0600 | [diff] [blame] | 271 | * \return GL_TRUE if OK to render, GL_FALSE if error found |
| Brian Paul | c5b1e81 | 2003-10-22 22:59:07 +0000 | [diff] [blame] | 272 | */ |
| Keith Whitwell | cab974c | 2000-12-26 05:09:27 +0000 | [diff] [blame] | 273 | GLboolean |
| Gareth Hughes | 22144ab | 2001-03-12 00:48:37 +0000 | [diff] [blame] | 274 | _mesa_validate_DrawArrays(GLcontext *ctx, |
| Keith Whitwell | cab974c | 2000-12-26 05:09:27 +0000 | [diff] [blame] | 275 | GLenum mode, GLint start, GLsizei count) |
| 276 | { |
| Gareth Hughes | 22144ab | 2001-03-12 00:48:37 +0000 | [diff] [blame] | 277 | ASSERT_OUTSIDE_BEGIN_END_WITH_RETVAL(ctx, GL_FALSE); |
| Keith Whitwell | cab974c | 2000-12-26 05:09:27 +0000 | [diff] [blame] | 278 | |
| Brian | 5cb2034 | 2007-10-31 09:38:51 -0600 | [diff] [blame] | 279 | if (count <= 0) { |
| 280 | if (count < 0) |
| 281 | _mesa_error(ctx, GL_INVALID_VALUE, "glDrawArrays(count)" ); |
| Keith Whitwell | cab974c | 2000-12-26 05:09:27 +0000 | [diff] [blame] | 282 | return GL_FALSE; |
| 283 | } |
| 284 | |
| Brian Paul | 464bc3b | 2003-04-21 15:04:30 +0000 | [diff] [blame] | 285 | if (mode > GL_POLYGON) { |
| Brian Paul | 0883634 | 2001-03-03 20:33:27 +0000 | [diff] [blame] | 286 | _mesa_error(ctx, GL_INVALID_ENUM, "glDrawArrays(mode)" ); |
| Keith Whitwell | cab974c | 2000-12-26 05:09:27 +0000 | [diff] [blame] | 287 | return GL_FALSE; |
| 288 | } |
| 289 | |
| 290 | if (ctx->NewState) |
| Brian Paul | a2b9bad | 2003-11-10 19:08:37 +0000 | [diff] [blame] | 291 | _mesa_update_state(ctx); |
| Keith Whitwell | cab974c | 2000-12-26 05:09:27 +0000 | [diff] [blame] | 292 | |
| Alan Hourihane | 263b96e | 2009-01-15 11:51:39 +0000 | [diff] [blame] | 293 | if (!check_valid_to_render(ctx, "Arrays")) |
| Brian Paul | a2b9bad | 2003-11-10 19:08:37 +0000 | [diff] [blame] | 294 | return GL_FALSE; |
| 295 | |
| 296 | if (ctx->Const.CheckArrayBounds) { |
| Brian Paul | a185bcb | 2009-05-14 13:24:24 -0600 | [diff] [blame] | 297 | if (start + count > (GLint) ctx->Array.ArrayObj->_MaxElement) |
| Brian Paul | c5b1e81 | 2003-10-22 22:59:07 +0000 | [diff] [blame] | 298 | return GL_FALSE; |
| 299 | } |
| Brian Paul | c5b1e81 | 2003-10-22 22:59:07 +0000 | [diff] [blame] | 300 | |
| 301 | return GL_TRUE; |
| Keith Whitwell | cab974c | 2000-12-26 05:09:27 +0000 | [diff] [blame] | 302 | } |