removed unnecessary line in KBestHaplotypeFinder that could cause infinite recursion (#5786)

davidbenjamin · web-flow · commit 913f24dccd0f · 2019-03-12T15:42:22.000-04:00
diff --git a/src/main/java/org/broadinstitute/hellbender/tools/walkers/haplotypecaller/graphs/KBestHaplotypeFinder.java b/src/main/java/org/broadinstitute/hellbender/tools/walkers/haplotypecaller/graphs/KBestHaplotypeFinder.java
@@ -161,7 +161,6 @@ private static boolean findGuiltyVerticesAndEdgesToRemoveCycles(final SeqGraph g
                 reachesSink = reachesSink || childReachSink;
             }
         }
-        parentVertices.remove(currentVertex);
         if (!reachesSink) {
             verticesToRemove.add(currentVertex);
         }

Original file line number	Diff line number	Diff line change
`@@ -161,7 +161,6 @@ private static boolean findGuiltyVerticesAndEdgesToRemoveCycles(final SeqGraph g`
`161`	`161`	`reachesSink = reachesSink \|\| childReachSink;`
`162`	`162`	`}`
`163`	`163`	`}`
`164`		`- parentVertices.remove(currentVertex);`
`165`	`164`	`if (!reachesSink) {`
`166`	`165`	`verticesToRemove.add(currentVertex);`
`167`	`166`	`}`