Re: [PATCH 04/22] t/annotate-tests.sh: avoid redundant use of cat
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Mar 6, 2024, 00:26 UTC
- Message-ID
- <xmqq7cigxl1o.fsf@gitster.g>
- In-Reply-To
- <bdae6d2d-af56-4bc0-a000-5cf2ef44cd44@gmail.com>
Rubén Justo <rjusto@gmail.com> writes:
Show 28 quoted lines
> On Tue, Mar 05, 2024 at 02:28:15PM -0800, Junio C Hamano wrote: >> "Beat Bolli" <bb@drbeat.li> writes: >> >> > Signed-off-by: Beat Bolli <dev+git@drbeat.li> >> > --- >> > t/annotate-tests.sh | 2 +- >> > 1 file changed, 1 insertion(+), 1 deletion(-) >> > >> > diff --git a/t/annotate-tests.sh b/t/annotate-tests.sh >> > index 5e21e84f3884..87572459e4b8 100644 >> > --- a/t/annotate-tests.sh >> > +++ b/t/annotate-tests.sh >> > @@ -532,7 +532,7 @@ test_expect_success 'blame -L :funcname with userdiff driver' ' >> > "$(cat file.template)" && >> > test_commit --author "B <B@test.git>" \ >> > "change" "$fortran_file" \ >> > - "$(cat file.template | sed -e s/ChangeMe/IWasChanged/)" && >> > + "$(sed -e s/ChangeMe/IWasChanged/ file.template)" && >> >> Obviously correct, but >> >> "$(sed -e s/ChangeMe/IWasChanged/ <file.template)" && >> >> might be a more faithful conversion (when "sed" looks at its ARGV[], >> it did not find anything before, and it would not find anything >> after this patch). > > Good point. Thank you for being careful.
Heh, I actually consider it the most irrelevant one among my comments. I actally do not think there is a way tell if your "sed" invocation is reading from one of the files listed on the command line, or reading from the standard input, from your sed script, unlike say Perl that has access to @ARGV. Certainly a simple s/A/B/ would not care.
Compared to that, rewriting $(cat file | wc -l) to $(wc -l <file) does matter, which was done in [05/22].