Re: Заместител на libmp4v2?

[Dimitur Kirov]

Yavor Doganov <address@hidden> wrote:
> Кръпката е
> http://patch-tracking.debian.net/patch/series/view/lynkeos.app/1.2-4/05_ffmpeg_build_fix,

Имам два коментара на сляпо (не съм дърпал кода).

1.)
+static struct SwsContext *img_convert_ctx;
тази променлива се ползва само във функцията ffmpegConvertCurrentFrame
значи спокойно може да е локална за дадената функция.

например:
{{{
int ffmpegConvertCurrentFrame (FfmpegMovie* movie)
{
struct SwsContext *img_convert_ctx;

img_convert_ctx = sws_getContext (movie->pCodecCtx->width,
.....
}
}}}


2.)
според кодът на swscale [1]
sws_getContext заделя динамично памет за върнатия резултат,
използвайки собствен алокатор av_malloc(...)

значи в края на функцията ffmpegConvertCurrentFrame
"img_convert_ctx" трябва да се освободи
av_free (img_convert_ctx);


[1] http://gnunet.org/libextractor/doxygen/html/imgresample_8c-source.html
===
Киров