[FFmpeg-devel] [PATCH] Prefix with opt_ the functions ffmpeg.c:show_{version, license, formats}

Benoit Fouet benoit.fouet
Fri Aug 10 09:29:57 CEST 2007


Michael Niedermayer wrote:
> On Thu, Aug 09, 2007 at 04:40:17PM +0200, Stefano Sabatini wrote:
>   
>> Hi all.
>>
>> This patch prefixes with opt_ the functions
>> ffmpeg.c:show_{version,license,formats}. 
>>
>> Log message suggested: 
>> Prefix with "opt_" the functions ffmpeg.c:show_{version,license,formats}.
>>
>> "opt_" prefixing seems to me a good idea for the following reasons:
>>
>> * such prefixing is used for all the remaining option handler
>>   functions in ffplay.c and ffmpeg.c.
>>
>> * it avoids conflicts between generic function names and option handler
>>   function names (e.g. show_help and opt_show_help).
>>     
>
> ok
>
> [...]
>   
>   

Applied

-- 
Ben
Purple Labs S.A.
www.purplelabs.com




More information about the ffmpeg-devel mailing list