Re: PyCXX question on __module__ attribute

[Barry S. <ba...@ba...>]

I have implemented this fix as you suggested in r341.
PyCxx uses PyCFunction_NewEx in all cases  now.

Barry


> On 28 Apr 2016, at 19:02, kishor sharma <kis...@gm...> wrote:
> 
> Sorry I hit enter too soon.
> 
> >You might like to add a bug report to tack this issue. If you have a patch to add __module__ I’ll review and add.
> 
> I kind of figured out the issue. Issue is we are using PyCFunction_New to register the modules in the method.  If we replace PyCFunction_New with PyCFunction_NewEx it solves the problem with module level function 
> 
> e.g. PyObject* func = PyCFunction_NewEx(&method_def->ext_meth_def, new_reference_to(args), PyString_FromString(m_module_name.c_str())
> 
> I am not sure about how to fix problem for function in the extension class. Any help/pointers?
> 
> On Thu, Apr 28, 2016 at 11:29 PM, kishor sharma <kis...@gm... <mailto:kis...@gm...>> wrote:
> >You might like to add a bug report to tack this issue. If you have a patch to add __module__ I’ll review and add.
> 
> I kind of figured out the issue. Issue is we are using PyCFunction_New to register the modules in the method.  If we replace PyCFunction_New with PyCFunction_NewEx it solves the problem with module level function 
> 
> e.g. PyObject* func = PyCFunction_NewEx(
> 
> 
> 
> ------------------------------------------------------------------------------
> Find and fix application performance issues faster with Applications Manager
> Applications Manager provides deep performance insights into multiple tiers of
> your business applications. It resolves application problems quickly and
> reduces your MTTR. Get your free trial!
> https://ad.doubleclick.net/ddm/clk/302982198;130105516;z_______________________________________________
> CXX-Users mailing list
> CXX...@li...
> https://lists.sourceforge.net/lists/listinfo/cxx-users