Re: [boost] CVS Bind on MSVC failure

From: Beman Dawes (bdawes_at_[hidden])
Date: 2001-11-29 10:35:58

At 09:47 AM 11/29/2001, Peter Dimov wrote:

 >From: "Beman Dawes" <bdawes_at_[hidden]>
 >> That rang a bell.  I just installed Metrowerks CodeWarrior 7.0 last 
night
 >> and it failed on bind_test.cpp too.  Slightly different location,
 >> however.  See below.
 >>
 >> --Beman
 >>
 >> *** libs/bind/bind_test.cpp ***
 >>
 >> ** Metrowerks CodeWarrior
 >> mwcc -o boosttmp.exe -maxerrors 10 -cwd source -I- -Ic:/boost/site
 >> c:/boost/site/libs/bind/bind_test.cpp
 >> ### mwcc Compiler:
 >> #    File: ..\site\libs\bind\bind_test.cpp
 >> # ----------------------------------------
 >> #     193:      bind<void>(Y(), i, _1, 9, 4)(k);
 >> #   Error:                                 ^
 >> #   illegal use of 'void'
 >
 >This particular test case never worked with MW. Dave originally had it
 >#ifdef-ed out but this is cheating, isn't it? ;-)
 >
 >How should we proceed in cases like this? #ifdef the test case just to 
see
 >a 'pass'? Or keep the test as-is and be honest with users?

I wouldn't want to see something like #if __COMPILER_VERSION <= 100. That 
seems to just hide the problem, and you might even call it "dishonest". 
OTOH, something like #if !defined(BOOST_NO_CWCHAR) seems OK to me.  I guess 
the difference is that the second form openly acknowledges exactly what the 
problem is.

--Beman