Hi Tim, On 17 May 2016 at 02:10, Tim Rowley <timothy.o.row...@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 ?
For example here 2/3 of the patch is whitespace changes, which one can go without. -Emil _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
