diff --git a/tools/aconfig/aconfig/src/codegen/java.rs b/tools/aconfig/aconfig/src/codegen/java.rs index dbc4ab500b..1ac58c1b84 100644 --- a/tools/aconfig/aconfig/src/codegen/java.rs +++ b/tools/aconfig/aconfig/src/codegen/java.rs @@ -745,10 +745,8 @@ mod tests { boolean val; try { val = reader.getBooleanFlagValue(1); - if (val == disabledRw) { - Log.i(TAG, String.format(SUCCESS_LOG, "disabledRw")); - } else { - Log.i(TAG, String.format(MISMATCH_LOG, "disabledRw", val, disabledRw)); + if (val != disabledRw) { + Log.w(TAG, String.format(MISMATCH_LOG, "disabledRw", val, disabledRw)); } if (useNewStorageValueAndDiscardOld) { @@ -756,10 +754,8 @@ mod tests { } val = reader.getBooleanFlagValue(2); - if (val == disabledRwExported) { - Log.i(TAG, String.format(SUCCESS_LOG, "disabledRwExported")); - } else { - Log.i(TAG, String.format(MISMATCH_LOG, "disabledRwExported", val, disabledRwExported)); + if (val != disabledRwExported) { + Log.w(TAG, String.format(MISMATCH_LOG, "disabledRwExported", val, disabledRwExported)); } if (useNewStorageValueAndDiscardOld) { @@ -767,10 +763,8 @@ mod tests { } val = reader.getBooleanFlagValue(8); - if (val == enabledRw) { - Log.i(TAG, String.format(SUCCESS_LOG, "enabledRw")); - } else { - Log.i(TAG, String.format(MISMATCH_LOG, "enabledRw", val, enabledRw)); + if (val != enabledRw) { + Log.w(TAG, String.format(MISMATCH_LOG, "enabledRw", val, enabledRw)); } if (useNewStorageValueAndDiscardOld) { @@ -804,10 +798,8 @@ mod tests { boolean val; try { val = reader.getBooleanFlagValue(3); - if (val == disabledRwInOtherNamespace) { - Log.i(TAG, String.format(SUCCESS_LOG, "disabledRwInOtherNamespace")); - } else { - Log.i(TAG, String.format(MISMATCH_LOG, "disabledRwInOtherNamespace", val, disabledRwInOtherNamespace)); + if (val != disabledRwInOtherNamespace) { + Log.w(TAG, String.format(MISMATCH_LOG, "disabledRwInOtherNamespace", val, disabledRwInOtherNamespace)); } if (useNewStorageValueAndDiscardOld) { diff --git a/tools/aconfig/aconfig/templates/FeatureFlagsImpl.java.template b/tools/aconfig/aconfig/templates/FeatureFlagsImpl.java.template index 9970b1f300..bc01aa4bab 100644 --- a/tools/aconfig/aconfig/templates/FeatureFlagsImpl.java.template +++ b/tools/aconfig/aconfig/templates/FeatureFlagsImpl.java.template @@ -90,10 +90,8 @@ public final class FeatureFlagsImpl implements FeatureFlags \{ {{ -if flag.is_read_write }} val = reader.getBooleanFlagValue({flag.flag_offset}); - if (val == {flag.method_name}) \{ - Log.i(TAG, String.format(SUCCESS_LOG, "{flag.method_name}")); - } else \{ - Log.i(TAG, String.format(MISMATCH_LOG, "{flag.method_name}", val, {flag.method_name})); + if (val != {flag.method_name}) \{ + Log.w(TAG, String.format(MISMATCH_LOG, "{flag.method_name}", val, {flag.method_name})); } if (useNewStorageValueAndDiscardOld) \{ diff --git a/tools/aconfig/fake_device_config/src/android/util/Log.java b/tools/aconfig/fake_device_config/src/android/util/Log.java index 3e7fd0f386..79de68060e 100644 --- a/tools/aconfig/fake_device_config/src/android/util/Log.java +++ b/tools/aconfig/fake_device_config/src/android/util/Log.java @@ -5,6 +5,10 @@ public final class Log { return 0; } + public static int w(String tag, String msg) { + return 0; + } + public static int e(String tag, String msg) { return 0; }