Fix LOG_EVERY_N with clang -Wunused-local-typedef
authorJim Ray <jimray@google.com>
Mon, 31 Jul 2017 05:30:33 +0000 (22:30 -0700)
committerJim Ray <jimray@google.com>
Sun, 6 Aug 2017 07:17:50 +0000 (00:17 -0700)
commit8b3023f7e4ca46e0ecf5f660dd7340c79139bc34
treea3e7397740b904b03be9a166c43288fc340d801a
parentb3695ee3256d371219a00e39cdd7e5e33a4a0214
Fix LOG_EVERY_N with clang -Wunused-local-typedef

Glog uses a pre-C++11 compile time assert to verify the validity of
the severity parameter for LOG_EVERY_N. Unfortunately, some compilers
will complain about the usage of LOG_EVERY_N with
"-Wunused-local-typedef" due to the way the compile time assert is
constructed. This makes it impossible to use LOG_EVERY_N with this
warning treated as an error.

The fix simply removes the assert entirely. This is safe to do since
you can't put anything invalid into the severity parameters without
generating a compile error elsewhere. This has been safe to do ever
since the GLOG_ prefixes were added as part of 6febec361e.

Fixes #223
src/glog/logging.h.in
src/windows/glog/logging.h