[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