[FFmpeg-devel] Cleanup libswscale and reimplement GPL code under LGPL

Michael Niedermayer michaelni
Mon Apr 7 15:48:16 CEST 2008


On Mon, Apr 07, 2008 at 03:42:28PM +0200, Luca Barbato wrote:
> Michael Niedermayer wrote:
> > On Mon, Apr 07, 2008 at 03:18:37PM +0200, Luca Barbato wrote:
> >> Michael Niedermayer wrote:
> >>>> Fine by me, still I'd like to have libswscale completely reshaped so it 
> >>>> can be used (if that as additional bonus makes the separate libswscale 
> >>>> tree die, bonus)
> >>> What do you mean by "completely reshaped"?
> >>> And  "can be used"?
> >> The swscale api isn't exactly documented 
> > 
> > Document it ...
> > 
> > 
> >> and last time I had to use it I 
> >> found it a bit different than the rest of libav* and not as nice.
> > 
> > elaborate
> 
> function naming a bit inconsistent (some camelCased some 
> underscore_separated), 5> params in many functions, large use of defines 
> while enums could be used.

send patch


> 
> > 
> > 
> >> I'd 
> >> like to have it in a better shape. Not to mention the internal 
> >> structure, that is a complete pain.
> > 
> > elaborate
> 
> It intermixes mmx code and arch independent code, it uses some kind of 
> templating, adding other arches may lead to an unmaintainable mess.

send patch

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Let us carefully observe those good qualities wherein our enemies excel us
and endeavor to excel them, by avoiding what is faulty, and imitating what
is excellent in them. -- Plutarch
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080407/9f06b8f5/attachment.pgp>



More information about the ffmpeg-devel mailing list