[Fix in git] Wrong type signatures?

[Haoyu Bai]

In python/converter/builtin_converters.hpp, I found these lines:

BOOST_PYTHON_TO_PYTHON_BY_VALUE(signed BOOST_PYTHON_LONG_LONG, ::PyLong_FromLongLong(x), &PyInt_Type) BOOST_PYTHON_TO_PYTHON_BY_VALUE(unsigned BOOST_PYTHON_LONG_LONG, ::PyLong_FromUnsignedLongLong(x), &PyInt_Type)

and,

BOOST_PYTHON_TO_PYTHON_BY_VALUE(std::wstring, ::PyUnicode_FromWideChar(x.data(),implicit_cast<ssize_t>(x.size())), &PyString_Type)

Seems it is a typo. Should the PyInt_Type be PyLong_Type, and the PyString_Type be PyUnicode_Type?

These macro argument is used for generate the return value of get_pytype(). And seems get_pytype() is only used to generate docstring. So there's accutally no problem caused by these.

But, should these be typo? If so, we'd better fix it.

Thanks!

[Dave Abrahams]

The best way to know if these are bugs that need to be fixed is to produce test cases (a .cpp and a matching .py file—see ​this directory for examples) that currently fail and would stop failing if the fixes were made. Can you do that and post an attachment?

Thanks!

[Haoyu Bai]

Patch to fix the typo

[Haoyu Bai]

Testcase showing the problem.

[Haoyu Bai]

Testcase showing the problem.

[Haoyu Bai]

Testcase and patch attached.

Without the patch, the test giving the following result:

Docstring say:  
return_longlong() -> int :

    C++ signature :
        long long return_longlong()
But actually type of return value is: <type 'long'>

Docstring say:  
return_wstring() -> str :

    C++ signature :
        std::basic_string<wchar_t, std::char_traits<wchar_t>, std::allocator<wchar_t> > return_wstring()
But actually type of return value is: <type 'unicode'>

Notice the "return_longlong() -> int" and "return_wstring() -> str".

With the patch, the result seems correct:

Docstring say:  
return_longlong() -> long :

    C++ signature :
        long long return_longlong()
But actually type of return value is: <type 'long'>

Docstring say:  
return_wstring() -> unicode :

    C++ signature :
        std::basic_string<wchar_t, std::char_traits<wchar_t>, std::allocator<wchar_t> > return_wstring()
But actually type of return value is: <type 'unicode'>

Also, this patch don't affect other test cases, as they are all passed. I think it is just related to the docstring and won't touch any other things.

[troy d. straszheim]

Looks like the fix made it to the trunk, but not the test, and I notice:

print return_string.doc

return_string() -> str :

C++ signature :

std::string return_string()

print return_wstring.doc

return_wstring() -> unicode :

C++ signature :

std::basic_string<wchar_t, std::char_traits<wchar_t>, std::allocator<wchar_t> > return_wstring()

that that basic_string<wchar... should be simplified to std::wstring for readability. Working on it.

[troy d. straszheim]

Fix here:

​http://gitorious.org/~straszheim/boost/straszheim/commit/aeefdf7abca8c33c25a98a5f7fc78e4e888f9c94

also clean up printing of std::wstring as the gcc demangler was being messy about it.