[FFmpeg-devel] [PATCH] Fix ffmpeg crash on G3 based computers introduced with revision 10637

[Guillaume POIRIER]

Hello,

On Feb 16, 2008 12:47 AM, Alexander Strange <astrange at ithinksw.com> wrote:
>
> On Feb 15, 2008, at 6:27 PM, Patrice Bensoussan wrote:
>
> >
> > On 15 Feb 2008, at 21:33, Guillaume Poirier wrote:
> >
> >> I'd prefer the attached patch.
> >>
> >> Please have a look at it, and test it.
> >
> > I can't test it as I only have an intel based Mac now, but patch looks
> > wrong... (missing not in front of has_altivec()). Otherwise, I am ok
> > with it...
>
> mm_flags & MM_ALTIVEC has already been set, you don't need another
> check.
>
> Also, has_altivec() shouldn't be a global symbol without a prefix...

Right. Patrice, I discarded your patch and applied Alexander's instead.

Guillaume

-- 
A soldier will fight long and hard for a bit of colored ribbon.
 -- Napoleon Bonaparte