Small memory-leak in MOD decoder

Max Horn max at quendi.de
Mon Oct 1 09:13:50 EDT 2001


At 13:57 Uhr +0200 01.10.2001, Torbjörn Andersson wrote:
>Max Horn wrote:
>
>>  If somebody could tell me what we are talking about here, sure, I will
>>  look at it :) I read the mails about the MP3-ness chaeck, but how is
>>  this related to libtool ??!? what libtool patch?!?
>
>You're right, they're not related. I was just lumping them together 
>as two items that have been discussed but not yet committed to CVS. 
>:-)
>
>>  You mean CVS rev 1.1 vs 1.2 of acinclude.m4, and 1.2 is not working -
>>  well, it contains libtool.m4. But one could argue whether that is a
>>  good idea, since we require libtool anyway... I think it is better to
>>  add a comment regarding this to the readme, and get rid of libtool.m4
>>  in acinclude - this way both users of 1.3.x and 1.4.x can be happy.
>
>Or the bootstrap script could be made to construct the appropriate 
>acinclude.m4 depending on what "libtool --version" says, I guess. 
>Might be overkill though.

It would be - because that is exactly what "aclocal" does :)

aclocal scans the configure.in file for all required macros; then it 
looks through acinclude.m4 to satisfy these requirments; finally, it 
searches $prefix/share/aclocal/ for .m4 files that define those 
macros. The resulting set of macros is put into the file aclocal.m4, 
which then is used by the other auto-tools to resolve the macros.

The idea behind having an acinclude.m4 at all is that if somebody has 
OGG not installed at all, he is also missing the vorbis.m4; but then 
the autotools could not handle the vorbis macros. Solution: put those 
into acinclude.m4. Problem: if a new version of the macros is 
substantially changed, you get problems.

This happend for libtool :/

But since libtool is required (whereas ogg is not), we can safely 
leave out our copy of libtool.m4


Max
-- 
-----------------------------------------------
Max Horn
Software Developer

email: <mailto:max at quendi.de>
phone: (+49) 6151-494890



More information about the sdlsound mailing list