spullara/netty — comment on commit ffea6a262b90dd613857516ca9b0ac8c3e9b662e
File: src/main/java/org/jboss/netty/handler/codec/replay/ReplayingDecoder.java line 571
By spullara:

No need, its all in the diff. The first line sets buf to cumulation, it then compares buf with null. If buf is null at the point then cumulation is null at the point and there is no reason to recheck it. That buf = cumulation line was unreachable. That code was probably left over from some previous refactoring.