add error for destroy this in constructor, fixes #754

Author: Frotty

de.peeeq.wurstscript/src/main/java/de/peeeq/wurstscript/validation/WurstValidator.java

@@ -1817,6 +1817,13 @@ private void visit(ModuleDef moduleDef) {
     }
 
     private void visit(ExprDestroy stmtDestroy) {
+        if (stmtDestroy.getDestroyedObj() instanceof ExprThis) {
+            if (isInConstructor(stmtDestroy)) {
+                stmtDestroy.addError("Cannot destroy 'this' in constructor");
+                return;
+            }
+        }
+
         WurstType typ = stmtDestroy.getDestroyedObj().attrTyp();
         if (typ instanceof WurstTypeModule) {
 

de.peeeq.wurstscript/src/test/java/tests/wurstscript/tests/BugTests.java

@@ -1442,5 +1442,35 @@ public void minusRewrite() {
         );
     }
 
+    @Test
+    public void duplicateNameInClassHierachy() {
+        testAssertErrorsLines(false, "Variable x in class B hides variable x from superclass A",
+            "package test",
+            "native testSuccess()",
+            "class A",
+            "    int x",
+            "class B extends A",
+            "    int x",
+            "init",
+            "	let b = new B()",
+            "	if b != null",
+            "		testSuccess()",
+            "endpackage");
+    }
+
+    @Test
+    public void callingDestroyThisInConstructor() {
+        testAssertErrorsLines(false, "Cannot destroy 'this' in constructor",
+            "package test",
+            "native testSuccess()",
+            "class A",
+            "    construct()",
+            "       destroy this",
+            "init",
+            "	let b = new A()",
+            "	if b != null",
+            "		testSuccess()",
+            "endpackage");
+    }
 
 }