Skip to content

v0.2.52..v0.2.53 changeset Way.cpp

Garret Voltz edited this page Feb 12, 2020 · 1 revision
diff --git a/hoot-core/src/main/cpp/hoot/core/elements/Way.cpp b/hoot-core/src/main/cpp/hoot/core/elements/Way.cpp
index 87f8749..95b254c 100644
--- a/hoot-core/src/main/cpp/hoot/core/elements/Way.cpp
+++ b/hoot-core/src/main/cpp/hoot/core/elements/Way.cpp
@@ -22,7 +22,7 @@
  * This will properly maintain the copyright information. DigitalGlobe
  * copyrights will be updated automatically.
  *
- * @copyright Copyright (C) 2015, 2016, 2017, 2018, 2019 DigitalGlobe (http://www.digitalglobe.com/)
+ * @copyright Copyright (C) 2015, 2016, 2017, 2018, 2019, 2020 DigitalGlobe (http://www.digitalglobe.com/)
  */
 
 #include "Way.h"
@@ -357,9 +357,10 @@ void Way::replaceNode(long oldId, long newId)
   // MergeNearbyNodes. Initially made some attempts in here to clean that up, but it introduced
   // unwanted logic and runtime complexity. Ultimately since the problem was only seen with
   // MergeNearbyNodes, it was updated to remove the duplicated way nodes at the end of its run and
-  // this logic was left alone. The argument can be made that this method allows you to replace
-  // whatever is desired, but its the caller's responsbility to make sure the result of the
-  // replacement is valid.
+  // this logic was left alone. To validate this behavior, the argument can be made that this method
+  // allows you to replace whatever is desired, but its the caller's responsbility to make sure the
+  // result of the replacement is valid. If we ever decide we want to make this more foolproof for
+  // the caller, then a different approach should be taken similar to the changes attempted before.
 
   if (oldId == newId)
   {
Clone this wiki locally