[FFmpeg-devel] Path for minor problems related to ffmpeg help text and x264 configuration

[Michael Niedermayer]

On Thu, Sep 25, 2008 at 09:43:12PM +0200, Stefano Sabatini wrote:
> On date Tuesday 2008-09-23 14:35:03 +0200, Francesco Cosoleto encoded:
> > Hi,
> > 
> > I am not able to submit this simple patch using the bug-tracker due to a
> > server error, but here I think file attachment works, then here the file.
> > 
> > * ffmpeg.c: Fixed a problem in printing option when ffmpeg is compiled
> > without --enable-swscale
> > * libavcodec/utils.c, libavcodec/utils.h: Fixed documentation for direct mv
> > prediction mode automatic
> > * libavcodec/libx264.c: Added support for PSNR flag, code cosmetic changes
> > to use x264_param2string()
> > * ffpresets/libx264-fastfirstpass, ffpresets/libx264-default.ffpres,
> > ffpresets/libx264-max.ffpreset, ffpresets/libx264-hq.ffpreset,
> > ffpresets/libx264-normal.ffprese: 'me' replaced with 'me_method', now 'me'
> > doesn't exist anymore in options list.
> > 
> > Regards,
> > Francesco Cosoleto
> 
> > Index: ffmpeg.c
> > ===================================================================
> > --- ffmpeg.c	(revisione 15383)
> > +++ ffmpeg.c	(copia locale)
> > @@ -3529,7 +3529,9 @@
> >      printf("\n");
> >      av_opt_show(avformat_opts, NULL);
> >      printf("\n");
> > +#if ENABLE_SWSCALE
> >      av_opt_show(sws_opts, NULL);
> > +#endif
> >  }
> 
> Looks OK.

i disagree, we are moving to making swscale default and this one hasnt
been complained about until now so i dont see why we now would
have to add ifdeffery. Even more so ifdeffery that will be cruft soon.

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

No great genius has ever existed without some touch of madness. -- Aristotle
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080925/a2f1733e/attachment.pgp>