On Wed, Sep 28, 2016 at 5:14 PM, Brian Paul <bri...@vmware.com> wrote:
> On 09/28/2016 07:44 AM, Marek Olšák wrote:
>>
>> From: Marek Olšák <marek.ol...@amd.com>
>>
>> The other flag seems to have a slightly different meaning.
>
>
> I don't recall what the GL spec says w.r.t. mixing floating point and
> integer color buffers.  I suspect it may be implementation dependent.

Mixing float and integer color buffers is allowed.

>
> I wonder if we should replace _IntegerColor and _Color0IsInteger with a
> bitmask indicating which color buffers are integer-valued.  That is:
>
> struct gl_framebuffer {
>    ...
>    GLbitfield _IntegerColorBuffers;
>    ...
> };
>
>
> _mesa_test_framebuffer_completeness()
> ...
> if (_mesa_is_format_integer_color(attFormat))
>     fb->_IntegerColorBuffers |= (1 << i);
>
>
> What do you think?

It depends on what it would be good for. Alpha-test,
alpha-to-coverage, and alpha-to-one only apply to color buffer 0.

Marek
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to