commit 4f124f593d0289363a98a39d601fd1009de9ab2f
author Søren Gjesse <sgjesse@google.com> Tue Jul 22 13:00:48 2025 +0200
committer Søren Gjesse <sgjesse@google.com> Mon Jul 28 01:47:16 2025 -0700
tree bb54b9ea85d2cf13f9bcf8afe653d3f5ccd8e2c2
parent 91ef690acf2e3851f0626c945f9d48e7977e410f

Reproduce incorrect R8 warning

Bug: b/433451526
Change-Id: Ica918320ec1c9767924c4c15c0983d0fbdf097d2

src/test/java/com/android/tools/r8/shaking/ProguardConfigurationParserTest.java

diff --git a/src/test/java/com/android/tools/r8/shaking/ProguardConfigurationParserTest.java b/src/test/java/com/android/tools/r8/shaking/ProguardConfigurationParserTest.java
index 92097d3..fbcdad8 100644
--- a/src/test/java/com/android/tools/r8/shaking/ProguardConfigurationParserTest.java
+++ b/src/test/java/com/android/tools/r8/shaking/ProguardConfigurationParserTest.java
@@ -6,11 +6,8 @@
 import static com.android.tools.r8.DiagnosticsChecker.checkDiagnostics;
 import static com.android.tools.r8.shaking.ProguardConfigurationSourceStrings.createConfigurationForTesting;
 import static com.android.tools.r8.utils.BooleanUtils.intValue;
-import static org.hamcrest.MatcherAssert.assertThat;
-import static org.hamcrest.core.StringContains.containsString;
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertFalse;
-import static org.junit.Assert.assertNotEquals;
 import static org.junit.Assert.assertNotNull;
 import static org.junit.Assert.assertNull;
 import static org.junit.Assert.assertSame;
@@ -21,7 +18,6 @@
 import com.android.tools.r8.TestParameters;
 import com.android.tools.r8.TestParametersCollection;
 import com.android.tools.r8.ToolHelper;
-import com.android.tools.r8.ToolHelper.ProcessResult;
 import com.android.tools.r8.errors.dontwarn.DontWarnConfiguration;
 import com.android.tools.r8.graph.ClassAccessFlags;
 import com.android.tools.r8.graph.DexItemFactory;
@@ -42,7 +38,6 @@
 import com.android.tools.r8.utils.ListUtils;
 import com.android.tools.r8.utils.Reporter;
 import com.android.tools.r8.utils.StringUtils;
-import com.android.tools.r8.utils.codeinspector.CodeInspector;
 import com.google.common.collect.ImmutableList;
 import com.google.common.collect.ImmutableMap;
 import java.io.File;
@@ -2349,12 +2344,9 @@
 
   @Test
   public void parse_regress110021323() throws Exception {
-    Path proguardConfig = writeTextToTempFile(
-      "-keepclassmembernames class A {",
-      "  <public methods>;",
-      "  <public fields>;",
-      "}"
-    );
+    Path proguardConfig =
+        writeTextToTempFile(
+            "-keepclassmembernames class A {", "  <public methods>;", "  <public fields>;", "}");
     ProguardConfigurationParser parser =
         new ProguardConfigurationParser(new DexItemFactory(), reporter);
     parser.parse(proguardConfig);
@@ -2365,6 +2357,21 @@
     checkDiagnostics(handler.warnings, 3, proguardConfig, 3, 11, "The field name \"fields>\" is");
   }
 
+  @Test
+  public void parse_regress433451526() throws Exception {
+    Path proguardConfig =
+        writeTextToTempFile(
+            "-if @kotlinx.serialization.Serializable class **",
+            "-keepclassmembers class <1> {",
+            "  static <1>$* Companion;",
+            "}");
+    ProguardConfigurationParser parser =
+        new ProguardConfigurationParser(new DexItemFactory(), reporter);
+    parser.parse(proguardConfig);
+    assertEquals(1, handler.warnings.size());
+    checkDiagnostics(handler.warnings, 0, proguardConfig, 3, 10, "The type \"<1>$*\" is");
+  }
+
   private void checkRulesSourceSnippet(List<String> sourceRules) {
     checkRulesSourceSnippet(sourceRules, sourceRules, false);
   }