Kenneth Graunke <kenn...@whitecape.org> writes:
> On 07/31/2012 03:01 PM, Eric Anholt wrote:
>> +const struct brw_tracked_state brw_vs_ubo_surfaces = {
>> +   .dirty = {
>> +      .mesa = (_NEW_PROGRAM |
>> +           _NEW_BUFFER_OBJECT),
>> +      .brw = (BRW_NEW_BATCH),
>
> You don't actually need the extra parenthesis...IMHO,
>
>   .foo = bits | morebits | yetmorebits,
>   .bar = ...
>
> is already quite readable, and definitely unambiguous (the = and , are
> clear delimiters).  Not a big deal either way though.

Yeah, just an artifact of how I copied some other struct and kept
dropping flags out of the middle of the parens.  Fixed.

Attachment: pgpQxPIgig7na.pgp
Description: PGP signature

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

Reply via email to