Mesa (master): mesa/fbo: do not assign a value that is never read later on
Martin Peres
mperes at kemper.freedesktop.org
Wed Apr 1 06:37:25 UTC 2015
Module: Mesa
Branch: master
Commit: fa3832155100850da8346faa64c3cb30c4e86e39
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=fa3832155100850da8346faa64c3cb30c4e86e39
Author: Martin Peres <martin.peres at linux.intel.com>
Date: Wed Mar 25 16:28:03 2015 +0200
mesa/fbo: do not assign a value that is never read later on
The issue has been detected by coverty.
v2:
- move the declaration of obj to the else clause (Brian Paul)
v3: Review by Brian Paul
- get rid of the obj declaration in favor of a direct reference
Reviewed-by: Brian Paul <brianp at vmware.com>
Signed-off-by: Martin Peres <martin.peres at linux.intel.com>
---
src/mesa/main/fbobject.c | 9 +++------
1 file changed, 3 insertions(+), 6 deletions(-)
diff --git a/src/mesa/main/fbobject.c b/src/mesa/main/fbobject.c
index 072e1a8..3808b56 100644
--- a/src/mesa/main/fbobject.c
+++ b/src/mesa/main/fbobject.c
@@ -1400,7 +1400,6 @@ create_render_buffers(struct gl_context *ctx, GLsizei n, GLuint *renderbuffers,
bool dsa)
{
const char *func = dsa ? "glCreateRenderbuffers" : "glGenRenderbuffers";
- struct gl_renderbuffer *obj;
GLuint first;
GLint i;
@@ -1419,13 +1418,11 @@ create_render_buffers(struct gl_context *ctx, GLsizei n, GLuint *renderbuffers,
renderbuffers[i] = name;
if (dsa) {
- obj = allocate_renderbuffer(ctx, name, func);
+ allocate_renderbuffer(ctx, name, func);
} else {
- obj = &DummyRenderbuffer;
-
- /* insert the object into the hash table */
+ /* insert a dummy renderbuffer into the hash table */
mtx_lock(&ctx->Shared->Mutex);
- _mesa_HashInsert(ctx->Shared->RenderBuffers, name, obj);
+ _mesa_HashInsert(ctx->Shared->RenderBuffers, name, &DummyRenderbuffer);
mtx_unlock(&ctx->Shared->Mutex);
}
}
More information about the mesa-commit
mailing list