[patch openjdk/icedtea] fix build failures with gcc-4.3

Thomas Fitzsimmons fitzsim at redhat.com
Fri Nov 30 10:40:50 PST 2007


Bernhard Rosenkraenzer wrote:
> On Fri, 30 Nov 2007 13:56:22 +0100, Matthias Klose <doko at ubuntu.com> wrote:
>> Fixes to build with current GCC from the trunk.
> 
> We've been applying a similar patch for a while. Minor nit:
> 
> -#if _MSC_VER >= 1300 // Visual C++ 7.0 or later 
> +#if _MSC_VER >= 1300 || (__GNUC__ * 10 + __GNUC_MINOR__ >= 43) // Visual
> C++ 7.0 or later, GCC-4.3 or later
>  #include <iostream> 
>  #else
>  #include <iostream.h>
> 
> I think it would be better to just use
> 
> #if defined(__GNUC__) || _MSC_VER >= 1300
> 
> #include <iostream> has worked in gcc forever, and #include <iostream.h>
> has been deprecated (but accepted before 4.3) for ages -- so by doing the
> right thing even for older gccs, we can get rid of warnings there.

Agreed.  Matthias, have you signed the SCA yet?  I'd like to start submitting 
patches upstream to the relevant OpenJDK list at the same time we commit them to 
IcedTea.

Tom



More information about the distro-pkg-dev mailing list