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

    https://github.com/apache/cassandra/pull/279#discussion_r225404070
  
    --- Diff: 
test/burn/org/apache/cassandra/utils/memory/LongBufferPoolTest.java ---
    @@ -230,6 +272,10 @@ else if (!recycleFromNeighbour())
                             }
                         }
     
    +                    if (currentTargetSize == 0) {
    --- End diff --
    
    According to the code style, you can skip braces here. We also follow 
follow Sun Style so braces go on new lines.
    Reference: 
http://cassandra.apache.org/doc/latest/development/code_style.html


---

---------------------------------------------------------------------
To unsubscribe, e-mail: pr-unsubscr...@cassandra.apache.org
For additional commands, e-mail: pr-h...@cassandra.apache.org

Reply via email to