[FFmpeg-devel] [PATCH] rawdec: rename top option to rawdec_top

Michael Niedermayer michaelni at gmx.at
Thu Apr 5 13:16:17 CEST 2012

On Thu, Apr 05, 2012 at 11:55:02AM +0200, Matthieu Bouron wrote:
> 2012/4/4 Michael Niedermayer <michaelni at gmx.at>:
> > On Mon, Apr 02, 2012 at 07:21:07PM +0200, Matthieu Bouron wrote:
> >> Hi,
> >>
> >> This patch rename the raw decoder option -top to -rawdec_top since
> >> -top is already used by ffmpeg.
> >
> > do you have a testcase that is fixed by this patch ?
> The sample test I used is a rawvideo bgr24 interlaced bottom field
> first in avi container. -rawdec_top 0 is required in order to tell the
> decoder that the content is interlaced/bff since it can't detect it
> automatically.

i think it would make sense to support a similar option in other (non
raw) decoders. A different name than -rawdec_top would allow all to
use the same. Not sure which name would be good decoder_tff maybe

also to avoid breaking backward compatibility the -top avoption
should be kept. Some applications might have used it as this existed
in there for almost a year


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/20120405/76a7ce55/attachment.asc>

More information about the ffmpeg-devel mailing list