Skip to content

v0.2.54..v0.2.55 changeset DuplicateNodeRemover.h

Garret Voltz edited this page Aug 14, 2020 · 1 revision
diff --git a/hoot-core/src/main/cpp/hoot/core/ops/DuplicateNodeRemover.h b/hoot-core/src/main/cpp/hoot/core/ops/DuplicateNodeRemover.h
index 064286f..25a1988 100644
--- a/hoot-core/src/main/cpp/hoot/core/ops/DuplicateNodeRemover.h
+++ b/hoot-core/src/main/cpp/hoot/core/ops/DuplicateNodeRemover.h
@@ -46,8 +46,14 @@ namespace hoot
  * Merges together nodes that are within a small configurable distance of each other and don't have
  * differing tags (outside of metadata tags).
  *
+ * Due to how Way::replaceNode is being called, we could end up with some duplicate way nodes, so
+ * its best to put RemoveDuplicateWayNodesVisitor in the cleaning chain immediately after this runs.
+ *
  * This class works with four pass as long as distance is less than the four pass buffer. The input
  * map can be in either a planar or geographic projection.
+ *
+ * No point in implementing FilteredByGeometryTypeCriteria here, as there is no such thing as a map
+ * with no nodes.
  */
 class DuplicateNodeRemover : public OsmMapOperation, public Serializable, public Boundable
 {
Clone this wiki locally