[FFmpeg-devel] [PATCH] Define cmdline_program_name and use it in show_help

Stefano Sabatini stefano.sabatini-lala
Mon Oct 27 21:18:46 CET 2008


On date Monday 2008-10-27 13:46:24 +0100, Luca Abeni encoded:
> Hi all,
> 
> Aurelien Jacobs wrote:
> [...]
> >>> Index: ffmpeg.c
> >>> ===================================================================
> >>> --- ffmpeg.c	(revision 15712)
> >>> +++ ffmpeg.c	(working copy)
> >>> @@ -71,6 +71,7 @@
> >>>  #undef exit
> >>>  
> >>>  const char program_name[] = "FFmpeg";
> >>> +const char* cmdline_program_name = NULL;
> >>>   
> >> why is it not static ? btw, I'd prefer char *foo to char* foo, but maybe
> >> that's just me...
> > 
> > I also prefer char *foo.
> > And no need to initialize it to NULL.

OK.
 
> Maybe this is a stupid question, but... Why using a global variable instead
> of passing argv[0] as an argument to show_help() (in ffplay.c and ffmpeg.c)?

For consistency with program_name, no strong opinion on this.

And yes the variable should be declared static as well program_name
and program_birthyear, patch will follow for that.

Regards.
-- 
FFmpeg = Faboulous & Free Meaningful Patchable Ecstatic Gem
-------------- next part --------------
A non-text attachment was scrubbed...
Name: define-cmdline-program-name-01.patch
Type: text/x-diff
Size: 2796 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20081027/30e8e806/attachment.patch>



More information about the ffmpeg-devel mailing list