Disallow ')' in encoded names

The JVM spec does not prohibit this character but it leads to codegen
crashes when using ASM 7.0, this has since been fixed in ASM master:
https://gitlab.ow2.org/asm/asm/issues/317868

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

Author: smarter

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

@@ -73,11 +73,15 @@ 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
+   *    (https://gitlab.ow2.org/asm/asm/issues/317868).
+   *    TODO: Remove this case when we upgrade ASM.
    */
   def isValidInEncodedName(c: Char): Boolean = (c: @switch) match {
-    case '.' | ';' | '[' | '/' | '<' | '>' | ':' => false
+    case '.' | ';' | '[' | '/' | '<' | '>' | ':' | ')' => false
     case _ => true
   }
 

tests/run/special-names.scala

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