[Mesa-dev] [PATCH 0/1]: Preparing for ARB_viewport_array

[Courtney Goeltzenleuchter]

Hi Ian,

I see that you have a shadow Viewport and Scissor attribute structure. I
was trying to avoid managing two copies of the data so I just updated all
the drivers to use index 0 of the array directly. You'd probably want to do
the same thing with the scissor enable flag that becomes a bitfield of
enables. Changing from Viewport to Viewports (I used ViewportArray) in the
drivers touches about 20 files. Have to touch those same files when
removing the x,y,w,h arguments from Viewport and Scissor anyway so I
figured it was no harder to change the references to Scissor and Viewport
to be array references to index 0.

Shall I just post my changes for folks to poke at? Or is a fdo personal
repository easier to work with?

Courtney


On Tue, Nov 19, 2013 at 5:54 PM, Ian Romanick <idr at freedesktop.org> wrote:

> On 10/31/2013 08:55 AM, Courtney Goeltzenleuchter wrote:
> > The following patch will begin the process of adding ARB_viewport_array
> > to Mesa. Next will be to extend the gl_context Scissor and Viewport
> > attributes to hold multiple viewport, scissor and scissor enables.
> > Then the DI side of ARB_viewport_array.
>
> I pushed a viewport-array branch to my freedesktop.org repository.
> There's a bunch of in-progress refactoring there.  The big thing is
> making the current viewport-related structures derived state.  This
> minimizes the changes necessary to non-array drivers.
>
> Opinions?
>
> > _______________________________________________
> > mesa-dev mailing list
> > mesa-dev at lists.freedesktop.org
> > http://lists.freedesktop.org/mailman/listinfo/mesa-dev
>
>


-- 
Courtney Goeltzenleuchter
LunarG
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20131120/6bea4c48/attachment.html>