From: Junio C Hamano <gitster@pobox.com>
Date: Tue, 23 Jul 2013 19:03:05 GMT
Subject: Re: [PATCH 2/5] t4211: demonstrate empty -L range crash
Message-ID: <7vvc416qvq.fsf@alter.siamese.dyndns.org>
In-Reply-To: <20130723175942.GA21019@goldbirke>

SZEDER Gábor <szeder@ira.uka.de> writes:

> You could avoid the 'cat' here and patch in 4/5 by doing $(wc -l <b.c).

Correct.

> (Side question: the test suite is full with similar constructs, i.e.
> redirecting file contents to stdin, but why not just use wc -l b.c,
> i.e. let wc open the file?)

"wc -l <b.c" is the correct form for the purpose of these tests;
otherwise you will see the filename in the output, and you need to
somehow strip it if you are only interested in the line count.