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.