refactor: Curly brackets around if/else one-line body

broadinstitute · Sep 20, 2017 · 11a9c1f · 11a9c1f
1 parent 31d042e
commit 11a9c1f
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/src/main/java/picard/sam/HitsForInsert.java b/src/main/java/picard/sam/HitsForInsert.java
@@ -211,8 +211,12 @@ record -> record.getIntegerAttribute(SAMTag.HI.name()),
             final Integer hiTagValueOfSecond = secondOfPair.get(i).getIntegerAttribute(SAMTag.HI.name());
 
             if (nonNull(hiTagValueOfFirst) && nonNull(hiTagValueOfSecond)) {
-                if (hiTagValueOfFirst < hiTagValueOfSecond) secondOfPair.add(i, null);
-                else if (hiTagValueOfSecond < hiTagValueOfFirst) firstOfPairOrFragment.add(i, null);
+                if (hiTagValueOfFirst < hiTagValueOfSecond) {
+                    secondOfPair.add(i, null);
+                }
+                else if (hiTagValueOfFirst > hiTagValueOfSecond) {
+                    firstOfPairOrFragment.add(i, null);
+                }
                 // else the are correlated
             }
 
@@ -283,7 +287,7 @@ void coordinateByMate() {
     /**
      * @return true if records belong to the same pairwise alignment
      */
-    private static boolean arePair(SAMRecord first, SAMRecord second) {
+    private static boolean arePair(final SAMRecord first, final SAMRecord second) {
         return nonNull(first) && nonNull(second)
                 && first.getMateAlignmentStart() == second.getAlignmentStart()
                 && first.getAlignmentStart() == second.getMateAlignmentStart()