v0.2.54..v0.2.55 changeset SuperfluousNodeRemover.h

diff --git a/hoot-core/src/main/cpp/hoot/core/ops/SuperfluousNodeRemover.h b/hoot-core/src/main/cpp/hoot/core/ops/SuperfluousNodeRemover.h
index 49bac8f..2bd8213 100644
--- a/hoot-core/src/main/cpp/hoot/core/ops/SuperfluousNodeRemover.h
+++ b/hoot-core/src/main/cpp/hoot/core/ops/SuperfluousNodeRemover.h
@@ -54,6 +54,9 @@ class OsmMap;
  * If the bounds have been set via Boundable's setBounds then only nodes that are both not part
  * of a way and inside the bounds will be removed. This is most useful when performing tile based
  * operations such as the FourPassDriver.
+ *
+ * No point in implementing FilteredByGeometryTypeCriteria here, as there is no such thing as a map
+ * with no nodes.
  */
 class SuperfluousNodeRemover : public OsmMapOperation, public Serializable, public Boundable
 {
@@ -96,8 +99,12 @@ public:
 protected:
 
   geos::geom::Envelope _bounds;
+
   std::set<long> _usedNodes;
+
+  // if true, a node can be removed even if it has information tags (non-metadata)
   bool _ignoreInformationTags;
+  // configurable set of tags where if found on a node, we always want to remove it
   QStringList _unallowedOrphanKvps;
 
   int _taskStatusUpdateInterval;