[MPlayer-dev-eng] [PATCH] Mac OS X updates for darwinfixlib, libmpeg2 and ao_macosx

[Steven M. Schultz]

On Tue, 17 Jun 2003, Alex Beregszaszi wrote:

> > In "libmpeg2/cpu_state.c" there is some AltiVec assembly which
> > wouldn't compile with my setup. I don't know anything about altivec
> > assembly, but I guess it shouldn't be compiled anyway unless

> 
> CAN_COMPILE_ALTIVEC -- if gcc supports compilation
>    (always true if HAVE_ALTIVEC is true)
> HAVE_ALTIVEC -- if altivec is supported on this arch
> HAVE_ALTIVEC_H -- ...

	I wonder if it's necessary to have both CAN_COMPILE_ALTIVEC
	and HAVE_ALTIVEC.   Does it make sense to say HAVE_ALTIVEC but
	not be able to compile it?   
	
	Would having only HAVE_ALTIVEC and HAVE_ALTIVEC_H be enough?  I
	know that on a G4 OS/X system HAVE_ALTIVEC is true but
        HAVE_ALTIVEC_H is false.

	With libmpeg2 all I needed to do was copy cpu_state.c from
	mpeg2dec/libmpeg2's CVS version into MPlayer/libmpeg2/ - the
	recent version compile without problem.   How does the version
	of libmpeg2 bundled with MPlayer differ from the CVS version of
	libmpeg2 I wonder?   Would updating the bundled version be 
	sufficient?

	Cheers,
	Steven Schultz