Github user bodewig commented on a diff in the pull request:
https://github.com/apache/commons-compress/pull/61#discussion_r161847670
--- Diff:
src/main/java/org/apache/commons/compress/archivers/ar/ArArchiveEntry.java ---
@@ -179,12 +179,7 @@ public boolean equals(final Object obj) {
}
final ArArchiveEntry other = (ArArchiveEntry) 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.
---
