[FFmpeg-devel] [PATCH] Implement a cmdutils functions for listing available elements

[Stefano Sabatini]

On date Sunday 2009-10-25 22:33:14 +0100, Stefano Sabatini encoded:
> On date Sunday 2009-10-25 20:35:24 +0100, Michael Niedermayer encoded:
[...]
> > i wonder if av_log() would be better, i cant think of any real advantage
> > for ff* but maybe one day this function could be used by an application
> > for which fprintf(stderr) is inconvenient
> 
> Yes using av_log() is a fine idea.
>  
> > [...]
> > 
> > > +            printf(
> > 
> > iam sure mixing stderr & stdout like that is not a good idea
> 
> Please check again, I'm using now av_log(NULL, AV_LOG_INFO, ...) for
> the headers / explanations, and printf() for the actual listing,
> I believe that's the expected behavior, as the user wants to
> read/grep/filter from stdout.

After more thought, well I don't believe anymore this is a good idea,
after all this is still an application function and following that
logic we should then always use av_log rather than fprintf.

New patch attached, with some minor cosmetics fixes, and with the
elements sorting.

[...]

Regards.
-- 
FFmpeg = Fundamentalist and Fiendish Maxi Problematic Empowered Ghost
-------------- next part --------------
A non-text attachment was scrubbed...
Name: implement-opt-list.patch
Type: text/x-diff
Size: 6051 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20091114/a9ca1d39/attachment.patch>