[Ffmpeg-devel] swscale patch

Luca Abeni lucabe72
Tue Jul 4 16:06:36 CEST 2006

Hi Michael,
On Tue, 2006-07-04 at 14:39 +0200, Michael Niedermayer wrote:
> > Index: mplayer/libswscale/swsutil.h
> why is this not put in swscale_internal.h ?
> [...]
> > Index: mplayer/libswscale/swscaler_glue.c
> why is this not put in swscale.c ?
Ok; I'll move that code to swscale_internal.h and swscale.c, and I'll
remove swsutil.h and swscaler_glue.c

> [...]
> > Index: mplayer/mplayer.c
> > Index: mplayer/mencoder.c
> that code duplication is unacceptable
This is the code duplication that I tried to avoid by introducing
sws_interface.c and sws_init() in my previous patch, but you did not
like such double level of initialization (if I understand well).

So, the only remaining solution that I can think about is to introduce a
sws_interface.c file containing the definitions of my_alloc() and
my_log(), and to put
sws_global_init(my_alloc, free, my_log);
in mplayer.c and mencoder.c. I do not like this solution much (because
in this way my_alloc() and my_log() cannot be static, and I will have to
create an additional header with their prototypes - or put the
prototypes directly in mplayer.c and mencoder.c). But if you think it's
better, I'll do it. Let me know.

			Thanks again,
Copy this in your signature, if you think it is important:
                               N O    W A R ! ! !

More information about the ffmpeg-devel mailing list