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

[Matthieu Bouron]

2012/4/5 Michael Niedermayer <michaelni at gmx.at>:
> 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
>

Maybe we should stick to the -top option in decoders and add an input
option in ffmpeg ?
Example: ffmpeg -decoder_top 0 -i input -decoder_top 1 -i input2 [...] output

>
> [...]
>
> --
> 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
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>