Fix review

Author: Changqing-JING

src/compiler.ts

@@ -7194,8 +7194,8 @@ export class Compiler extends DiagnosticEmitter {
       let existingLocal = flow.getScopedLocal(declaration.name.text);
       if (existingLocal) {
         this.error(
-          DiagnosticCode.Duplicate_identifier_0,
-          declaration.name.range, declaration.name.text
+          DiagnosticCode.Duplicate_function_implementation,
+          declaration.name.range
         );
         return this.module.unreachable();
       }

tests/compiler/duplicate-function-in-scope.json

@@ -2,6 +2,6 @@
   "asc_flags": [],
   "stderr": [
     "EOF",
-    "TS2300: Duplicate identifier 'inner'."
+    "TS2393: Duplicate function implementation."
   ]
 }

tests/compiler/duplicate-function-in-scope.ts

@@ -2,14 +2,8 @@
 // produce a diagnostic instead of crashing the compiler.
 
 export function test(): void {
-  function inner(): i32 {
-    let x: i32 = 0;
-    return x;
-  }
-  function inner(): i32 {
-    let x: i32 = 0;
-    return x + 1;
-  }
+  function inner(): void {}
+  function inner(): void {}
   inner();
 }