If you look where the patch applies some changes to the Makefile it checks to see if its being compiled under Darwin (codename for MacOSX).  If it is, then the libavcodec is reverted to its last known good...otherwise it will compile with the SSE code.

<br><br>Attached is an updated version of the patch, it should be a little cleaner.  I've tested it several times and it works fine.<br><br>Enrique<br><br><br>

<div><span class="gmail_quote">On 10/16/07, <b class="gmail_sendername">Jean-Paul Saman</b> <<a href="mailto:jean-paul.saman@planet.nl" target="_blank" onclick="return top.js.OpenExtLink(window,event,this)">jean-paul.saman@planet.nl

</a>> wrote:</span>

<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0px 0px 0px 0.8ex; padding-left: 1ex;">Enrique Osuna wrote: > Resending patch.  Forgot to include -R switch when applying patch to ffmpeg.

<br>

><br>> Enrique<br>><br>> On 10/13/07, Enrique Osuna <<a href="mailto:enrique.osuna@gmail.com" target="_blank" onclick="return top.js.OpenExtLink(window,event,this)">enrique.osuna@gmail.com</a>> wrote:<br>

>> Attached patch is a work around to a problem introduced in ffmpeg r10621.

>> It's associated with new assembly code in ffmpeg/i386/dsputils_mmx.c file. >> The file should compile, but there is a bug with gcc.  This patch will >> revert dsputils_mmx.c to the last known good (where gcc won't error out).

<br>>><br>>> I'll try to bring it up to ffmpeg dev team.<br><br>It looks like this patch is removing support for SSE2 which is probably<br>needed for Windows. Isn't it a better idea to split the patch up in a

windows specific part and MacOSX specific part? And then modify extras/contrib/src/Makefile to only apply the windows part when compiling for Windows. The same counts for the MacOSX part to only apply when compiling for Mac.

<br><br>Gtz,<br>Jean-Paul Saman.<br>_______________________________________________<br>vlc-devel mailing list<br>To unsubscribe or modify your subscription options:<br><a href="http://mailman.videolan.org/listinfo/vlc-devel" target="_blank" onclick="return top.js.OpenExtLink(window,event,this)">

http://mailman.videolan.org/listinfo/vlc-devel</a><br></blockquote></div><br>