[GitHub] commons-lang pull request #334: Code refactoring and cleaning

[aaabramov]

Github user aaabramov commented on a diff in the pull request:

    https://github.com/apache/commons-lang/pull/334#discussion_r197231903
  
    --- Diff: 
src/main/java/org/apache/commons/lang3/builder/CompareToBuilder.java ---
    @@ -600,10 +600,10 @@ public CompareToBuilder append(final boolean lhs, 
final boolean rhs) {
             if (lhs == rhs) {
                 return this;
             }
    -        if (!lhs) {
    -            comparison = -1;
    -        } else {
    +        if (lhs) {
                 comparison = +1;
    --- End diff --
    
    Why do we need `+` here?


---