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.

Reply via email to