StpBase: commented to note that calling ie->configChanged() from refreshDisplay() smells

Author: avarga

src/inet/linklayer/ieee8021d/common/StpBase.cc

@@ -121,7 +121,7 @@ void StpBase::refreshDisplay() const
                 if (nicModule != nullptr) {
                     std::string buf = std::string(port->getRoleShortName()) + "/" + port->getStateShortName();
                     nicModule->getDisplayString().setTagArg("t", 0, buf.c_str());
-                    ie->configChanged(NetworkInterface::F_IEEE8021D_DATA);
+                    ie->configChanged(NetworkInterface::F_IEEE8021D_DATA); // KLUDGE: abuse change notification mechanism to trick visualizer into copying the NIC display string onto the link
                 }
             }
             else {
@@ -131,7 +131,7 @@ void StpBase::refreshDisplay() const
                 // label ethernet interface with port status and role
                 if (nicModule != nullptr) {
                     nicModule->getDisplayString().setTagArg("t", 0, "");
-                    ie->configChanged(NetworkInterface::F_IEEE8021D_DATA);
+                    ie->configChanged(NetworkInterface::F_IEEE8021D_DATA); // KLUDGE: abuse change notification mechanism to trick visualizer into copying the NIC display string onto the link
                 }
             }
         }