Oswald Buddenhagen <oswald.buddenhagen@xxxxxx> writes:
> @@ -61,8 +61,8 @@ test_no_confirm () {
> --smtp-server="$(pwd)/fake.sendmail" \
> $@ \
> $patches >stdout &&
> - ! grep "Send this email" stdout &&
> - >no_confirm_okay
> + ! grep "Send this email" stdout &&
> + >no_confirm_okay
> }
It is hard to see what is going on here, but ...
> @@ -1197,7 +1197,7 @@ test_expect_success $PREREQ 'utf8 Cc is rfc2047 encoded' '
> --smtp-server="$(pwd)/fake.sendmail" \
> outdir/*.patch &&
> grep "^ " msgtxt1 |
> - grep "=?UTF-8?q?=C3=A0=C3=A9=C3=AC=C3=B6=C3=BA?= <utf8@xxxxxxxxxxx>"
> + grep "=?UTF-8?q?=C3=A0=C3=A9=C3=AC=C3=B6=C3=BA?= <utf8@xxxxxxxxxxx>"
... I do not think we want this. A long pipeline should be written
without extra indentation like
A |
B |
C
but more like
A |
B |
C
If we do not have it in the coding guidelines document, perhaps we
should add an entry for it.
Thanks.
