[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