t4018 (funcname patterns): minor cleanups
authorJonathan Nieder <jrnieder@gmail.com>
Sat, 21 May 2011 19:25:14 +0000 (14:25 -0500)
committerJunio C Hamano <gitster@pobox.com>
Sun, 22 May 2011 05:29:31 +0000 (22:29 -0700)
commitd64d6cdc2071d1eb7f6a45118edc42627e6fc692
treec70d6b85dd3d4b2eec31018b2954ade0df1e6bda
parentf792a0b88ec24dd20c29282b4e022c7b48abd59b
t4018 (funcname patterns): minor cleanups

Introduce a test_expect_funcname function to make a diff and apply a
regexp anchored on the left to the function name it writes, avoiding
some repetition.

Omit the space after >, <<, and < operators for consistency with
other scripts.  Quote the <<here document delimiter and $ signs in
quotes so readers don't have to worry about the effect of shell
metacharacters.

Remove some unnecessary blank lines.

Run "git diff" as a separate command instead of as upstream of a pipe
that checks its output, so the exit status can be tested.  In
particular, this way if "git diff" starts segfaulting the test harness
will notice.

Allow "error:" as a synonym for "fatal:" when checking error messages,
since whether a command uses die() or "return error()" is a small
implementation detail.

Anchor some more regexes on the right.

None of the above is very important on its own; the point is just to
make the script a little easier to read and the code less scary to
modify.

Signed-off-by: Jonathan Nieder <jrnieder@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
t/t4018-diff-funcname.sh