[PATCH resent 0/5] xfree86: vgaarb: clean up and device hidden

Tiago Vignatti tiago.vignatti at nokia.com
Thu Jun 3 02:45:28 PDT 2010


On Thu, Jun 03, 2010 at 11:42:42AM +0200, ext Julien Cristau wrote:
> On Thu, Jun  3, 2010 at 16:38:13 +0700, Mikhail Gusarov wrote:
> 
> > This way every patch is self-contained (and easily reviewed), which is
> > Good Thing. I'd be puzzled if first patch would convert every macros
> > _except two_ to inline functions, justified only by the next patch which
> > removes those two macros.
> > 
> OTOH, if you first get rid of two said macros, and in the next patch
> convert the remaining ones, you get the best of both worlds.  Unless I'm
> missing something?

yeah, this probably would be the best.

             Tiago


More information about the xorg-devel mailing list