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.
---