edouarda/brigand

Minor Patch for logical and/or

Alexhuszagh opened this issue · 1 comments

Alexhuszagh commented

On MSVC, with /W3 or higher (for warning C4800) issues a warning for my pull request to support variadic and/or. The fix is quite simple: Change bool(sizeof...(Ts) / 9) to (sizeof...(Ts) / 9) != 0 in all locations. This really have minimal repercussions otherwise, and I can submit a PR if these changes are desirable.

edouarda commented

Thank you, your PR is welcomed!

  • 👍1