[FFmpeg-devel] [Patch] Fix for static leaks in h264.c
Wed Jun 25 15:14:11 CEST 2008
On Wed, Jun 25, 2008 at 9:01 AM, M?ns Rullg?rd <mans at mansr.com> wrote:
> Michael Niedermayer wrote:
> > On Tue, Jun 24, 2008 at 08:34:34PM -0400, Art Clarke wrote:
> >> This is a continuation of the patch for leaks as in:
> >> http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/2008-June/048859.html
> >> Same test case applies; this time though pass in a test file that
> >> the h263 decoder. This one is slightly hairier than h263.c because I
> had to
> >> unroll some loops to get sufficient space in the data segment. Yes, it
> >> makes me feel squeamish to look at it too.
> > Maybe init_vlc( ..., INIT_VLC_USE_NEW_STATIC) could be used in a loop
> > a table of the sizes
> That doesn't work. The size of static arrays must be an integer constant
> expression. Such an expression can, by definition, not vary in a loop.
> M?ns Rullg?rd
> mans at mansr.com
> ffmpeg-devel mailing list
> ffmpeg-devel at mplayerhq.hu
Agree with both of you, but Michael does point the way to a more elegant
solution I think.
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
More information about the ffmpeg-devel