diff --git a/tika-core/src/main/java/org/apache/tika/metadata/Metadata.java b/tika-core/src/main/java/org/apache/tika/metadata/Metadata.java index 9b8e3b86cd..5f6b6ec294 100644 --- a/tika-core/src/main/java/org/apache/tika/metadata/Metadata.java +++ b/tika-core/src/main/java/org/apache/tika/metadata/Metadata.java @@ -665,16 +665,11 @@ private int getMetadataEntryHashCode(Entry e) { public boolean equals(Object o) { - if (o == null) { + if (!(o instanceof Metadata)) { return false; } - Metadata other = null; - try { - other = (Metadata) o; - } catch (ClassCastException cce) { - return false; - } + Metadata other = (Metadata) o; if (other.size() != size()) { return false; diff --git a/tika-langdetect/tika-langdetect-tika/src/main/java/org/apache/tika/langdetect/tika/LanguageProfilerBuilder.java b/tika-langdetect/tika-langdetect-tika/src/main/java/org/apache/tika/langdetect/tika/LanguageProfilerBuilder.java index c53c3f6236..b78ad70a5e 100644 --- a/tika-langdetect/tika-langdetect-tika/src/main/java/org/apache/tika/langdetect/tika/LanguageProfilerBuilder.java +++ b/tika-langdetect/tika-langdetect-tika/src/main/java/org/apache/tika/langdetect/tika/LanguageProfilerBuilder.java @@ -673,14 +673,11 @@ public int hashCode() { // Inherited JavaDoc public boolean equals(Object obj) { - - NGramEntry ngram = null; - try { - ngram = (NGramEntry) obj; - return ngram.seq.equals(seq); - } catch (Exception e) { + if (!(obj instanceof NGramEntry)) { return false; } + NGramEntry ngram = (NGramEntry) obj; + return ngram.seq.equals(seq); } } diff --git a/tika-parsers/tika-parsers-ml/tika-parser-nlp-module/src/main/java/org/apache/tika/parser/journal/TEIDOMParser.java b/tika-parsers/tika-parsers-ml/tika-parser-nlp-module/src/main/java/org/apache/tika/parser/journal/TEIDOMParser.java index 70de06c8f6..2ea8a0a241 100644 --- a/tika-parsers/tika-parsers-ml/tika-parser-nlp-module/src/main/java/org/apache/tika/parser/journal/TEIDOMParser.java +++ b/tika-parsers/tika-parsers-ml/tika-parser-nlp-module/src/main/java/org/apache/tika/parser/journal/TEIDOMParser.java @@ -561,6 +561,9 @@ public void setAddress(Address address) { */ @Override public boolean equals(Object obj) { + if (!(obj instanceof Affiliation)) { + return false; + } Affiliation otherA = (Affiliation) obj; return this.getAddress().equals(otherA.getAddress()) && this.getOrgName().equals(otherA.getOrgName()); @@ -633,6 +636,10 @@ public int hashCode() { */ @Override public boolean equals(Object obj) { + if (!(obj instanceof OrgName)) { + return false; + } + OrgName otherA = (OrgName) obj; if (otherA.getTypeNames() != null) { @@ -697,6 +704,9 @@ public void setType(String type) { */ @Override public boolean equals(Object obj) { + if (!(obj instanceof OrgTypeName)) { + return false; + } OrgTypeName otherOrgName = (OrgTypeName) obj; return this.type.equals(otherOrgName.getType()) && this.name.equals(otherOrgName.getName()); @@ -785,6 +795,9 @@ public void setCountry(Country country) { */ @Override public boolean equals(Object obj) { + if (!(obj instanceof Address)) { + return false; + } Address otherA = (Address) obj; if (this.settlment == null) { return otherA.getSettlment() == null; @@ -870,6 +883,9 @@ public void setContent(String content) { */ @Override public boolean equals(Object obj) { + if (!(obj instanceof Country)) { + return false; + } Country otherC = (Country) obj; if (this.key == null) { diff --git a/tika-parsers/tika-parsers-standard/tika-parsers-standard-modules/tika-parser-microsoft-module/src/main/java/org/apache/tika/parser/microsoft/onenote/fsshttpb/streamobj/basic/CellID.java b/tika-parsers/tika-parsers-standard/tika-parsers-standard-modules/tika-parser-microsoft-module/src/main/java/org/apache/tika/parser/microsoft/onenote/fsshttpb/streamobj/basic/CellID.java index 54fe16c268..db62129d56 100644 --- a/tika-parsers/tika-parsers-standard/tika-parsers-standard-modules/tika-parser-microsoft-module/src/main/java/org/apache/tika/parser/microsoft/onenote/fsshttpb/streamobj/basic/CellID.java +++ b/tika-parsers/tika-parsers-standard/tika-parsers-standard-modules/tika-parser-microsoft-module/src/main/java/org/apache/tika/parser/microsoft/onenote/fsshttpb/streamobj/basic/CellID.java @@ -82,11 +82,10 @@ public List serializeToByteList() throws IOException { */ @Override public boolean equals(Object obj) { - CellID another = (CellID) obj; - - if (another == null) { + if (!(obj instanceof CellID)) { return false; } + CellID another = (CellID) obj; if (another.extendGUID1 != null && another.extendGUID2 != null && this.extendGUID1 != null && this.extendGUID2 != null) {