[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
Sat Jul 13 14:27:52 CEST 2013


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

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

When you are offended at any man's fault, turn to yourself and study your
own failings. Then you will forget your anger. -- Epictetus
-------------- 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/20130713/8e6590ad/attachment.asc>


More information about the ffmpeg-devel mailing list