[PATCH] t7422: fix extra printf argument, eliminate loops
- From
- Subhaditya Nath <sn03.general@gmail.com>
- Date
- Apr 9, 2025, 19:11 UTC
- Message-ID
- <20250409191139.29644-1-sn03.general@gmail.com>
- In-Reply-To
- <xmqqr021qkeh.fsf@gitster.g>
On Wed Apr 9, 2025 at 11:36 PM IST, Junio C Hamano wrote:
Show 8 quoted lines
> The other one in the example in this thread could lose the for loop > by doing > > printf "160000 commit $COMMIT\trecursive-submodule-path-%d\n" \ > $(test_seq 2000) > > but for uniformity with other parts that cannot lose "for" loop, I > would not recommend going in that direction.
Interesting. I didn't know printf could be used like so.
The patch I sent earlier can also be re-written to lose the loop by simply passing the output of test_seq via 'sed p' to print each number twice!