[FFmpeg-devel] [PATCH] move QUANT_SHIFT #define to a sensible place

Diego Biurrun diego
Wed Nov 26 10:59:18 CET 2008


On Wed, Nov 26, 2008 at 03:07:31AM +0100, Michael Niedermayer wrote:
> On Wed, Nov 26, 2008 at 01:41:25AM +0100, Diego Biurrun wrote:
> > On Mon, Nov 24, 2008 at 04:26:53PM +0100, Michael Niedermayer wrote:
> > > On Mon, Nov 24, 2008 at 02:08:14PM +0100, Diego Biurrun wrote:
> > > > QUANT_SHIFT is only used in one function in h264.c, but it is hidden
> > > > somewhere between long tables in h264data.h.  This patch moves it to
> > > > h264.c.
> > > 
> > > both the table as well as the single function using it belong to our
> > > unfinished h264 encoder
> > > 
> > > also the values of the table are scaled by QUANT_SHIFT, thus the define
> > > does in some sense belong to the table
> > 
> > OK, alternative patch attached.  
> 
> ok

Applying in a moment.

Diego




More information about the ffmpeg-devel mailing list