Github user bodewig commented on a diff in the pull request:
https://github.com/apache/commons-compress/pull/61#discussion_r161847721
--- Diff:
src/main/java/org/apache/commons/compress/archivers/cpio/CpioArchiveEntry.java
---
@@ -886,12 +886,7 @@ public boolean equals(final Object obj) {
}
final CpioArchiveEntry other = (CpioArchiveEntry) obj;
if (name == null) {
- if (other.name != null) {
- return false;
- }
- } else if (!name.equals(other.name)) {
- return false;
- }
- return true;
+ return other.name == null;
+ } else return name.equals(other.name);
--- End diff --
please add braces for the else block.
---
