[FFmpeg-devel] [PATCH] Set loglevel with strings in ffmpeg.c.

[Stefano Sabatini]

On date Thursday 2009-06-11 18:48:59 -0300, Ramiro Polla encoded:
> On Thu, Jun 11, 2009 at 6:30 PM, Michael Niedermayer<michaelni at gmx.at> wrote:
> > On Thu, Jun 11, 2009 at 02:08:41PM -0300, Ramiro Polla wrote:
> >> $subj makes it more user-friendly, i.e.:
> >> ffmpeg -loglevel info -i input output
> >>
> >> Ramiro Polla
> >
> >> ?ffmpeg.c | ? 34 ++++++++++++++++++++++++++++++++--
> >> ?1 file changed, 32 insertions(+), 2 deletions(-)
> >> b69c3b8cd2469ff83024c20d90ce7b5da1fe5117 ?loglevel.diff
> >
> > iam not sure if this is worth it but i am sure ffmpeg.c is the wrong place
> > ffplay & ffserver should also support that
> 
> libavutil/options.c maybe?

In order to set an AVOption, that option needs to correspond to a
field of a context with an AVClass, which is not the case, so
cmdutils.[hc] is the place.

> I don't want to spend much more time on this patch, but maybe Stefano
> wants to pick it up =)

I always enjoy to improve the interface and move code from apps to
utils code ;-), check attached.

Regards.
-- 
FFmpeg = Furious and Free Most Portentous Erudite Ghost
-------------- next part --------------
A non-text attachment was scrubbed...
Name: extend-opt-loglevel.patch
Type: text/x-diff
Size: 3362 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090612/59a98331/attachment.patch>