Reviewed-by: Marek Olšák <marek.ol...@amd.com> Marek
On Fri, Sep 1, 2017 at 1:53 AM, <srol...@vmware.com> wrote: > From: Roland Scheidegger <srol...@vmware.com> > > I think this is what the code was meant to do, albeit as far as I can tell > the redundant initialization some analyzers complain about should work as > well just fine (only the first layer will be used, if the view contains one > or more layers doesn't really matter). > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=102467 > --- > src/mesa/state_tracker/st_cb_readpixels.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/src/mesa/state_tracker/st_cb_readpixels.c > b/src/mesa/state_tracker/st_cb_readpixels.c > index 0bcf2eb..84dd2d5 100644 > --- a/src/mesa/state_tracker/st_cb_readpixels.c > +++ b/src/mesa/state_tracker/st_cb_readpixels.c > @@ -175,7 +175,7 @@ try_pbo_readpixels(struct st_context *st, struct > st_renderbuffer *strb, > > if (view_target != PIPE_TEXTURE_3D) { > templ.u.tex.first_layer = surface->u.tex.first_layer; > - templ.u.tex.last_layer = templ.u.tex.last_layer; > + templ.u.tex.last_layer = templ.u.tex.first_layer; > } else { > addr.constants.layer_offset = surface->u.tex.first_layer; > } > -- > 2.7.4 > > _______________________________________________ > mesa-dev mailing list > mesa-dev@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/mesa-dev _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
