Removing this, which was suggested by a linter, turns out to break things by breaking equality comparison. The NetworkConfig struct would have to be re-implemented in a more modern way to allow the compiler to properly generate comparison operators, which is not important right now.

node/NetworkConfig.hpp

@@ -240,6 +240,15 @@ class NetworkConfig {
   public:
 	NetworkConfig();
 
+	inline bool operator==(const NetworkConfig& nc) const
+	{
+		return (memcmp(this, &nc, sizeof(NetworkConfig)) == 0);
+	}
+	inline bool operator!=(const NetworkConfig& nc) const
+	{
+		return (! (*this == nc));
+	}
+
 	/**
 	 * Write this network config to a dictionary for transport
 	 *