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: [Patch, libstdc++/64239] Use std::swap instead of swap in regex

From: Tim Shen <timshen at google dot com>
To: Jonathan Wakely <jwakely at redhat dot com>
Cc: "libstdc++" <libstdc++ at gcc dot gnu dot org>, gcc-patches <gcc-patches at gcc dot gnu dot org>
Date: Thu, 8 Jan 2015 11:44:11 -0800
Subject: Re: [Patch, libstdc++/64239] Use std::swap instead of swap in regex
Authentication-results: sourceware.org; auth=none
References: <CAG4ZjN=UdMVQUw2qmT0jCnqfn_Ed-yqCjRPG18HzrnLE-o3p7w at mail dot gmail dot com> <20150107123919 dot GC3360 at redhat dot com>

On Wed, Jan 7, 2015 at 4:39 AM, Jonathan Wakely <jwakely@redhat.com> wrote:
> No blank line here in the ChangeLog.

Done.

> This is swapping iterators, which can be user-defined types, so it
> should support finding a swap function by ADL, i.e.

Done.

> Also, there should be a new test, since apparently we don't have any
> test that tries to call match_results::swap(match_results&).

Done.


-- 
Regards,
Tim Shen

Attachment: a.diff
Description: Text document

Follow-Ups:
- Re: [Patch, libstdc++/64239] Use std::swap instead of swap in regex
  - From: Jonathan Wakely

References:
- [Patch, libstdc++/64239] Use std::swap instead of swap in regex
  - From: Tim Shen
- Re: [Patch, libstdc++/64239] Use std::swap instead of swap in regex
  - From: Jonathan Wakely

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