On 11/28/2012 04:43 AM, Aleksandar Kurtakov wrote:
And please don't mix formatting changes with code changes as it becomes really 
hard to review what actually was changed.


The only reason I did that was to avoid having more patches and creating a bigger series :). I actually believe that each patch should contain only one logical or formatting change.

Gerrit highlights non white-space changes in darker red and darker green. This should catch most formatting changes and allow you to focus on actual code changes.

  Sami
_______________________________________________
linuxtools-dev mailing list
linuxtools-dev@eclipse.org
https://dev.eclipse.org/mailman/listinfo/linuxtools-dev

Reply via email to