]> git.ipfire.org Git - thirdparty/git.git/commit
t7422: replace confusing printf with echo
authorJeff King <peff@peff.net>
Mon, 23 Jun 2025 10:55:42 +0000 (06:55 -0400)
committerJunio C Hamano <gitster@pobox.com>
Mon, 23 Jun 2025 17:30:40 +0000 (10:30 -0700)
commit1129596dc88a5b0ad53e0d36981bd91c2352cba8
tree3ed8825b558da0b6189f4adda7537df16fc87a8c
parent16bd9f20a403117f2e0d9bcda6c6e621d3763e77
t7422: replace confusing printf with echo

While looping over a counter "i", we do:

  printf "[submodule \"sm-$i\"]\npath = recursive-submodule-path-$i\n" "$i"

So we are passing "$i" as an argument to be filled in, but there is no
"%" placeholder in the format string, which is a bit confusing to read.

We could switch both instances of "$i" to "%d" (and pass $i twice). But
that makes the line even longer. Let's just keep interpolating the value
in the string, and drop the confusing extra "$i" argument.

And since we are not using any printf specifiers at all, it becomes
clear that we can swap it out for echo. We do use a "\n" in the middle
of the string, but breaking this into two separate echo statements
actually makes it easier to read.

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
t/t7422-submodule-output.sh