On Thu, Oct 13, 2011 at 10:46:29AM +0200, Daniel Vetter wrote: > On Wed, Oct 12, 2011 at 03:56:21PM -0700, Ben Widawsky wrote: > > Simple refactor. > > > > Signed-off-by: Ben Widawsky <b...@bwidawsk.net> > > Imo you can fold this in with the fix. Usually it's good to separate fixes > from refactoring, but it this special case of just moving around code > blocks, it doesn't add anything. >
I agree, and I had it like this locally, but Chris suggested that he preferred the separation as it highlighted the fix. I largely do not care, I just want an r-b. So you two can fight it out. Ben _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx
