thejohnfreeman / autocheck

Header-only C++17 library for property-based testing.
ISC License
125 stars 17 forks source link

More msvc bugs #7

Closed graydon closed 9 years ago

graydon commented 9 years ago

MSVC chokes on parameter-pack expansion in a couple contexts, so I pretty much just shook the code around until it stopped complaining. As far as I can tell the Gens... related changes have no semantic difference, they just entertain the compiler in slightly different ways.

The SignedIntegral change here I think is more legit. Previous code was negating an unsigned type (size_t). Probably would have underflowed.

graydon commented 9 years ago

Updated with comment.