[FFmpeg-devel] [PATCH 0/2] Move show_*() code from cmdutils to libs
u at pkh.me
Mon Nov 7 15:33:44 EET 2016
On Fri, Nov 04, 2016 at 09:16:34PM +0100, Michael Niedermayer wrote:
> Printing the available codecs, decoders, encoders, bitstream filters and their
> options and capabilities is usefull for Apps in general and not just ffmpeg.
> This patch-set is also required for moving ffserver into a separate
> repository as it too uses these functions and they otherwise would need to
> be duplicated.
I don't like this. It is *not* usefull for Apps: every app will use its
logger or UI toolkit thing if they need to. They will never use that
except for debugging purpose.
This printing is specific to our tools. If ffserver goes out of the
repository, it's not considered an FFmpeg tool. So you have 2 solutions
here, duplicate or simply drop them.
Exposing printing helps makes no sense to me. Maybe it would make sense as
a string, but even then it's broken by design IMO.
More information about the ffmpeg-devel