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

Michael Niedermayer michaelni
Sun Jan 31 01:20:55 CET 2010


On Sat, Jan 30, 2010 at 09:54:48PM +0100, Stefano Sabatini wrote:
> On date Saturday 2010-01-30 19:39:36 +0100, Michael Niedermayer encoded:
> > On Sat, Jan 30, 2010 at 06:25:46PM +0100, Stefano Sabatini wrote:
> > > On date Saturday 2010-01-30 17:21:27 +0100, Michael Niedermayer encoded:
> > > > On Sat, Jan 30, 2010 at 04:54:08PM +0100, Stefano Sabatini wrote:
> > > > > Hi, as in subject.
> > > > > 
> > > > > Resulting output looks like:
> > > > > name       nb_channels depth is_alpha input  output
> > > > > yuv420p         3         8      n      y       y
> > > > > yuyv422         1         8      n      y       y
> > > > > rgb24           3         8      n      y       y
> > > > > bgr24           3         8      n      y       y
> > > > > yuv422p         3         8      n      y       y
> > > > > yuv444p         3         8      n      y       y
> > > > > yuv410p         3         8      n      y       y
> > > > > yuv411p         3         8      n      y       y
> > > > > gray            1         8      n      y       y
> > > > > monow           1         1      n      y       y
> > > > > monob           1         1      n      y       y
> > > > > pal8            4         8      y      y       n
> > > > 
> > > > > yuvj420p        3         8      n      n       n
> > > > > yuvj422p        3         8      n      n       n
> > > > > yuvj444p        3         8      n      n       n
> > > > 
> > > > no ?
> > > 
> > > This is the complete output for your reference:
> > 
> > it still lists them as no and it should be yes because we do
> > support them if iam not mistaken
> 
> ------------------------------------------------------------------------
> 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

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

Dictatorship naturally arises out of democracy, and the most aggravated
form of tyranny and slavery out of the most extreme liberty. -- Plato
-------------- 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/20100131/f18b03d8/attachment.pgp>



More information about the ffmpeg-devel mailing list