Sorry for breaking this one. It didn't strike me as an issue because the 
va_copy macro is defined in C99 and has been supported by GCC for quite a 
while, so I thought we could easily rely on that one instead of using the GNU 
extension __va_copy. We could also add an #ifdef for __va_copy as a more 
granular solution.

Cheers,
Jelle
-----Original Message-----
From: Ulrich Wilkens <m...@uwilkens.de>
Date: Wed, 17 Oct 2012 01:39:52 
To: <cdesktopenv-devel@lists.sourceforge.net>
Subject: Re: [cdesktopenv-devel] [PATCH] Fix dtappbuilder on Linux

On 10/17/12 00:42, Marcin Cieslak wrote:

>> One of the latest NetBSD patches have broken dtappbuilder on Linux.
>> This fixes it again.
>
> Could this depend on the compiler version?

Could be. I'm testing on Xubuntu 12.04 and there the va_copy macros are
defined in /usr/lib/gcc/i686-linux-gnu/4.6/include/stdarg.h
respectively /usr/lib/gcc/x86_64-linux-gnu/4.6/include/stdarg.h:

     #if !defined(__STRICT_ANSI__) || __STDC_VERSION__ + 0 >= 199900L || 
defined(__GXX_EXPERIMENTAL_CXX0X__)
     #define va_copy(d,s)    __builtin_va_copy(d,s)
     #endif
     #define __va_copy(d,s)  __builtin_va_copy(d,s)

You can see that __va_copy is always defined whereas va_copy depends on
other conditions which are not fulfilled obviously.

The patch just reverts to the old dtappbuilder code for Linux and that 
code used __va_copy.


-- 
Ulrich Wilkens
Email: m...@uwilkens.de


------------------------------------------------------------------------------
Everyone hates slow websites. So do we.
Make your web apps faster with AppDynamics
Download AppDynamics Lite for free today:
http://p.sf.net/sfu/appdyn_sfd2d_oct
_______________________________________________
cdesktopenv-devel mailing list
cdesktopenv-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/cdesktopenv-devel
------------------------------------------------------------------------------
Everyone hates slow websites. So do we.
Make your web apps faster with AppDynamics
Download AppDynamics Lite for free today:
http://p.sf.net/sfu/appdyn_sfd2d_oct
_______________________________________________
cdesktopenv-devel mailing list
cdesktopenv-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/cdesktopenv-devel

Reply via email to