GROOVY-11297: Fail to identify duplicate constructor declaration (por…

…t to 2_5_X)

src/main/java/org/codehaus/groovy/classgen/Verifier.java

@@ -255,6 +255,7 @@ public void visitClass(final ClassNode node) {
         node.getObjectInitializerStatements().clear();
         node.visitContents(this);
         checkForDuplicateMethods(node);
+        checkForDuplicateConstructors(node);
         addCovariantMethods(node);
 
         checkFinalVariables(node);
@@ -307,6 +308,18 @@ private static void checkForDuplicateMethods(ClassNode cn) {
         }
     }
 
+    private static void checkForDuplicateConstructors(final ClassNode cn) {
+        Set<String> descriptors = new HashSet<>();
+        for (ConstructorNode cons : cn.getDeclaredConstructors()) {
+            String mySig = methodDescriptorWithoutReturnType(cons);
+            if (descriptors.contains(mySig)) {
+                throw new RuntimeParserException("The constructor " + cons.getText() +
+                    " duplicates another constructor of the same signature", sourceOf(cons));
+            }
+            descriptors.add(mySig);
+        }
+    }
+
     private static String scriptBodySignatureWithoutReturnType(ClassNode cn) {
         for (MethodNode mn : cn.getMethods()) {
             if (mn.isScriptBody()) return methodDescriptorWithoutReturnType(mn);

src/test/gls/classes/methods/RepetitiveMethodTest.groovy

@@ -48,4 +48,14 @@ class RepetitiveMethodTest extends CompilableTestSupport {
             }
         '''
     }
+
+    void testRepetitiveConstructor() {
+        def message = shouldNotCompile('''
+            class A {
+                A(int x) {}
+                A(int y) {}
+            }
+        ''')
+        assert message.contains('duplicates another constructor of the same signature')
+    }
 }