Re: [boost] [program_options] patch to clear Borland regressions

From: Peter Dimov (pdimov_at_[hidden])
Date: 2006-10-18 10:08:57

John Maddock wrote:

> Vladimir Prus wrote:
>> John Maddock wrote:
>>
>>> The following patch clears the failures with the Borland compiler
>>> for the program options lib.  Is this OK to commit or was there a
>>> specific reason why those functions were declared outside the class
>>> body?  It shouldn't make any difference either which way, but you
>>> never know :-)
>>
>> Looks fine, thanks for fixing this!
>
> OK applied to cvs HEAD and the release branch.  Note however that cvs
> HEAD has other issues - that look to be related to changes in
> Boost.Bind - that prevent the tests passing with Borland's compiler.

CVS-HEAD isn't being tested against Borland, is it?

Someone with access to a variety of Borland versions needs to try to replace 
bind.hpp line 937:

#if 0

with

#if defined( __BORLANDC__ ) && BOOST_WORKAROUND( __BORLANDC__, < 0x??? )

and report the correct value of 0x???. :-)