Whitespace and bullying

[Timur I. Bakeyev]

On 6 April 2018 at 23:12, Jeremy Allison via samba-technical <
samba-technical at lists.samba.org> wrote:

> On Fri, Apr 06, 2018 at 10:53:58PM +0200, Ralph Böhme wrote:
> > On Fri, Apr 06, 2018 at 01:51:48PM -0700, Jeremy Allison wrote:
> > > On Fri, Apr 06, 2018 at 10:29:54PM +0200, Ralph Böhme wrote:
> > > > On Fri, Apr 06, 2018 at 01:34:41PM -0700, Jeremy Allison wrote:
> > > > > Some people find following this rule onerous enough that I think
> Andrew is correct,
> > > > > we should use an automated solution here.
> > > >
> > > > well, maybe I've been through metze's school of coding for too long.
> One lesson
> > > > you'll learn there: there's exactly one way for a patch to be
> correct. And that
> > > > doesn't include trailing whitespace. :)))
> > > >
> > > > > After all, computers are supposed to be our servants, not the
> > > > > other way sound :-).
> > > >
> > > > sure, if this can be automated let's do that, but my understanding
> was that
> > > > there is stuff where trailing whitespace has a purpose, so we
> couldn't automate
> > > > it. Am I missing something?
> > >
> > > We could enforce on .c and .h files only.
> >
> > If that's possible, sure.
>
> I think it should. It only has to look at lines starting with '+'


Just to up-vote for the automated git-hook solution - we are using Travis
and flake8 for our Python code(Samba also has some .py files, right?) on
commits and though it caused some anticipation in the beginning, now our
code looks much cleaner and consistent in style.

With regards,
Timur Bakeyev.