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???. :-)