[FFmpeg-devel] [PATCH] move some bitstream related functions
Aurelien Jacobs
aurel
Fri Jul 6 01:44:01 CEST 2007
On Thu, 5 Jul 2007 20:11:30 +0200
Michael Niedermayer <michaelni at gmx.at> wrote:
> Hi
>
> On Thu, Jul 05, 2007 at 03:39:05PM +0200, Aurelien Jacobs wrote:
> [...]
> > > > +void ff_write_quant_matrix(PutBitContext *pb, uint16_t *matrix){
> > > > + int i;
> > > > +
> > > > + if(matrix){
> > > > + put_bits(pb, 1, 1);
> > > > + for(i=0;i<64;i++) {
> > > > + put_bits(pb, 8, matrix[ ff_zigzag_direct[i] ]);
> > > > + }
> > > > + }else
> > > > + put_bits(pb, 1, 0);
> > > > +}
> > > > +
> > > > /* VLC decoding */
> > >
> > > doesnt belong in bitstream.* at all
> >
> > I have to agree with this. On the other hand, this function is used
> > in h263.c and create a dependency on mpegvideo_enc.c for just one
> > small function. Any idea where I could move this function ?
>
> h263 encoding does depend on most of mpegvideo_enc.c
> h263 decoding does not need ff_write_quant_matrix()
Ok, well. I will try to untangle dependencies on h263.c at the same time.
This should do the trick.
Now what about the simplified patch I sent (which just move ff_copy_bits
to bitstream.c) ?
Aurel
More information about the ffmpeg-devel
mailing list