Re: [Mesa-dev] [PATCH 1/5] mesa: Track number of layers in layered framebuffers.

2013-11-20 Thread Chris Forbes
With those two changes, this is Reviewed-by: Chris Forbes On Thu, Nov 21, 2013 at 8:05 AM, Paul Berry wrote: > On 20 November 2013 03:18, Marek Olšák wrote: >> >> On Wed, Nov 20, 2013 at 5:47 AM, Paul Berry >> wrote: >> > >> > + /** >> > +* If Layered is true, the number of layers in th

Re: [Mesa-dev] [PATCH 1/5] mesa: Track number of layers in layered framebuffers.

2013-11-20 Thread Paul Berry
On 19 November 2013 22:22, Chris Forbes wrote: > + if (layer_count > 0) { > + fb->NumLayers = layer_count; > > It seems like in the nonlayered case there will just be junk left in > fb->NumLayers, which might trip people up in future? > Yeah, that's a fair critique. On further reflection

Re: [Mesa-dev] [PATCH 1/5] mesa: Track number of layers in layered framebuffers.

2013-11-20 Thread Paul Berry
On 20 November 2013 03:18, Marek Olšák wrote: > On Wed, Nov 20, 2013 at 5:47 AM, Paul Berry > wrote: > > > > + /** > > +* If Layered is true, the number of layers in the framebuffer. For > cube > > +* maps and cube map arrays, this includes the factor of 6. > > +*/ > > + GLuint

Re: [Mesa-dev] [PATCH 1/5] mesa: Track number of layers in layered framebuffers.

2013-11-20 Thread Marek Olšák
On Wed, Nov 20, 2013 at 5:47 AM, Paul Berry wrote: > In order to properly clear layered framebuffers, we need to know how > many layers they have. The easiest way to do this is to record it in > the gl_framebuffer struct when we check framebuffer completeness, just > like we do for the Layered bo

Re: [Mesa-dev] [PATCH 1/5] mesa: Track number of layers in layered framebuffers.

2013-11-19 Thread Chris Forbes
+ if (layer_count > 0) { + fb->NumLayers = layer_count; It seems like in the nonlayered case there will just be junk left in fb->NumLayers, which might trip people up in future? ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists