From 721b9cd4da7949bfa895104eec5896041b1bbbe0 Mon Sep 17 00:00:00 2001
From: Luke De Feo <lukedefeo@fb.com>
Date: Tue, 13 Sep 2022 11:05:42 -0700
Subject: [PATCH] Fix logging

Summary: typing uidebugger into ADB will hit both the log tag and any thing else such as package name. Turned Enum warning down to verbose reduce load on main thread

Reviewed By: lblasa

Differential Revision: D39466932

fbshipit-source-id: 0106a042a92755cf644c0e66683e7e5993e924e1
---
 .../flipper/plugins/uidebugger/UIDebuggerFlipperPlugin.kt       | 2 +-
 .../facebook/flipper/plugins/uidebugger/common/EnumMapping.kt   | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/android/src/main/java/com/facebook/flipper/plugins/uidebugger/UIDebuggerFlipperPlugin.kt b/android/src/main/java/com/facebook/flipper/plugins/uidebugger/UIDebuggerFlipperPlugin.kt
index b77d017a5..886b95113 100644
--- a/android/src/main/java/com/facebook/flipper/plugins/uidebugger/UIDebuggerFlipperPlugin.kt
+++ b/android/src/main/java/com/facebook/flipper/plugins/uidebugger/UIDebuggerFlipperPlugin.kt
@@ -19,7 +19,7 @@ import com.facebook.flipper.plugins.uidebugger.scheduler.Scheduler
 import kotlinx.coroutines.*
 import kotlinx.serialization.json.Json
 
-const val LogTag = "FlipperUIDebugger"
+const val LogTag = "uidebugger"
 
 class UIDebuggerFlipperPlugin(
     val application: Application,
diff --git a/android/src/main/java/com/facebook/flipper/plugins/uidebugger/common/EnumMapping.kt b/android/src/main/java/com/facebook/flipper/plugins/uidebugger/common/EnumMapping.kt
index 64f2eb657..3c286d98b 100644
--- a/android/src/main/java/com/facebook/flipper/plugins/uidebugger/common/EnumMapping.kt
+++ b/android/src/main/java/com/facebook/flipper/plugins/uidebugger/common/EnumMapping.kt
@@ -18,7 +18,7 @@ open class EnumMapping<T>(val mapping: Map<String, T>) {
     if (entry != null) {
       return entry.key
     } else {
-      Log.d(
+      Log.v(
           LogTag,
           "Could not convert enum value ${enumValue.toString()} to string, known values ${mapping.entries}")
       return NoMapping