[MPlayer-cvslog] r27288 - trunk/configure

[Reimar Döffinger]

On Tue, Jul 15, 2008 at 10:54:23PM +0300, Uoti Urpala wrote:
> And actually testing the macros in configure doesn't seem
> any more useful than testing any other feature of the headers.

Well, alloca is know to not be available in some cases. Not that I know
if someone ever tried to use MPlayer on such a thing or if they even
have ALSA...

> > Of course there are corner cases that are too unlikely and too much
> > effort to test and/or maintain to be worth testing, but that is a
> > different point.
> > That said, does someone remember why we are using that alloca mess
> > instead of the proper malloc variants? Weren't they available in earlier
> > versions or something?
> 
> Using malloc with explicit free() would at least make it more
> complicated and error-prone.

Well, it would at least use portable and standards-compliant features.
Though maybe ALSA working on anything beyond Linux and a few select
hardware platforms is a lost hope anyway...