From f6198e9146b5115676ae89d9515fc5d9e8693c05 Mon Sep 17 00:00:00 2001 From: Timothy Arceri Date: Thu, 11 May 2017 15:43:44 +1000 Subject: [PATCH] mesa: add no error support to frame_buffer_texture_layer() helper MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit Reviewed-by: Nicolai Hähnle --- src/mesa/main/fbobject.c | 70 ++++++++++++++++++++++++---------------- 1 file changed, 43 insertions(+), 27 deletions(-) diff --git a/src/mesa/main/fbobject.c b/src/mesa/main/fbobject.c index 8e650a24b83..bdce10d8e6a 100644 --- a/src/mesa/main/fbobject.c +++ b/src/mesa/main/fbobject.c @@ -3430,41 +3430,61 @@ static ALWAYS_INLINE void frame_buffer_texture_layer(GLuint framebuffer, GLenum target, GLenum attachment, GLuint texture, GLint level, GLint layer, const char *func, - bool dsa) + bool dsa, bool no_error) { GET_CURRENT_CONTEXT(ctx); /* Get the framebuffer object */ struct gl_framebuffer *fb; - if (dsa) { - fb = _mesa_lookup_framebuffer_err(ctx, framebuffer, func); - if (!fb) - return; - + if (no_error) { + if (dsa) { + fb = _mesa_lookup_framebuffer(ctx, framebuffer); + } else { + fb = get_framebuffer_target(ctx, target); + } } else { - fb = get_framebuffer_target(ctx, target); - if (!fb) { - _mesa_error(ctx, GL_INVALID_ENUM, "%s(invalid target %s)", - func, _mesa_enum_to_string(target)); - return; + if (dsa) { + fb = _mesa_lookup_framebuffer_err(ctx, framebuffer, func); + if (!fb) + return; + } else { + fb = get_framebuffer_target(ctx, target); + if (!fb) { + _mesa_error(ctx, GL_INVALID_ENUM, "%s(invalid target %s)", + func, _mesa_enum_to_string(target)); + return; + } } } - /* Get the texture object */ + /* Get the texture object and framebuffer attachment*/ + struct gl_renderbuffer_attachment *att; struct gl_texture_object *texObj; - if (!get_texture_for_framebuffer_err(ctx, texture, false, func, &texObj)) - return; + if (no_error) { + texObj = get_texture_for_framebuffer(ctx, texture); + att = get_attachment(ctx, fb, attachment, NULL); + } else { + if (!get_texture_for_framebuffer_err(ctx, texture, false, func, + &texObj)) + return; + + att = _mesa_get_and_validate_attachment(ctx, fb, attachment, func); + if (!att) + return; + } GLenum textarget = 0; if (texObj) { - if (!check_texture_target(ctx, texObj->Target, func)) - return; + if (!no_error) { + if (!check_texture_target(ctx, texObj->Target, func)) + return; - if (!check_layer(ctx, texObj->Target, layer, func)) - return; + if (!check_layer(ctx, texObj->Target, layer, func)) + return; - if (!check_level(ctx, texObj->Target, level, func)) - return; + if (!check_level(ctx, texObj->Target, level, func)) + return; + } if (texObj->Target == GL_TEXTURE_CUBE_MAP) { assert(layer >= 0 && layer < 6); @@ -3473,11 +3493,6 @@ frame_buffer_texture_layer(GLuint framebuffer, GLenum target, } } - struct gl_renderbuffer_attachment *att = - _mesa_get_and_validate_attachment(ctx, fb, attachment, func); - if (!att) - return; - _mesa_framebuffer_texture(ctx, fb, attachment, att, texObj, textarget, level, layer, GL_FALSE); } @@ -3488,7 +3503,7 @@ _mesa_FramebufferTextureLayer(GLenum target, GLenum attachment, GLuint texture, GLint level, GLint layer) { frame_buffer_texture_layer(0, target, attachment, texture, level, layer, - "glFramebufferTextureLayer", false); + "glFramebufferTextureLayer", false, false); } @@ -3497,7 +3512,8 @@ _mesa_NamedFramebufferTextureLayer(GLuint framebuffer, GLenum attachment, GLuint texture, GLint level, GLint layer) { frame_buffer_texture_layer(framebuffer, 0, attachment, texture, level, - layer, "glNamedFramebufferTextureLayer", true); + layer, "glNamedFramebufferTextureLayer", true, + false); } -- 2.30.2