[FFmpeg-devel] [PATCH] add fieldorder video filter

Michael Niedermayer michaelni at gmx.at
Mon Apr 4 00:01:42 CEST 2011


On Sun, Apr 03, 2011 at 04:21:01PM +0200, Stefano Sabatini wrote:
> On date Friday 2011-04-01 19:31:41 +0100, Mark Himsley encoded:
> > On 31/03/11 15:15, Stefano Sabatini wrote:
[...]
> [...]
> 
> I see that this should work with all the listed formats, but may break
> if we add support for formats of the kind:
> * bitstream formats with components on plane 2/3
> * formats which have more than one component on planes 2/3 with log2_chroma_h != 0
>   (e.g. a freak variant of NV12, maybe Bayer formats?)
> 
> On the other hand I see that it is not very likely to happen, but I'd
> still prefer to explicitely list the formats in query_formats().
> Other opinions, Michael?

iam already quite happy with the code, i think the author should
decide which way he prefers.
Thinking too much ahead about what theoretically could break one
day is maybe not worth it

[...]
-- 
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: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20110404/016f08d3/attachment.asc>


More information about the ffmpeg-devel mailing list