Re: [PATCH v2 04/20] t0040-parse-options: test parse_options() with various 'parse_opt_flags'

SZEDER Gábor [off-list ref] writes:

On Fri, Aug 19, 2022 at 11:18:20AM -0700, Junio C Hamano wrote:

quoted

SZEDER Gábor [off-list ref] writes:

quoted

+static void print_args(int argc, const char **argv)
+{
+	for (int i = 0; i < argc; i++)
+		printf("arg %02d: %s\n", i, argv[i]);
+}

It is not November 2022 yet (cf. Documentation/CodingGuidelines).

Oh, I've misunderstood Ævar's remarks about this in the previous
round, and thought it's fair game.

If we make it a "fair game", when we find a platform that has
problems with the syntax, we will have to find them and fix up many
places.  At least the number of the ones we let in by mistake are
small and known, it may be still be manageable.

It is how to be conservative.

It doesn't matter for these tests, but 'test-tool parse-options' uses
the same format to print args:

As long as 99 is enough for us, I do not very much care.  I just
noticed an attempt to align that does not do a thorough job at it,
and found it strange, that's all.