On Sat, Apr 28, 2012 at 5:41 PM, Marc Glisse <marc.gli...@inria.fr> wrote: > Hello, > > the attached follows the precisions on the definition of is_modulo in DR > 612. I believe this is what the values always should have been, so I didn't > make the change conditional to C++11.
Thanks for the patch. I think 'char' shouldn't be considered modulo even if -funsigned-char -- I will bring this to LWG attention. > > PR 22200 can remain open if people want to discuss the interaction with > -fwrapv, but false is the safe value. I believe the interaction with -fwrapv is an interesting one. > > I ran the testsuite, to make sure there wasn't a huge typo that would > prevent the file from compiling, and the only failure was the usual > 22_locale/time_get/get_date/wchar_t/4.cc. OK. > > 2012-04-29 Marc Glisse <marc.gli...@inria.fr> > > PR libstdc++/22200 > * include/std/limits (numeric_limits<>::is_modulo): false for > signed types. > > -- > Marc Glisse