On 30.03.2015 22:37, Huebl, Axel wrote: > On 30.03.2015 17:42, Stefan Seefeld wrote: >> On 30/03/15 11:16 AM, Huebl, Axel wrote: >>> Hi, >>> >>> >>> adding more Python 3 features, the "contains" method was added (in 1.55.0?) >>> http://www.boost.org/doc/libs/1_55_0/libs/python/doc/news.html >>> >>> Unfortunately, I found a nasty compile bug when compiling with nvcc that >>> I tried to fix in >>> https://github.com/boostorg/python/pull/14 >>> >>> Does anyone know how to test my patch to see if this function is still >>> working as expected? >> >> I have looked at your patch, but am not convinced that this is the right >> fix. It would be best to really map "contains" to "__contains__" to >> preserve the precise semantic of that call. >> Do you see the "incomplete type" error only with nvcc or also with other >> compilers (clang, notably) ? >> >> Thanks, >> Stefan >> > > Hi Stefan, > > > I did only triggered the compile error it with nvcc since that is the > target I want to move Boost.Python to.
oh that wasn't precise enough: to answer your question, I did not try it with clang (yet). > > The main problem is actually in accessing the this->attr() method since > it's return type "const_object_attribute" is not yet fully typed. > > Maybe it is therefore enough to just move the implementation of > api::object_operators<U>::contains > > down to a source file, but I did not find an according > object_core.cpp > > Do you have any hints on that? > > > Thanks, > Axel >
smime.p7s
Description: S/MIME Cryptographic Signature
_______________________________________________ Cplusplus-sig mailing list Cplusplus-sig@python.org https://mail.python.org/mailman/listinfo/cplusplus-sig
