Disallow ')' in encoded names

The JVM spec does not prohibit this character so this appears to be an
ASM bug, it worked fine in ASM 6.0 and crashes with 6.2 and 7.0.

This fixes a crash in `dotty-semanticdb/test:compile`.

Author: smarter

library/src/scala/tasty/util/Chars.scala

@@ -73,11 +73,13 @@ object Chars {
   }
 
   /** Can `c` appear in an encoded name ?
-   *  This is true for all characters, except those prohibited in JVM signatures
-   *  (https://docs.oracle.com/javase/specs/jvms/se8/html/jvms-4.html#jvms-4.7.9.1).
+   *  This is true for all characters, except:
+   *  - the characters prohibited in JVM signatures
+   *    (https://docs.oracle.com/javase/specs/jvms/se8/html/jvms-4.html#jvms-4.7.9.1).
+   *  - ')' because it leads to crashes during code generation when using ASM 7.0.
    */
   def isValidInEncodedName(c: Char): Boolean = (c: @switch) match {
-    case '.' | ';' | '[' | '/' | '<' | '>' | ':' => false
+    case '.' | ';' | '[' | '/' | '<' | '>' | ':' | ')' => false
     case _ => true
   }
 

tests/run/special-names.scala

@@ -5,6 +5,7 @@ class `/`
 class `<`
 class `>`
 class `:`
+class `)`
 
 object Test {
   def `.`(x: `.`) = x
@@ -14,6 +15,7 @@ object Test {
   def `<`(x: `<`) = x
   def `>`(x: `>`) = x
   def `:`(x: `:`) = x
+  def `)`(x: `)`) = x
 
   def main(args: Array[String]): Unit = {
     `.`(new `.`)
@@ -23,5 +25,6 @@ object Test {
     `<`(new `<`)
     `>`(new `>`)
     `:`(new `:`)
+    `)`(new `)`)
   }
 }