Anton Khirnov <an...@khirnov.net> writes:

> It's not available on all systems and using argc/argv directly actually
> makes the code simpler.
> ---
>  libavcodec/motion-test.c |   12 +++---------
>  1 file changed, 3 insertions(+), 9 deletions(-)
>
> diff --git a/libavcodec/motion-test.c b/libavcodec/motion-test.c
> index bf63182..2c3a0af 100644
> --- a/libavcodec/motion-test.c
> +++ b/libavcodec/motion-test.c
> @@ -119,15 +119,9 @@ int main(int argc, char **argv)
>      int flags[2] = { AV_CPU_FLAG_MMX, AV_CPU_FLAG_MMX2 };
>      int flags_size = HAVE_MMX2 ? 2 : 1;
>
> -    for(;;) {
> -        c = getopt(argc, argv, "h");
> -        if (c == -1)
> -            break;
> -        switch(c) {
> -        case 'h':
> -            help();
> -            return 1;
> -        }
> +    if (argc > 1) {
> +        help();
> +        return 1;
>      }
>
>      printf("Libav motion test\n");
> -- 

OK

-- 
Måns Rullgård
m...@mansr.com
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to