On Tue, Feb 21, 2017 at 09:58:39AM +0000, Tvrtko Ursulin wrote: > From: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxx> > > Compact the name of the macro and reg_state variable, and cache > some data in local variables to make the function more compact > and more readable. > > Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxx> > --- > Not that valuable but those ",0);" in new lines are a bit of an > eyesore so I thought we could compact it all a bit to avoid that > as much as possible. I found it hard to follow because of the line breaks, so having done the work to make it easier to read and so maintain: Reviewed-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx