This fixes #146: Browser js console warning: unreachable code after r…

…eturn statement
dragome · Mar 7, 2017 · e3fd2f1 · e3fd2f1
1 parent 42c3ea9
commit e3fd2f1
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/...js-compiler/src/main/java/com/dragome/compiler/generators/DragomeJavaScriptGenerator.java b/...js-compiler/src/main/java/com/dragome/compiler/generators/DragomeJavaScriptGenerator.java
@@ -307,7 +307,8 @@ else if (classCompilerType != null)
 		Signature signature= Project.getSingleton().getSignature(methodBinding.toString()).relative();
 		String signatureReplaced= normalizeExpression(signature);
 
-		if (typeDecl.getClassName().equals("java.lang.String") && method.isInstanceConstructor())
+		boolean isString= typeDecl.getClassName().equals("java.lang.String");
+		if (isString && method.isInstanceConstructor())
 		{
 
 			Block body= method.getBody();
@@ -381,7 +382,7 @@ else if (classCompilerType != null)
 		else if (isNative)
 			println("return dragomeJs.resolveNativeMethod(this, \"" + signatureReplaced + "\").apply(this, arguments);");
 
-		if (method.isInstanceConstructor())
+		if (method.isInstanceConstructor() && !isString)
 			print("return this;\n");
 
 		print(closingString);