Re: [PATCH 3/4] Added --help screen info about -M / --refresh-on-max in cyclictest

[John Kacur <jkacur@redhat.com>]

On Tue, 22 Dec 2009, David Sommerseth wrote:

> Also renamed --refresh_on_max to --refresh-on-max
> 
> Signed-off-by: David Sommerseth <davids@redhat.com>
> ---
>  src/cyclictest/cyclictest.c |    3 ++-
>  1 files changed, 2 insertions(+), 1 deletions(-)
> 
> diff --git a/src/cyclictest/cyclictest.c b/src/cyclictest/cyclictest.c
> index 223f343..7979a77 100644
> --- a/src/cyclictest/cyclictest.c
> +++ b/src/cyclictest/cyclictest.c
> @@ -788,6 +788,7 @@ static void display_help(int error)
>  	       "-I       --irqsoff         Irqsoff tracing (used with -b)\n"
>  	       "-l LOOPS --loops=LOOPS     number of loops: default=0(endless)\n"
>  	       "-m       --mlockall        lock current and future memory allocations\n"
> +	       "-M       --refresh-on-max  delay updating the screen until a new max latency is hit\n"
>  	       "-n       --nanosleep       use clock_nanosleep\n"
>  	       "-N       --nsecs           print results in ns instead of us (default us)\n"
>  	       "-o RED   --oscope=RED      oscilloscope mode, reduce verbose output by RED\n"
> @@ -910,7 +911,7 @@ static void process_options (int argc, char *argv[])
>  			{"irqsoff", no_argument, NULL, 'I'},
>  			{"loops", required_argument, NULL, 'l'},
>  			{"mlockall", no_argument, NULL, 'm' },
> -			{"refresh_on_max", no_argument, NULL, 'M' },
> +			{"refresh-on-max", no_argument, NULL, 'M' },
>  			{"nanosleep", no_argument, NULL, 'n'},
>  			{"nsecs", no_argument, NULL, 'N'},
>  			{"oscope", required_argument, NULL, 'o'},
> -- 
> 1.6.2.5
> 
> 
Hmnn, I'm not sure that I want to apply this one.
What is the motifivation for changing the name to refresh-on-max?
Right now refresh_on_max matches the global variable.