On Thu, 23 Aug 2018 at 18:41, Emil Velikov <emil.l.veli...@gmail.com> wrote:
>
> Hi Fritz,
>
> On Wed, 22 Aug 2018 at 23:01, Fritz Koenig <frkoe...@google.com> wrote:
> >
> > In the GL_MESA_framebuffer_flip_y implementation
> > _mesa_is_winsys_fbo checks were replaced with
> > FlipY checks. rb->Name is also used to determine
> > if a buffer is winsys.
>
> This seems like it fixes your earlier patch, so you normally want a
> fixes tag. Something like the following should be fine.
> It adds nice reference tracking, allowing the release manager to track
> and land in the correct stable branches.
>
> Fixes: ab05dd183cc ("i965: implement GL_MESA_framebuffer_flip_y [v3]")
>
Quick grep shows a number of rb->Name instances in intel_fbo.c - DBG
and assert statements.
If they should stay as-is, perhaps it'll be better to add some comments?
Just a fly-by idea.
-Emil
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
