[syslinux] com32/mboot/map.c: removed trailing spaces

Geert Stappers stappers at stappers.nl
Tue Oct 13 14:17:47 PDT 2015


On Sat, Oct 10, 2015 at 03:10:26PM +0300, Ady via Syslinux wrote:
> From: Geert Stappers <stappers op nero.gpm.stappers.nl>
> > 
> > com32/mboot/map.c: removed trailing spaces
> > 
> > They were introduced by the patch for ELF64 support.
> 
> 
> IMHO, the trivial trailing-space cleanup could be included in the same 
> commit too, instead of adding an additional commit just for it. That 
> is, if there is no additional correction / improvement required.

I think it is a good thing to just accept a patch that is good enough
and do reflow work in an additional patch.

I see several advantages:

 * a clear signal of "patch accepted"
 * making visible how a patch is made better
 * no merge conflict for those who have the original patch applied


Groeten
Geert Stappers
-- 
Leven en laten leven


More information about the Syslinux mailing list