This is the mail archive of the gcc-patches@gcc.gnu.org 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] Use C++11 direct initialization in Debug associative containers

From: Jonathan Wakely <jwakely at redhat dot com>
To: François Dumont <frs dot dumont at gmail dot com>
Cc: "libstdc++ at gcc dot gnu dot org" <libstdc++ at gcc dot gnu dot org>, gcc-patches <gcc-patches at gcc dot gnu dot org>
Date: Wed, 3 Oct 2018 11:09:46 +0100
Subject: Re: [PATCH] Use C++11 direct initialization in Debug associative containers
References: <053c97a4-ad22-de0a-b958-24b5ef1af524@gmail.com>

On 03/10/18 07:50 +0200, François Dumont wrote:

Just some code cleanup extending use of C++11 direct initialization.


    * include/debug/map.h
    (map<>::emplace<>(_Args&&...)): Use C++11 direct initialization.
    (map<>::emplace_hint<>(const_iterator, _Args&&...)): Likewise.
    (map<>::insert(value_type&&)): Likewise.
    (map<>::insert<>(_Pair&&)): Likewise.
    (map<>::insert<>(const_iterator, _Pair&&)): Likewise.
    (map<>::try_emplace): Likewise.
    (map<>::insert_or_assign): Likewise.
    (map<>::insert(node_type&&)): Likewise.
    (map<>::insert(const_iterator, node_type&&)): Likewise.
    (map<>::erase(const_iterator)): Likewise.
    (map<>::erase(const_iterator, const_iterator)): Likewise.
    * include/debug/multimap.h
    (multimap<>::emplace<>(_Args&&...)): Use C++11 direct initialization.
    (multimap<>::emplace_hint<>(const_iterator, _Args&&...)): Likewise.
    (multimap<>::insert<>(_Pair&&)): Likewise.
    (multimap<>::insert<>(const_iterator, _Pair&&)): Likewise.
    (multimap<>::insert(node_type&&)): Likewise.
    (multimap<>::insert(const_iterator, node_type&&)): Likewise.
    (multimap<>::erase(const_iterator)): Likewise.
    (multimap<>::erase(const_iterator, const_iterator)): Likewise.
    * include/debug/set.h
    (set<>::emplace<>(_Args&&...)): Use C++11 direct initialization.
    (set<>::emplace_hint<>(const_iterator, _Args&&...)): Likewise.
    (set<>::insert(value_type&&)): Likewise.
    (set<>::insert<>(const_iterator, value_type&&)): Likewise.
    (set<>::insert(const_iterator, node_type&&)): Likewise.
    (set<>::erase(const_iterator)): Likewise.
    (set<>::erase(const_iterator, const_iterator)): Likewise.
    * include/debug/multiset.h
    (multiset<>::emplace<>(_Args&&...)): Use C++11 direct initialization.
    (multiset<>::emplace_hint<>(const_iterator, _Args&&...)): Likewise.
    (multiset<>::insert<>(value_type&&)): Likewise.
    (multiset<>::insert<>(const_iterator, value_type&&)): Likewise.
    (multiset<>::insert(node_type&&)): Likewise.
    (multiset<>::insert(const_iterator, node_type&&)): Likewise.
    (multiset<>::erase(const_iterator)): Likewise.
    (multiset<>::erase(const_iterator, const_iterator)): Likewise.

Tested under Linux x86_64.

Already formerly accepted in another mailing thread so committed.


Thanks!

References:
- [PATCH] Use C++11 direct initialization in Debug associative containers
  - From: François Dumont

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