[Ffmpeg-devel] [PATCH] Fixes for Darwin/x86
Michael Niedermayer
michaelni
Wed Feb 8 01:38:29 CET 2006
Hi
On Tue, Feb 07, 2006 at 01:29:31PM +0100, Dan Villiom Podlaski Christiansen wrote:
> On 07/02/2006, at 12.52, Michael Niedermayer wrote:
>
> >Hi
> >
> >On Tue, Feb 07, 2006 at 12:05:32PM +0100, Dan Villiom Podlaski
> >Christiansen wrote:
> >>>*ALIGN* isnt mmx specific so doesnt belong in mmx.h
> >>
> >>However, AFAICT it is only used in code that requires MMX. Which
> >>header would you rather have it in?
> >
> >dsputil.h, its not even x86 specific!
>
> Very well. Is this acceptable?
>
> While at it, I made a minor change in the way PIC is defined on Darwin.
how is this related to the align stuff? this should be a seperate patch ...
> --- libavcodec/i386/dsputil_mmx_rnd.h 12 Jan 2006 22:43:17 -0000 1.22
> +++ libavcodec/i386/dsputil_mmx_rnd.h 7 Feb 2006 12:28:06 -0000
> @@ -22,13 +22,15 @@
> * and improved by Zdenek Kabelac <kabi at users.sf.net>
> */
>
> +#include "mmx.h"
> +
why?
and the whitespace change i already complained about is still in there
[...]
--
Michael
More information about the ffmpeg-devel
mailing list