Re: Xvid 1.2.0 is available!

Hello,

On Mon, Dec 1, 2008 at 2:25 PM, Robert Swain <[email protected]> wrote:
> Hello,
>
> Guillaume POIRIER wrote:
>> On Mon, Dec 1, 2008 at 12:01 PM, Michael Militzer <[email protected]> wrote:
>>> This is Xvid 1.2.0 release.
>>>
>>> This release is Xvid 1.2.0 stable release. It is API compatible with
>>> the previous 1.1.3 stable release.
>>>
>>> Changes since 1.1.3:
>>>
>>> * xvidcore library
>>>   - Complete AMD64/EM64T 64-bit support
>>>   - Added support for WIN64 platform
>>
>> [..]
>>
>> Outstanding! Well, it looks like WIN64 support broke Unix support (at
>> least OSX/Leopard).
>> Please consider applying attached patch that fixes compilation without
>> "-fnested-functions"... or, add that option to compiler options.
>
> I'd be interested in this patch, but where is it? :)

The mailing list seems to strip them (I checked the message I sent,
and I didn't forget to attach it).

Here it is inline:
Index: src/xvid.c
===================================================================
RCS file: /xvid/xvidcore/src/xvid.c,v
retrieving revision 1.79
diff -u -r1.79 xvid.c
--- src/xvid.c  30 Nov 2008 16:36:44 -0000      1.79
+++ src/xvid.c  1 Dec 2008 13:03:30 -0000
@@ -28,6 +28,10 @@
 #include <string.h>
 #include <time.h>

+#if !defined(_WIN32)
+  #include <unistd.h>
+#endif
+
 #include "xvid.h"
 #include "decoder.h"
 #include "encoder.h"
@@ -675,7 +679,6 @@

 #else

-  #include <unistd.h>
   info->num_threads = sysconf(_SC_NPROCESSORS_CONF);

 #endif