On Wed, Oct 5, 2011 at 5:28 PM, Joseph S. Myers <jos...@codesourcery.com> wrote:
> On Wed, 5 Oct 2011, Artem Shinkarov wrote:
>
>> Joseph, is it possible to commit the patch together with the space fixes?
>
> You should not commit whitespace fixes to lines not otherwise modified by
> a patch, except in a separate commit that *only* has whitespace or other
> formatting fixes, so that "svn blame" results are more meaningful.  That
> is: commit whitespace fixes (assuming they are genuinely fixes rather than
> making things worse) *separately* in a commit whose commit message makes
> clear it is just whitespace fixes.

Ok, I see.  In the given patch all the fixes are mainly auto-generated
and remove the trailing spaces.  But I see your point.

Thanks,
Artem.
> --
> Joseph S. Myers
> jos...@codesourcery.com
>

Reply via email to