RE: [pooma-dev] [PATCH] Fix Utilities/Inform.h for icc

[Richard Guenther]

On Tue, 7 Jan 2003, James Crotinger wrote:

> This looks like a case where the logic should probably be reversed, checking
> for the compilers that don't know about std::ios_base::fmtflags.

Yes, but for these we dont know ;) Of course this is fixed by
autodetecting this with a configure test (I'll try to merge the compiler
feature tests using autoconf next).

The patch just makes builds using the intel compiler possible.

Richard.

> To: Jeffrey D. Oldham
> Cc: address@hidden
> Subject: [pooma-dev] [PATCH] Fix Utilities/Inform.h for icc
>
> Hi!
>
> The following patch is necessary to build pooma with icc (apart from
> the LINUXICC.conf file which I added yesterday already). Together with
> this patch all tests pass as expected. Ok to commit?
>
> Richard.
>
> diff --minimal -Nru a/r2/src/Utilities/Inform.h
> b/r2/src/Utilities/Inform.h
> --- a/r2/src/Utilities/Inform.h       Wed Jul 24 10:53:59 2002
> +++ b/r2/src/Utilities/Inform.h       Wed Jul 24 10:53:59 2002
> @@ -278,7 +278,8 @@
>    void output() { flush(); }
>
>  #if (defined(__MWERKS__) && __MWERKS__ >= 0x2301) \
> - || (defined(__GLIBCPP__) && __GLIBCPP__ >= 20001129)
> + || (defined(__GLIBCPP__) && __GLIBCPP__ >= 20001129) \
> + || (defined(__ICC))
>    typedef std::ios_base::fmtflags FmtFlags_t;
>  #else
>    typedef long FmtFlags_t;
>

--
Richard Guenther <address@hidden>
WWW: http://www.tat.physik.uni-tuebingen.de/~rguenth/