[FFmpeg-devel] [PATCH] Extend show_pix_fmts() to make it print the input/output support

Stefano Sabatini stefano.sabatini-lala
Sun Jan 31 11:15:45 CET 2010


On date Sunday 2010-01-31 02:57:48 +0100, Michael Niedermayer encoded:
> On Sun, Jan 31, 2010 at 02:38:43AM +0100, Stefano Sabatini wrote:
> > On date Sunday 2010-01-31 01:20:55 +0100, Michael Niedermayer encoded:
> > > On Sat, Jan 30, 2010 at 09:54:48PM +0100, Stefano Sabatini wrote:
> > [...]
> > > > ------------------------------------------------------------------------
> > > > r19993 | lucabe | 2006-09-27 13:41:36 +0200 (Wed, 27 Sep 2006) | 2 lines
> > > > 
> > > > Add support for YUVJ formats
> > > > ------------------------------------------------------------------------
> > > > 
> > > > but then if you look at the diff there is no addition to the
> > > > sws_isSupported{In,Out} macros.
> > > 
> > > exact, thats how it was intended
> > > adding them there at least feels odd to me. They are not true pixel formats
> > > but hacks specific to jpeg in libavcodec. They are deprected and should be
> > > removed (that is from libavcodec first and swscale second)
> > > adding them to more places is not pretty ...
> > > but they must be tested by swscale-test, that said iam not against your
> > > patch i just dont feel its pretty
> > 
> > Another variant which doesn't require to declare yuvj* support, and
> > still makes it possible to test them with my swscale-test-all.sh
> > script.
> 
> add them to the macro as you originally intended

Done.
-- 
FFmpeg = Fast & Fantastic Miracolous Portable Eager Gnome



More information about the ffmpeg-devel mailing list