[PATCH 04/18] Three minor warning and whitespace fixes

[Eric Anholt]

Keith Packard <keithp at keithp.com> writes:

> Eric Anholt <eric at anholt.net> writes:
>
>> So there's a bunch of disagreement between this new code and existing
>> practice about whitespace, and still misc unrelated whitespace changes
>> sprinkled into the remaining patches.  That makes me grumpy when
>> reviewing, but I don't actually care to block things due to that.  This
>> driver is enough of a disaster already that whitespace inconsistency
>> isn't its main problem.
>
> Yeah, I'm trying to use the server standard settings for all X work
> these days, so I'm using spaces instead of tabs. I can fix that if you
> want, but as you say, it's just annoyingly inconsistent.

Well, except that the indentation size is 8 instead of 4, and your
patches are inconsistent with themselves, even. :P

>> Patches 1-3, 5-6, 8, 12-15 are:
>>
>> Reviewed-by: Eric Anholt <eric at anholt.net>
>
> Thanks!
>
>> 16-18: I'm still looking at them.
>
> Cool.

I ended up putting up a keithp-glamor-rewrite branch in my tree -- I was
thinking I could pull out more non-glamor changes from the glamor patch,
and I got a couple of things that made the glamor patches more sensible,
but the rest looked harder.  There are some other little squashes in
that tree that I noticed along the way.  I think that's all I've got for
review on this series.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 818 bytes
Desc: not available
URL: <http://lists.x.org/archives/xorg-devel/attachments/20140826/6802143f/attachment.sig>