Re: [PATCH] libstdc++: Fix arithmetic bug in chrono::year_month::operator+

2020-08-27 Thread Jonathan Wakely via Gcc-patches
form is rejected it seems. Does the following look OK to commit? OK, thanks. -- >8 -- Subject: [PATCH] libstdc++: Fix arithmetic bug in chrono::year_month::operator+ This fixes the months-based addition for year_month when the year_month's month component is zero. libstdc++-v3/Chang

Re: [PATCH] libstdc++: Fix arithmetic bug in chrono::year_month::operator+

2020-08-27 Thread Patrick Palka via Gcc-patches
ike it. > > Would either int(unsigned(__my)) or static_cast(__ym) make sense > instead, do avoid people wondering about it in future? The first form works for me. The second form is rejected it seems. Does the following look OK to commit? -- >8 -- Subject: [PATCH] libstdc++: Fix

Re: [PATCH] libstdc++: Fix arithmetic bug in chrono::year_month::operator+

2020-08-27 Thread Jonathan Wakely via Gcc-patches
On 27/08/20 11:29 -0400, Patrick Palka via Libstdc++ wrote: This fixes the months-based addition for year_month when the year_month's month component is zero. Successfully tested on x86_64-pc-linux-gnu, on the 'date' library's calendar and (now) on libcxx's calendar tests. Does this look OK to

[PATCH] libstdc++: Fix arithmetic bug in chrono::year_month::operator+

2020-08-27 Thread Patrick Palka via Gcc-patches
This fixes the months-based addition for year_month when the year_month's month component is zero. Successfully tested on x86_64-pc-linux-gnu, on the 'date' library's calendar and (now) on libcxx's calendar tests. Does this look OK to commit? libstdc++-v3/ChangeLog: * include/std/chrono