Re: [v3 PATCH] Implement C++17 GB50 resolution

2017-02-19 Thread Jonathan Wakely
On 18/02/17 20:48 +, Dinka Ranns wrote: Comments addressed. Please find the new diff attached to this e-mail. Changelog after review comments : 2017-02-18 Dinka Ranns GNU ChangeLog rules say two spaces after the date and after your name. C++17 GB50 resolution * include/st

Re: [v3 PATCH] Implement C++17 GB50 resolution

2017-02-18 Thread Dinka Ranns
Comments addressed. Please find the new diff attached to this e-mail. Changelog after review comments : 2017-02-18 Dinka Ranns C++17 GB50 resolution * include/std/chrono: (duration::operator++()): Add _GLIBCXX17_CONSTEXPR. (duration::operator++(int)): Likewise

Re: [v3 PATCH] Implement C++17 GB50 resolution

2017-02-15 Thread Jonathan Wakely
Hi, Dinka, thanks for the patch. On 14/02/17 21:22 +, Dinka Ranns wrote: diff --git a/libstdc++-v3/include/std/chrono b/libstdc++-v3/include/std/chrono index ceae7f8..6a6995c 100644 --- a/libstdc++-v3/include/std/chrono +++ b/libstdc++-v3/include/std/chrono @@ -349,50 +349,50 @@ _GLIBCXX_END

Re: [v3 PATCH] Implement C++17 GB50 resolution

2017-02-14 Thread Ville Voutilainen
On 14 February 2017 at 23:22, Dinka Ranns wrote: > C++17 GB50 resolution > * libstdc++-v3/include/std/chrono: Pardon me for not noticing this while looking at the earlier versions of this patch, but these should not include the libstdc++-v3 prefix, so it should be * include/std/c