[C++-sig] Re: args.hpp from CVS

[David Abrahams]

Nikolay Mladenov <nickm at sitius.com> writes:

> There is a problem in the args.hpp from the CVS(today)
>
> I am getting this error 
> H:\work\include\ui/selection.h(145) : warning C4355: 'this' : used in
> base member initializer list
> D:\Shared\boost_1_31_0\boost\boost/python/args.hpp(145) : error C2039:
> ',' : is not a member of 'keywords<1>'
>         D:\Shared\boost_1_31_0\boost\boost/python/args.hpp(58) : see
> declaration of 'keywords<1>'
>         H:\work\src\ui\angle_field_py.cpp(130) : see reference to
> function template instantiation 'struct
> boost::python::detail::keywords<2> __cdecl
> boost::python::detail::operator ,(const struct
> boost::python::detail::keywords<1> &,
> char *)' being compiled
>
>
> This fixes it:
>
> $ cvs diff args.hpp
> Index: args.hpp
> ===================================================================
> RCS file: /boost/boost/boost/python/args.hpp,v
> retrieving revision 1.18
> diff -r1.18 args.hpp
> 145c145
> <       return l.operator,(arg(name));
> ---
>>       return l,arg(name);

I'm a little concerned about this fix because using the comma
operator instead of operator, might just be picking up the usual
sequencing operator.  

1. Which version(s) of VC++ are you having this problem with?
2. does it fail with any of the existing tests?
3. Can you enhance one of the tests to reproduce the failure?

-- 
Dave Abrahams
Boost Consulting
www.boost-consulting.com