[FFmpeg-devel] [PATCH 3/3] avcodec/ff_init_vlc_sparse: use a local array for static vlcs avoiding malloc()

Michael Niedermayer michaelni at gmx.at
Mon Jul 29 17:01:16 CEST 2013


On Sat, Jul 13, 2013 at 02:27:52PM +0200, Michael Niedermayer wrote:
> On Sat, Jul 13, 2013 at 02:13:42PM +0200, Nicolas George wrote:
> > Le quintidi 25 messidor, an CCXXI, Michael Niedermayer a écrit :
> > > the largest allocation i could spot was 1296, 1500 is a bit rounded up
> > > from that. If some future codec needs more then this would need to
> > > be increased
> > 
> > Thanks. Then I believe a comment should explain that:
> > 
> > 	/* maximum known size: 1296 for $codec ($sample) */
> 
> it doesnt depend on the sample, it also couldnt because it sets static
> tables and they are shared between samples
> 
> added note about the max and codec

applied

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Concerning the gods, I have no means of knowing whether they exist or not
or of what sort they may be, because of the obscurity of the subject, and
the brevity of human life -- Protagoras
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20130729/4f3e7bc8/attachment.asc>


More information about the ffmpeg-devel mailing list