Boost logo

Boost :

From: SourceForge.net (noreply_at_[hidden])
Date: 2007-04-25 09:06:25


Patches item #1707377, was opened at 2007-04-25 15:06
Message generated for change (Tracker Item Submitted) made by Item Submitter
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=307586&aid=1707377&group_id=7586

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Python
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Sebastian Ramacher (sebastinas)
Assigned to: David Abrahams (david_abrahams)
Summary: inconsistent usage of function pointer typedefs

Initial Comment:
The us of the function pointer typedef 'convertible_function' is inconsistent in boost/python/converter/registry.hpp and libs/python/converter/registry.cpp.

In registry.hpp 'insert' is defined as 'void insert(void* (*convert)(PyObject*), type_info)' whereas it's 'void insert(convertible_function, type_info)' in registry.cpp. The same applies to 'void insert(convertible_function, constructor_function, type_info)' and 'void push_back(convertible_function, constructor_function, type_info)'.

The attached patch replaces the function pointers with the appropriate typedefs.

----------------------------------------------------------------------

You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=307586&aid=1707377&group_id=7586

-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/
_______________________________________________
Boost-bugs mailing list
Boost-bugs_at_[hidden]
https://lists.sourceforge.net/lists/listinfo/boost-bugs


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