Skip to content

v0.2.55..v0.2.56 changeset NodeDensityTilesCmd.cpp

Garret Voltz edited this page Aug 14, 2020 · 3 revisions
diff --git a/hoot-core/src/main/cpp/hoot/core/cmd/NodeDensityTilesCmd.cpp b/hoot-core/src/main/cpp/hoot/core/cmd/NodeDensityTilesCmd.cpp
index 9df08c6..08be0c5 100644
--- a/hoot-core/src/main/cpp/hoot/core/cmd/NodeDensityTilesCmd.cpp
+++ b/hoot-core/src/main/cpp/hoot/core/cmd/NodeDensityTilesCmd.cpp
@@ -51,7 +51,7 @@ public:
 
   static std::string className() { return "hoot::NodeDensityTilesCmd"; }
 
-  NodeDensityTilesCmd() { }
+  NodeDensityTilesCmd() = default;
 
   virtual QString getName() const override { return "node-density-tiles"; }
 
@@ -249,6 +249,7 @@ private:
     for (int i = 0; i < inputs.size(); i++)
     {
       const QString input = inputs.at(i);
+      // IMPORTANT: has to be loaded as Unknown1 in order for the calculation to work
       std::shared_ptr<OsmMapReader> reader =
         OsmMapReaderFactory::createReader(input, true, Status::Unknown1);
 
@@ -265,10 +266,10 @@ private:
         std::dynamic_pointer_cast<ApiDbReader>(reader);
       if (apiDbReader)
       {
-        //the tiles calculation is only concerned with nodes, and the only readers capable of
-        //filtering down to nodes up front right now are the api db readers; more importantly,
-        //setting this to true also prevents any features from being returned outside of
-        //convert.bounding.box, if it was specified (ways partially inside the bounds, etc.)
+        // The tiles calculation is only concerned with nodes, and the only readers capable of
+        // filtering down to nodes up front right now are the api db readers. More importantly,
+        // setting this to true also prevents any features from being returned outside of
+        // convert.bounding.box, if it was specified (ways partially inside the bounds, etc.)
         apiDbReader->setReturnNodesOnly(true);
       }
 
Clone this wiki locally