[PATCH] Fix trailing whitespace in new (!!!) code

Jeremy Allison jra at samba.org
Fri Apr 6 17:22:52 UTC 2018


On Fri, Apr 06, 2018 at 02:06:09PM +0200, Volker Lendecke via samba-technical wrote:
> On Fri, Apr 06, 2018 at 01:54:18PM +0200, Ralph Böhme via samba-technical wrote:
> > On Fri, Apr 06, 2018 at 01:48:34PM +0200, Volker Lendecke via samba-technical wrote:
> > > Attached find a patch that cleans up fresh code to match
> > > README.Coding. Gary, if you review Andrew's code in the future, can
> > > you please point him at the relevant section of README.Coding, quoted
> > > here for your convenience?
> > 
> > I messed this up many times as well, the only thing that helped me to avoid
> > this error was to configure my editor to highlight trailing whitespace. :)
> 
> Right, but if I would have to do a ranking regarding resistance
> against README.Coding, quite a few people would beat you in that
> discipline :-)

Configuring the editor to flag trailing whitespace
is the only way this can be fixed for people (it's
not humanly possible to do it otherwise).

For anyone using vim, I can give them my settings,
for emacs - try this:

https://www.gnu.org/software/emacs/manual/html_node/emacs/Useless-Whitespace.html



More information about the samba-technical mailing list