Merge pull request #35 from dityas/dev

Log null children while pre computing hashes
dityas · Dec 30, 2019 · f232a24 · f232a24
2 parents 565e9cb + 69816dc
commit f232a24
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/Protos/build/Protos.jar b/Protos/build/Protos.jar
diff --git a/Protos/src/thinclab/legacy/DDnode.java b/Protos/src/thinclab/legacy/DDnode.java
@@ -44,8 +44,14 @@ private void precomputeHash() {
 		HashCodeBuilder builder = new HashCodeBuilder();
 		builder.append(this.var);
 
-		for (int i=0; i<children.length; i++)
-			builder.append(this.children[i].hashCode());
+		for (int i=0; i<children.length; i++) {
+
+			/* check for null children, if this happens, something is wrong */
+			if (this.children[i] != null)
+				builder.append(this.children[i].hashCode());
+
+			else logger.error("Null child at " + i + " something might be seriously wrong.");
+		}
 
 		this.hash = builder.toHashCode();
 	}