This is the mail archive of the libstdc++@gcc.gnu.org mailing list for the libstdc++ 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: Default associative containers constructors/destructor/assignment

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: Fri, 9 Dec 2016 09:34:53 +0000
Subject: Re: Default associative containers constructors/destructor/assignment
Authentication-results: sourceware.org; auth=none
References: <87049605-cb4e-a451-01f7-4cf106dd0633@gmail.com> <20161117175241.GQ3145@redhat.com> <22ae1824-80d7-0875-7693-67660414b428@gmail.com> <20161206115425.GG6326@redhat.com> <184d89e1-8991-77f6-dfb1-c750a7109267@gmail.com>

On 08/12/16 21:40 +0100, François Dumont wrote:

On 06/12/2016 12:54, Jonathan Wakely wrote:
On 20/11/16 19:14 +0100, François Dumont wrote:
Talking about _M_color I just realize that move semantic doesn'tcopy _M_color. Shouldn't we capture it along with all the other_M_header information ?
Is it needed? I think the current code does the right thing, doesn't
it? I'd prefer not to change it without a testcase showing why we need
to.
I was hoping you knew it. So I kept it unchanged and will add to myTODO to check this.


I think it's OK already :-)

Committed now.


Thanks!

References:
- Re: Default associative containers constructors/destructor/assignment
  - From: Jonathan Wakely
- Re: Default associative containers constructors/destructor/assignment
  - From: François Dumont

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