Andreas Enge <[email protected]> skribis:

> Am Samstag, 23. Februar 2013 schrieb Ludovic Courtès:
>> > (make-regexp
>> > "^[[:blank:]]*([[:graph:]]+)[[:blank:]]*([[:graph:]]*)(.*)$")))
>> 
>> I think it should be a + after the second [[:blank:]], because there
>> should be space between the interpreter name and its argument.
>
> Sometimes, there is no space at all: In
> #!/usr/bin/perl
> for instance, when there is no argument. Then the "+" version would fail. 
> With "*", it works as expected.

Oh, right.

>> > +                                    (if has-env
>> > +                                      (begin
>> Please align with the ‘h’ here...
>> > +                                        (patch p bin
>> > +                                          (string-append " " second
>> > third)))))
>> ... and with the ‘p’ here.
>
> Ok. The annoying thing with this way of indenting is that lines become very 
> long very quickly; I like keeping them at less than 80 characters.

When it becomes too long, it’s sometimes a sign that an auxiliary
function may be needed.

Thanks!

Ludo’.

Reply via email to