On Thu, Aug 20, 2015 at 1:52 AM, Tapani Pälli <tapani.pa...@intel.com> wrote:
> We have to re-validate FBOs rendering to the texture like is done > with TexImage and CopyTexImage. > > Signed-off-by: Tapani Pälli <tapani.pa...@intel.com> > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=91673 > Cc: "10.6" <mesa-sta...@lists.freedesktop.org> > --- > src/mesa/main/texstorage.c | 15 +++++++++++++++ > 1 file changed, 15 insertions(+) > > diff --git a/src/mesa/main/texstorage.c b/src/mesa/main/texstorage.c > index db2b63d..51f49b3 100644 > --- a/src/mesa/main/texstorage.c > +++ b/src/mesa/main/texstorage.c > @@ -189,6 +189,20 @@ clear_texture_fields(struct gl_context *ctx, > } > > > +/** > + * Update/re-validate framebuffer object. > + */ > +static void > +update_fbo_texture(struct gl_context *ctx, struct gl_texture_object > *texObj) > +{ > + const unsigned numFaces = _mesa_num_tex_faces(texObj->Target); > + for (int level = 0; level < ARRAY_SIZE(texObj->Image[0]); level++) { > + for (unsigned face = 0; face < numFaces; face++) > We don't normally declare variable in the loops like that. Not sure if that'll compile everywhere. -Brian > + _mesa_update_fbo_texture(ctx, texObj, face, level); > + } > +} > + > + > GLboolean > _mesa_is_legal_tex_storage_format(struct gl_context *ctx, GLenum > internalformat) > { > @@ -446,6 +460,7 @@ _mesa_texture_storage(struct gl_context *ctx, GLuint > dims, > > _mesa_set_texture_view_state(ctx, texObj, target, levels); > > + update_fbo_texture(ctx, texObj); > } > } > > -- > 2.1.0 > > _______________________________________________ > mesa-dev mailing list > mesa-dev@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/mesa-dev >
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
