Update RelocatorRemapper class pattern to cover more Java method descriptors (#1731)

* Test more cases like `void method(boolean arg1, org.package.ClassA arg2)` in `relocateSignaturePatterns`

* Update `classPattern`

* Update changelog

Author: Goooler

docs/changes/README.md

@@ -25,6 +25,7 @@
 - Move injecting `Main-Class` manifest attr logic from `doFirst` into `copy`. ([#1724](https://github.com/GradleUp/shadow/pull/1724))
 - Deprecate `InheritManifest`. ([#1722](https://github.com/GradleUp/shadow/pull/1722))
 - Use default `JavaExec` error message when main class is not set. ([#1725](https://github.com/GradleUp/shadow/pull/1725))
+- Update `RelocatorRemapper` class pattern to cover more Java method descriptors. ([#1731](https://github.com/GradleUp/shadow/pull/1731))
 
 ### Fixed
 

src/main/kotlin/com/github/jengelman/gradle/plugins/shadow/internal/RelocatorRemapper.kt

@@ -65,6 +65,9 @@ internal class RelocatorRemapper(
   }
 
   private companion object {
-    val classPattern: Pattern = Pattern.compile("([\\[()]*)?L([^;]+);?")
+    /**
+     * https://docs.oracle.com/javase/specs/jvms/se8/html/jvms-4.html
+     */
+    val classPattern: Pattern = Pattern.compile("([\\[()BCDFIJSZ]*)?L([^;]+);?")
   }
 }

src/test/kotlin/com/github/jengelman/gradle/plugins/shadow/relocation/RelocatorRemapperTest.kt

@@ -20,6 +20,13 @@ class RelocatorRemapperTest {
   }
 
   private companion object {
+    val primitiveTypes = setOf('B', 'C', 'D', 'F', 'I', 'J', 'S', 'Z')
+
+    val primitiveTypeArguments = primitiveTypes.map {
+      // Methods like `void method(boolean arg1, org.package.ClassA arg2)`
+      Arguments.of("(${it}Lorg/package/ClassA;)V", "(${it}Lshadow/org/package/ClassA;)V")
+    }
+
     @JvmStatic
     fun classSignatureStringConstants() = listOf(
       // Normal class.
@@ -36,6 +43,6 @@ class RelocatorRemapperTest {
       Arguments.of("(Lorg/package/ClassA;Lorg/package/ClassB;)", "(Lshadow/org/package/ClassA;Lshadow/org/package/ClassB;)"),
       // Method return types.
       Arguments.of("()Lorg/package/ClassA;Lorg/package/ClassB;", "()Lshadow/org/package/ClassA;Lshadow/org/package/ClassB;"),
-    )
+    ) + primitiveTypeArguments
   }
 }