-
Notifications
You must be signed in to change notification settings - Fork 74
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)
{