On 11/06/16 11:46 -0700, Tim Shen wrote:
On Sat, Jun 11, 2016 at 5:01 AM, Jonathan Wakely wrote:
N.B. The "typename" and "::type" are redundant here, because it names
the same type as the integral_constant itself, and you could
use __bool_constant<__collate> instead:
return _M_transform_impl(_M_translate(__ch),
__bool_constant<__collate>());
OK for trunk without the redundant typename ...::type, your choice
whether to use __bool_constant or not.
Thanks! I was looking at std::bool_constant but that's in C++17.
__bool_constant is even better. :)
Will this fix apply cleanly to the branches too?
For gcc6 yes; for gcc5 there needs more work. I guess it's OK for
backporting to gcc6?
Yes, OK for trunk and gcc-6-branch, thanks.