Farid Zaripov wrote:
-----Original Message-----
From: Martin Sebor [mailto:[EMAIL PROTECTED] Sent: Tuesday, May 29, 2007 8:39 PM
To: [email protected]
Subject: Re: svn commit: r541672 - /incubator/stdcxx/trunk/include/rw/_config-msvc.h

Well, I guess we need to decide if it's a good idea to maintain two sets of the same chunks of code or if it would be better to factor the code into a common file. If the latter, we need to decide on the mechanism we want to use to use the code with each compiler. Here are the possibilities I see:

1. Status quo: For an Intel C++ installation on top of MSVC _config.h #includes both _config-msvc.h and _config-icc.h, and _config-msvc.h must be prepared to deal with it.

2. Duplicate CRT-specific macro hackery from _config-msvc.h in _config-icc.h for Intel C++ on top of MSVC.

3. Move CRT-specific macros hackery from _config-msvc.h to a new config header, say _config-msvcrt.h, and #include this header from both _config-msvc.h and _config-icc.h, making sure _config-msvc.h is #included only for MSVC and not for Intel C++ on top of it.

Preferences? Any other possible ways to deal with this?

  3 is optimal way, as for me.

Sounds good to me too. I created a new issue/improvement to
track this change, assigned it you, and scheduled it for 4.2:
  https://issues.apache.org/jira/browse/STDCXX-429

Martin

Reply via email to