[FFmpeg-devel] [PATCH] Implement a cmdutils functions for listing available elements
Sat Nov 14 00:55:09 CET 2009
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
FFmpeg = Fundamentalist and Fiendish Maxi Problematic Empowered Ghost
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 6051 bytes
Desc: not available
More information about the ffmpeg-devel