[Mesa-dev] [PATCH 06/18] swr: [rasterizer core] cygwin cleanup

Rowley, Timothy O timothy.o.rowley at intel.com
Tue May 17 15:38:15 UTC 2016


> On May 17, 2016, at 6:03 AM, Emil Velikov <emil.l.velikov at gmail.com> wrote:
> 
> Hi Tim,
> 
> On 17 May 2016 at 02:10, Tim Rowley <timothy.o.rowley at intel.com> wrote:
>> ---
>> .../drivers/swr/rasterizer/core/threads.cpp        | 60 ++++++++++++++--------
>> 1 file changed, 38 insertions(+), 22 deletions(-)
>> 
> May I suggest that one splits up patches appropriately and a bit more
> descriptive summaries ?

Personally I’m very much in favor of separating style and functional changes, but there are some developers who like to tidy nearby code during development.  I’ll ask again to try consider keeping them separate.

I’ll go through this patch set and try to make the commit messages more descriptive.

-Tim



More information about the mesa-dev mailing list