From c7df93d22abb2197e75aaeed25c0057a149813b7 Mon Sep 17 00:00:00 2001
From: Andreas Dangel <adangel@users.sourceforge.net>
Date: Tue, 15 Sep 2015 08:17:11 +0200
Subject: [PATCH] More braces for code samples with if

---
 .../src/main/resources/rulesets/java/design.xml    | 14 ++++++++------
 1 file changed, 8 insertions(+), 6 deletions(-)

diff --git a/pmd-java/src/main/resources/rulesets/java/design.xml b/pmd-java/src/main/resources/rulesets/java/design.xml
index 449f2236fc..0725c6837b 100644
--- a/pmd-java/src/main/resources/rulesets/java/design.xml
+++ b/pmd-java/src/main/resources/rulesets/java/design.xml
@@ -737,13 +737,13 @@ have unusual conventions, i.e. Turkish.
     <![CDATA[
 class Foo {
  // BAD
- if (x.toLowerCase().equals("list"))...
+ if (x.toLowerCase().equals("list")) { }
  /*
  This will not match "LIST" when in Turkish locale
  The above could be
- if (x.toLowerCase(Locale.US).equals("list")) ...
+ if (x.toLowerCase(Locale.US).equals("list")) { }
  or simply
- if (x.equalsIgnoreCase("list")) ...
+ if (x.equalsIgnoreCase("list")) { }
  */
  // GOOD
  String z = a.toLowerCase(Locale.EN);
@@ -1794,13 +1794,15 @@ Use opposite operator instead of negating the whole expression with a logic comp
     <![CDATA[
 public boolean bar(int a, int b) {
 
-	if (!(a == b)) // use !=
+    if (!(a == b)) { // use !=
          return false;
+     }
 
-	if (!(a < b)) // use >=
+    if (!(a < b)) { // use >=
          return false;
+    }
 
-	return true;
+    return true;
 }
     ]]>
     </example>