commit 6174b387749cb82e0a4ada610c021fd5341a1f12
author Inseob Kim <inseob@google.com> Thu Mar 11 03:17:20 2021 +0000
committer Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> Thu Mar 11 03:17:20 2021 +0000
tree 7ea316b0c6df46b22304b9173c0f56e8402f4204
parent 88c0c383f9d0f6ad496c99fcb0cd90fe8972457c
parent 6824eaebd31801e6a57bc9e98eb3c6b3a8ba62c0

Change fallback log to verbose am: f2effd1063 am: 6824eaebd3

Original change: https://android-review.googlesource.com/c/platform/system/tools/sysprop/+/1624460

MUST ONLY BE SUBMITTED BY AUTOMERGER

Change-Id: I9d5db30d5a83fee3eb3e257760fe18faa9b3c542

CppGen.cpp

diff --git a/CppGen.cpp b/CppGen.cpp
index e91d03c..58efde2 100644
--- a/CppGen.cpp
+++ b/CppGen.cpp
@@ -225,7 +225,7 @@
     value = android::base::GetProperty(key, "");
 #endif
     if (value.empty() && legacy) {
-        ALOGD("prop %s doesn't exist; fallback to legacy prop %s", key, legacy);
+        ALOGV("prop %s doesn't exist; fallback to legacy prop %s", key, legacy);
         return GetProp<T>(legacy);
     }
     return TryParse<T>(value.c_str());

JavaGen.cpp

diff --git a/JavaGen.cpp b/JavaGen.cpp
index c677163..c2cdaeb 100644
--- a/JavaGen.cpp
+++ b/JavaGen.cpp
@@ -428,7 +428,7 @@
       writer.Write("if (\"\".equals(value)) {\n");
       writer.Indent();
       writer.Write(
-          "Log.d(\"%s\", \"prop %s doesn't exist; fallback to legacy prop "
+          "Log.v(\"%s\", \"prop %s doesn't exist; fallback to legacy prop "
           "%s\");\n",
           class_name.c_str(), prop.prop_name().c_str(),
           prop.legacy_prop_name().c_str());

tests/CppGenTest.cpp

diff --git a/tests/CppGenTest.cpp b/tests/CppGenTest.cpp
index e856900..8deb780 100644
--- a/tests/CppGenTest.cpp
+++ b/tests/CppGenTest.cpp
@@ -438,7 +438,7 @@
     value = android::base::GetProperty(key, "");
 #endif
     if (value.empty() && legacy) {
-        ALOGD("prop %s doesn't exist; fallback to legacy prop %s", key, legacy);
+        ALOGV("prop %s doesn't exist; fallback to legacy prop %s", key, legacy);
         return GetProp<T>(legacy);
     }
     return TryParse<T>(value.c_str());

tests/JavaGenTest.cpp

diff --git a/tests/JavaGenTest.cpp b/tests/JavaGenTest.cpp
index 69a9807..6ae0474 100644
--- a/tests/JavaGenTest.cpp
+++ b/tests/JavaGenTest.cpp
@@ -275,7 +275,7 @@
     public static Optional<String> test_string() {
         String value = SystemProperties.get("vendor.test.string");
         if ("".equals(value)) {
-            Log.d("TestProperties", "prop vendor.test.string doesn't exist; fallback to legacy prop vendor.old.string");
+            Log.v("TestProperties", "prop vendor.test.string doesn't exist; fallback to legacy prop vendor.old.string");
             value = SystemProperties.get("vendor.old.string");
         }
         return Optional.ofNullable(tryParseString(value));
@@ -504,7 +504,7 @@
     public static Optional<String> test_string() {
         String value = SystemProperties.get("vendor.test.string");
         if ("".equals(value)) {
-            Log.d("TestProperties", "prop vendor.test.string doesn't exist; fallback to legacy prop vendor.old.string");
+            Log.v("TestProperties", "prop vendor.test.string doesn't exist; fallback to legacy prop vendor.old.string");
             value = SystemProperties.get("vendor.old.string");
         }
         return Optional.ofNullable(tryParseString(value));