This is the mail archive of the mailing list for the GCC project.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] [libstdc++/65839] whatis support for xmethods

On 29/04/15 10:57 -0700, Doug Evans wrote:
On Tue, Apr 28, 2015 at 5:16 AM, Jonathan Wakely <> wrote:
On 27/04/15 15:44 -0700, Doug Evans wrote:

        PR libstdc++/65839
        * python/libstdcxx/v6/ (get_bool_type): New function.
        Replace all lookups of "bool" with this.
        (get_std_size_type): New function.  Replace all lookups of
        with this.
        (ArrayWorkerBase): Rename arg valtype to elem_type for

I'd say ArrayWorkerBase's _valtype is correct and deque and vector are
wrong to use _elemtype.

C++ containers use value_type for the type of the container objects.
Smart pointers use element_type for the type of the owned object. So
using _valtype for containers and _elemtype for unique_ptr would be
consistent with the C++ library types.

Here's v2.
It's assumes the naming cleanup patch has been applied.

Regression tested on amd64-linux.

Looks good, OK for trunk, thanks.

If you want to fix this on the branch too then the renaming patch and
this one are both OK for the gcc-5-branch as well.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]