Re: [PATCH v2 20/25] sequencer: left-trim lines read from the script

From: Johannes Schindelin <johannes.schindelin@gmx.de>
Date: Sep 12, 2016, 08:23 UTC
Message-ID: <alpine.DEB.2.20.1609121019290.129229@virtualbox>
In-Reply-To: <xmqqsht657ls.fsf@gitster.mtv.corp.google.com>

Hi Junio,

On Sun, 11 Sep 2016, Junio C Hamano wrote:

Show 8 quoted lines

> Johannes Schindelin <johannes.schindelin@gmx.de> writes:
> 
> > Interactive rebase's scripts may be indented; we need to handle this
> > case, too, now that we prepare the sequencer to process interactive
> > rebases.
> 
> Hmph, have we ever given the sequencer instructions indented to the
> user to edit?

No.

We also never provide the user with an indented todo script in the sequencer. Yet we still parse it. Because Postel's Law.

> I do not offhand see why we want to be lenient here,
> especially only to the left.

Postel's Law.

We do not care about the right: that's only comments (onelines, for the users' pleasure).

Ciao, Dscho