added emptyBlockCheck config for BlockFormatCheck

Author: adireddy

checkstyle/checks/BlockFormatCheck.hx

@@ -7,6 +7,7 @@ import checkstyle.LintMessage.SeverityLevel;
 class BlockFormatCheck extends Check {
 
 	public var severity:String = "INFO";
+	public var emptyBlockCheck:Bool = true;
 
 	var firstLineRE = ~/\{[\/*]?\s*$/;
 	var lastLineRE = ~/^\s*\}[,;\/*]?/;
@@ -15,7 +16,7 @@ class BlockFormatCheck extends Check {
 		ExprUtils.walkFile(_checker.ast, function(e) {
 			switch(e.expr){
 				case EBlock([]) | EObjectDecl([]):
-					if (e.pos.max - e.pos.min > "{}".length)
+					if (emptyBlockCheck && e.pos.max - e.pos.min > "{}".length)
 						logPos("Empty block should be written as {}", e.pos, Reflect.field(SeverityLevel, severity));
 				case EBlock(_) | EObjectDecl(_):
 					var lmin = _checker.getLinePos(e.pos.min).line;

haxelib.json

@@ -10,8 +10,8 @@
 	],
 	"description": "Automated code analysis ideal for projects that want to enforce a coding standard.",
 	"contributors": ["adireddy"],
-	"releasenote": "added allowEmptyReturn config for ReturnCheck",
-	"version": "1.0.6",
+	"releasenote": "added emptyBlockCheck config for BlockFormatCheck",
+	"version": "1.0.7",
 	"url": "https://github.com/adireddy/haxe-checkstyle",
 	"dependencies": {
 

package.json

@@ -1,6 +1,6 @@
 {
 	"name": "checkstyle",
-	"version": "1.0.6",
+	"version": "1.0.7",
 	"description": "Automated code analysis ideal for projects that want to enforce a coding standard.",
 	"repository": {
 		"type": "git",

resources/config.json

@@ -15,7 +15,8 @@
 		{
 			"type": "BlockFormat",
 			"props": {
-				"severity": "ERROR"
+				"severity": "ERROR",
+				"emptyBlockCheck": false
 			}
 		},
 		{

run.n

@@ -23,6 +23,13 @@ class BlockFormatCheckTest extends CheckTestCase {
 		var msg = checkMessage(BlockFormatTests.TEST3, new BlockFormatCheck());
 		assertEquals(msg, 'Last line of multiline block should contain only } and maybe , or ;');
 	}
+
+	public function testDisabledBlockFormat() {
+		var test = new BlockFormatCheck();
+		test.emptyBlockCheck = false;
+		var msg = checkMessage(BlockFormatTests.TEST4, test);
+		assertEquals(msg, '');
+	}
 }
 
 class BlockFormatTests {
@@ -50,4 +57,11 @@ class BlockFormatTests {
 		public function new() {
 			var a:Int; }
 	}";
+
+	public static inline var TEST4:String = "
+	class Test {
+		public function new() {
+		
+		}
+	}";
 }