[FFmpeg-devel] [libav-devel] [PATCH] qpeg: use reget_buffer() in decode_frame()

Ronald S. Bultje rsbultje at gmail.com
Thu Apr 28 00:51:23 CEST 2011


Hi,

On Wed, Apr 27, 2011 at 6:07 PM, Stefano Sabatini
<stefano.sabatini-lala at poste.it> wrote:
> On date Wednesday 2011-04-27 17:02:32 -0400, Ronald S. Bultje encoded:
>> Hi,
>>
>> On Wed, Apr 27, 2011 at 4:59 PM, Reimar Döffinger
>> <Reimar.Doeffinger at gmx.de> wrote:
>> >> That is incorrect. reference=1 means only the top field is referenced.
>> >> You need to set reference=3. (Not that it matters, but just to be
>> >> correct.)
>> >
>> > There's obviously a lot of codecs that need fixing in that regard,
>> > at least all recently changed to reget_buffer.
>> > Obvious question is what that reference=0 is there, I naively
>> > would assume that someone had a reason. I guess I'm most likely
>> > wrong though.
>>
>> Looks like a mistake to me also.
>
> I also note that having an int for the "reference" field doesn't help
> to understand its meaning. Expect a patch for converting the int to an
> enum.

It should be a flag field, not really an enum... & 1 means top field,
& means bottom field, & 3 means both.

Ronald


More information about the ffmpeg-devel mailing list