checkstyle warning cleanup

Author: AlexHaxe

checkstyle/checks/CyclomaticComplexityCheck.hx

@@ -42,16 +42,16 @@ class CyclomaticComplexityCheck extends Check {
 
 	function calculateComplexity(method:Target) {
 		var complexity:Int = 1 + evaluateExpr(method.expr);
-		
+
 		var risk:Null<Threshold> = thresholds.filter(function(t:Threshold):Bool {
 			return complexity >= t.complexity;
 		}).pop();
-		
+
 		if (risk != null) {
 			notify(method, complexity, risk);
 		}
 	}
-	
+
 	// This would not pass the cyclomatic complexity test.
 	function evaluateExpr(e:Expr):Int {
 		if (e == null || e.expr == null) {
@@ -65,9 +65,9 @@ class CyclomaticComplexityCheck extends Check {
 				default : 0;
 			};
 			case ExprDef.EParenthesis(e) : evaluateExpr(e);
-			case ExprDef.EObjectDecl(fields) : 
-				fields.map(function(f):Expr { 
-					return f.expr; 
+			case ExprDef.EObjectDecl(fields) :
+				fields.map(function(f):Expr {
+					return f.expr;
 				}).fold(function(e:Expr, total:Int):Int {
 					return total + evaluateExpr(e);
 				}, 0);

checkstyle/checks/NestedForDepthCheck.hx

@@ -8,8 +8,14 @@ import haxe.macro.Expr;
 @desc("Max number of nested for blocks (default 1)")
 class NestedForDepthCheck extends Check {
 
-	public var severity:String = "ERROR";
-	public var max:Int = 1;
+	public var severity:String;
+	public var max:Int;
+
+	public function new() {
+		super();
+		severity = "ERROR";
+		max = 1;
+	}
 
 	override function _actualRun() {
 		for (td in _checker.ast.decls) {
@@ -61,6 +67,7 @@ class NestedForDepthCheck extends Check {
 	}
 
 	function _warnNestedForDepth(depth:Int, pos:Position) {
-		logPos('Nested for depth is $depth (max allowed is ${max})', pos, Reflect.field(SeverityLevel, severity));
+		logPos('Nested for depth is $depth (max allowed is ${max})',
+			pos, Reflect.field(SeverityLevel, severity));
 	}
-}
+}

checkstyle/checks/NestedIfDepthCheck.hx

@@ -8,8 +8,14 @@ import haxe.macro.Expr;
 @desc("Max number of nested if-else blocks (default 1)")
 class NestedIfDepthCheck extends Check {
 
-	public var severity:String = "ERROR";
-	public var max:Int = 1;
+	public var severity:String;
+	public var max:Int;
+
+	public function new() {
+		super();
+		severity = "ERROR";
+		max = 1;
+	}
 
 	override function _actualRun() {
 		for (td in _checker.ast.decls) {
@@ -60,6 +66,7 @@ class NestedIfDepthCheck extends Check {
 	}
 
 	function _warnNestedIfDepth(depth:Int, pos:Position) {
-		logPos('Nested if-else depth is $depth (max allowed is ${max})', pos, Reflect.field(SeverityLevel, severity));
+		logPos('Nested if-else depth is $depth (max allowed is ${max})',
+			pos, Reflect.field(SeverityLevel, severity));
 	}
-}
+}

checkstyle/checks/NestedTryDepthCheck.hx

@@ -8,8 +8,14 @@ import haxe.macro.Expr;
 @desc("Max number of nested try blocks (default 1)")
 class NestedTryDepthCheck extends Check {
 
-	public var severity:String = "ERROR";
-	public var max:Int = 1;
+	public var severity:String;
+	public var max:Int;
+
+	public function new() {
+		super();
+		severity = "ERROR";
+		max = 1;
+	}
 
 	override function _actualRun() {
 		for (td in _checker.ast.decls) {
@@ -66,6 +72,7 @@ class NestedTryDepthCheck extends Check {
 	}
 
 	function _warnNestedTryDepth(depth:Int, pos:Position) {
-		logPos('Nested try depth is $depth (max allowed is ${max})', pos, Reflect.field(SeverityLevel, severity));
+		logPos('Nested try depth is $depth (max allowed is ${max})',
+			pos, Reflect.field(SeverityLevel, severity));
 	}
-}
+}