[Ffmpeg-devel] Weird line in cabac.h

[Michael Niedermayer]

Hi

On Wed, Oct 18, 2006 at 08:21:40AM +0200, Jindrich Makovicka wrote:
> On Tue, 17 Oct 2006 19:50:07 +0100 (BST)
> M?ns Rullg?rd <mru at inprovide.com> wrote:
> 
> > 
> > Steve Lhomme said:
> > > Hi everyone,
> > >
> > > We're currently adding most of the current FFMPEG to DrFFMPEG and
> > > MSVC spits a warning on the following line in cabac.h:
> > 
> > Telling us the actual warning might have been helpful, don't you
> > think?
> > 
> > > static void put_cabac(CABACContext *c, uint8_t * const state, int
> > > bit){ int RangeLPS= ff_h264_lps_range[2*(c->range&0xC0) + *state];
> > >
> > [...]
> 
> IMO there should be ff_h264_lps_range[0][2*(c->range&0xC0) + *state];
>                                      ^^^
> or better

patch looks ok (and note put_cabac() is unused so this wasnt a real bug)

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

In the past you could go to a library and read, borrow or copy any book
Today you'd get arrested for mere telling someone where the library is