[Ffmpeg-devel] [PATCH]video codec nine bitstream fixes

Ivan Kalvachev ivan
Mon May 9 01:48:49 CEST 2005


Michael Niedermayer said:
> Hi
>
> On Sunday 08 May 2005 20:29, Alexander Strasser wrote:
>> Hi Michael,
>>
>> Michael Niedermayer wrote:
>> > On Sunday 08 May 2005 18:28, Alexander Strasser wrote:
>> > [...]
>> >
>> > > >  #if HAS_ADVANCED_PROFILE
>> > > >      if (v->profile == PROFILE_ADVANCED)
>> > > >      {
>> > > >          v->level = get_bits(gb, 3);
>> > > > +	if(v->level >= 9)
>> > > > +	{
>> > > > +	    av_log(avctx, AV_LOG_ERROR, "Reserved LEVEL %i\n");
>> > > > +	}
>> > >
>> > >   Didn't you miss v->level in that av_log call?
>> >
>> > not to mention that checking 3 bits >= 9 is a little silly
>>
>>   Right. Also this makes me guess there is some other underlying
>> issue, because if there really is something reserved like 9 or
>> higher how will it work in future when we are reading 3 bits here.
>> Or the 9 is just a typo, or the check doesn't belong there at all
>> and the code was just correct before.

It is typo, it have to be 5 (101b).
yes, I will add the variable :O

>
> yes
>
> all changes to vc9* should be double checked until we have a working vc9
> decoder, as newly introduced bugs cant be detected at all (decoding doesnt
> work yet) and findig bugs in such decoders is quite time consuming ...

That's why I send patch for review, not to say that I wrote it
in nearly half asleep state ;)

Tabs are something that original mc keep adding, despire my efforts
to disable automatic tab insertion.
Do you have some text editor that is light,
have nice syntax highlight and
you can use arrows to navigate (no vi and no emacs pls)

Wish You Best
  Ivan Kalvachev
 iive






More information about the ffmpeg-devel mailing list