Boost logo

Boost :

From: Vladimir Prus (ghost_at_[hidden])
Date: 2005-05-17 05:43:46


John Maddock wrote:

> The following patches get Boost.Program Options compiling with Borland C++
> 5.6.4 (it's enough to fix the regex grep test program which was what I
> wanted). Is it OK to apply these?

Yes, please!

> I can't see how they would cause
> problems with other compilers: in fact I'm not at all clear how "cmdline"
> was ever found without the qualifier on other compilers.

The 'basic_command_line_parser' class template is derived from
detail::cmdline, and so, I believe the 'cmdline' name should be injected in
the scope of 'basic_command_line_parser'. Since all uses are inside that
class, unqualified 'cmdline' works fine.

> I haven't looked at the runtime test failures with this compiler yet (has
> anyone?),

I'll try to reproduce with my version of borland later this week. But IIRC,
previously tests uses to pass for me while failing on metacomm.

Thanks,
Volodya


Boost list run by bdawes at acm.org, gregod at cs.rpi.edu, cpdaniel at pacbell.net, john at johnmaddock.co.uk