Re: [PATCH v13 2/6] test-parse-options: print quiet as integer
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Apr 12, 2016, 21:33 UTC
- Message-ID
- <xmqq60vmpl5c.fsf@gitster.mtv.corp.google.com>
- In-Reply-To
- <01020153faf9fcbb-adadd774-db3a-43ce-b908-d08ac1312b31-000000@eu-west-1.amazonses.com>
Pranit Bauva <pranit.bauva@gmail.com> writes:
> Current implementation of parse-options.c treats OPT__QUIET() as integer > and not boolean and thus it is more appropriate to print it as integer > to avoid confusion.
There is no "confusion" factor involved, as we do not use native "boolean" type in our C code. IIUC, the reason why we want to do this is because we may want to see how it would affect the value of the underlying variable to give multiple --quiet options from the command line, which is a policy issue (i.e. we want to allow commands to react to multiple quiet options differently), not an implementation one (i.e. "current implementation happens to use integer").
We would want to see how multiple --quiet options affect the value of the underlying variable (we may want "--quiet --quiet" to still be 1, or we may want to see the value incremented to 2). Show the value as integer to allow us to inspect it.
perhaps?
Show 144 quoted lines
>
> Signed-off-by: Pranit Bauva <pranit.bauva@gmail.com>
> ---
> t/t0040-parse-options.sh | 26 +++++++++++++-------------
> test-parse-options.c | 2 +-
> 2 files changed, 14 insertions(+), 14 deletions(-)
>
> diff --git a/t/t0040-parse-options.sh b/t/t0040-parse-options.sh
> index 477fcff..450da45 100755
> --- a/t/t0040-parse-options.sh
> +++ b/t/t0040-parse-options.sh
> @@ -64,7 +64,7 @@ timestamp: 0
> string: (not set)
> abbrev: 7
> verbose: 0
> -quiet: no
> +quiet: 0
> dry run: no
> file: (not set)
> EOF
> @@ -164,7 +164,7 @@ timestamp: 0
> string: 123
> abbrev: 7
> verbose: 2
> -quiet: no
> +quiet: 0
> dry run: yes
> file: prefix/my.file
> EOF
> @@ -184,7 +184,7 @@ timestamp: 0
> string: 321
> abbrev: 10
> verbose: 2
> -quiet: no
> +quiet: 0
> dry run: no
> file: prefix/fi.le
> EOF
> @@ -212,7 +212,7 @@ timestamp: 0
> string: 123
> abbrev: 7
> verbose: 0
> -quiet: no
> +quiet: 0
> dry run: no
> file: (not set)
> arg 00: a1
> @@ -235,7 +235,7 @@ timestamp: 0
> string: (not set)
> abbrev: 7
> verbose: 0
> -quiet: no
> +quiet: 0
> dry run: no
> file: (not set)
> EOF
> @@ -264,7 +264,7 @@ timestamp: 0
> string: 123
> abbrev: 7
> verbose: 0
> -quiet: no
> +quiet: 0
> dry run: no
> file: (not set)
> EOF
> @@ -303,7 +303,7 @@ timestamp: 0
> string: (not set)
> abbrev: 7
> verbose: 0
> -quiet: no
> +quiet: 0
> dry run: no
> file: (not set)
> arg 00: --quux
> @@ -323,7 +323,7 @@ timestamp: 1
> string: (not set)
> abbrev: 7
> verbose: 0
> -quiet: yes
> +quiet: 1
> dry run: no
> file: (not set)
> arg 00: foo
> @@ -345,7 +345,7 @@ timestamp: 0
> string: (not set)
> abbrev: 7
> verbose: 0
> -quiet: no
> +quiet: 0
> dry run: no
> file: (not set)
> EOF
> @@ -374,7 +374,7 @@ timestamp: 0
> string: (not set)
> abbrev: 7
> verbose: 0
> -quiet: no
> +quiet: 0
> dry run: no
> file: (not set)
> EOF
> @@ -399,7 +399,7 @@ timestamp: 0
> string: (not set)
> abbrev: 7
> verbose: 0
> -quiet: no
> +quiet: 0
> dry run: no
> file: (not set)
> EOF
> @@ -430,7 +430,7 @@ timestamp: 0
> string: (not set)
> abbrev: 7
> verbose: 0
> -quiet: no
> +quiet: 0
> dry run: no
> file: (not set)
> EOF
> @@ -449,7 +449,7 @@ timestamp: 0
> string: (not set)
> abbrev: 7
> verbose: 0
> -quiet: no
> +quiet: 0
> dry run: no
> file: (not set)
> EOF
> diff --git a/test-parse-options.c b/test-parse-options.c
> index 2c8c8f1..86afa98 100644
> --- a/test-parse-options.c
> +++ b/test-parse-options.c
> @@ -90,7 +90,7 @@ int main(int argc, char **argv)
> printf("string: %s\n", string ? string : "(not set)");
> printf("abbrev: %d\n", abbrev);
> printf("verbose: %d\n", verbose);
> - printf("quiet: %s\n", quiet ? "yes" : "no");
> + printf("quiet: %d\n", quiet);
> printf("dry run: %s\n", dry_run ? "yes" : "no");
> printf("file: %s\n", file ? file : "(not set)");
>
>
> --
> https://github.com/git/git/pull/218