Re: [PATCH 21/22] sequencer: left-trim the lines read from the script

2016-09-09 Thread Johannes Schindelin
Hi Junio, On Thu, 1 Sep 2016, Junio C Hamano wrote: > Johannes Schindelin 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. > >> > >> s/;

Re: [PATCH 21/22] sequencer: left-trim the lines read from the script

2016-09-09 Thread Johannes Schindelin
Hi Kuba, On Fri, 2 Sep 2016, Jakub Narębski wrote: > Hello Johannes, > > W dniu 01.09.2016 o 16:13, Johannes Schindelin pisze: > > On Thu, 1 Sep 2016, Jakub Narębski wrote: > > >> 'bol' is beginning-of-line, isn't it (a complement to eol)? > > > > Yep. How did you guess? :-) > > Wouldn't

Re: [PATCH 21/22] sequencer: left-trim the lines read from the script

2016-09-01 Thread Jakub Narębski
Hello Johannes, W dniu 01.09.2016 o 16:13, Johannes Schindelin pisze: > On Thu, 1 Sep 2016, Jakub Narębski wrote: >> 'bol' is beginning-of-line, isn't it (a complement to eol)? > > Yep. How did you guess? :-) Wouldn't 'beg' and 'end' instead of 'bol' and 'eol' be easier to understand, thus

Re: [PATCH 21/22] sequencer: left-trim the lines read from the script

2016-09-01 Thread Junio C Hamano
Johannes Schindelin 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. >> >> s/; We need/; we need/ > > Hrmpf. From

Re: [PATCH 21/22] sequencer: left-trim the lines read from the script

2016-09-01 Thread Johannes Schindelin
Hi Kuba, On Thu, 1 Sep 2016, Jakub Narębski wrote: > W dniu 29.08.2016 o 10:06, Johannes Schindelin pisze: > > Subject: [PATCH 21/22] sequencer: left-trim the lines read from the script > > In the subject, it should probably be without "the", as "lines" >

Re: [PATCH 21/22] sequencer: left-trim the lines read from the script

2016-09-01 Thread Jakub Narębski
Hello Johannes, W dniu 29.08.2016 o 10:06, Johannes Schindelin pisze: Subject: [PATCH 21/22] sequencer: left-trim the lines read from the script In the subject, it should probably be without "the", as "lines" are plural. s/left-trim the lines/left-trim lines/ > Inter

[PATCH 21/22] sequencer: left-trim the lines read from the script

2016-08-29 Thread Johannes Schindelin
Interactive rebase's scripts may be indented; We need to handle this case, too, now that we prepare the sequencer to process interactive rebases. Signed-off-by: Johannes Schindelin --- sequencer.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/sequencer.c