commit 10c5a93eb7416e31706d3b8ef93d0a4816afcf56
author Amit Hilbuch <amithi@webrtc.org> Mon Mar 18 13:09:18 2019 -0700
committer Commit Bot <commit-bot@chromium.org> Tue Mar 19 17:13:47 2019 +0000
tree 2dcaa9a61851de6e1c4b7956d6ca24e4183f81df
parent 1f850a6dc76a2403e1d893025cd7bd87410a5e7c

Relanding #128202 Adding support for enum class in RTC_CHECK and RTC_LOG.

This is a reland of: https://webrtc-review.googlesource.com/c/src/+/128202
Logging unit test has been fixed to unregister the log listener.

Bug: webrtc:10418
Change-Id: Idac9cb71807e16a585ccb7e3e4271aa45d69f547
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/128541
Reviewed-by: Steve Anton <steveanton@webrtc.org>
Reviewed-by: Karl Wiberg <kwiberg@webrtc.org>
Commit-Queue: Amit Hilbuch <amithi@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#27192}

rtc_base/logging.h

diff --git a/rtc_base/logging.h b/rtc_base/logging.h
index 5cd4f72..3f48eeb 100644
--- a/rtc_base/logging.h
+++ b/rtc_base/logging.h
@@ -248,6 +248,16 @@
   return {x};
 }
 
+// The enum class types are not implicitly convertible to arithmetic types.
+template <
+    typename T,
+    typename std::enable_if<std::is_enum<T>::value &&
+                            !std::is_arithmetic<T>::value>::type* = nullptr>
+inline decltype(MakeVal(std::declval<typename std::underlying_type<T>::type>()))
+MakeVal(T x) {
+  return {static_cast<typename std::underlying_type<T>::type>(x)};
+}
+
 #ifdef WEBRTC_ANDROID
 inline Val<LogArgType::kLogMetadataTag, LogMetadataTag> MakeVal(
     const LogMetadataTag& x) {
@@ -285,18 +295,18 @@
 template <>
 class LogStreamer<> final {
  public:
-  template <
-      typename U,
-      typename std::enable_if<std::is_arithmetic<U>::value>::type* = nullptr>
+  template <typename U,
+            typename std::enable_if<std::is_arithmetic<U>::value ||
+                                    std::is_enum<U>::value>::type* = nullptr>
   RTC_FORCE_INLINE LogStreamer<decltype(MakeVal(std::declval<U>()))> operator<<(
       U arg) const {
     return LogStreamer<decltype(MakeVal(std::declval<U>()))>(MakeVal(arg),
                                                              this);
   }
 
-  template <
-      typename U,
-      typename std::enable_if<!std::is_arithmetic<U>::value>::type* = nullptr>
+  template <typename U,
+            typename std::enable_if<!std::is_arithmetic<U>::value &&
+                                    !std::is_enum<U>::value>::type* = nullptr>
   RTC_FORCE_INLINE LogStreamer<decltype(MakeVal(std::declval<U>()))> operator<<(
       const U& arg) const {
     return LogStreamer<decltype(MakeVal(std::declval<U>()))>(MakeVal(arg),
@@ -318,18 +328,18 @@
   RTC_FORCE_INLINE LogStreamer(T arg, const LogStreamer<Ts...>* prior)
       : arg_(arg), prior_(prior) {}
 
-  template <
-      typename U,
-      typename std::enable_if<std::is_arithmetic<U>::value>::type* = nullptr>
+  template <typename U,
+            typename std::enable_if<std::is_arithmetic<U>::value ||
+                                    std::is_enum<U>::value>::type* = nullptr>
   RTC_FORCE_INLINE LogStreamer<decltype(MakeVal(std::declval<U>())), T, Ts...>
   operator<<(U arg) const {
     return LogStreamer<decltype(MakeVal(std::declval<U>())), T, Ts...>(
         MakeVal(arg), this);
   }
 
-  template <
-      typename U,
-      typename std::enable_if<!std::is_arithmetic<U>::value>::type* = nullptr>
+  template <typename U,
+            typename std::enable_if<!std::is_arithmetic<U>::value &&
+                                    !std::is_enum<U>::value>::type* = nullptr>
   RTC_FORCE_INLINE LogStreamer<decltype(MakeVal(std::declval<U>())), T, Ts...>
   operator<<(const U& arg) const {
     return LogStreamer<decltype(MakeVal(std::declval<U>())), T, Ts...>(