Re: [Boost-bugs] [Boost C++ Libraries] #903: inconsistent usage of function pointer typedefs

Subject: Re: [Boost-bugs] [Boost C++ Libraries] #903: inconsistent usage of function pointer typedefs
From: Boost C++ Libraries (noreply_at_[hidden])
Date: 2007-07-03 14:31:52


#903: inconsistent usage of function pointer typedefs
-------------------------+--------------------------------------------------
  Reporter: sebastinas | Owner: dave
      Type: Patches | Status: new
 Milestone: | Component: Python
   Version: None | Severity: Showstopper
Resolution: None | Keywords:
-------------------------+--------------------------------------------------
Changes (by dave):

  * owner: david_abrahams => dave
  * status: assigned => new
  * severity: => Showstopper

Old description:

> {{{
> 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.
> }}}

New description:

 {{{
 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.
 }}}

--
Ticket URL: <http://svn.boost.org/trac/boost/ticket/903#comment:1>
Boost C++ Libraries <http://www.boost.org/>
Boost provides free peer-reviewed portable C++ source libraries.


This archive was generated by hypermail 2.1.7 : 2017-02-16 18:49:55 UTC