Comment on potentially confusing code

git-svn-id: http://svn.inspircd.org/repository/trunk/inspircd@10616 e03df62e-2008-0410-955e-edbf42e46eb7

1 files changed, 8 insertions, 0 deletions

diff --git a/src/modules/m_servprotect.cpp b/src/modules/m_servprotect.cpp
index 687c45a94..f77bdbc6a 100644
--- a/src/modules/m_servprotect.cpp
+++ b/src/modules/m_servprotect.cpp
@@ -30,6 +30,14 @@ class ServProtectMode : public ModeHandler
 
 	ModeAction OnModeChange(User* source, User* dest, Channel* channel, std::string &parameter, bool adding, bool)
 	{
+		/* Because this returns MODEACTION_DENY all the time, there is only ONE 
+		 * way to add this mode and that is at client introduction in the UID command,
+		 * as this calls OnModeChange for each mode but disregards the return values.
+		 * The mode cannot be manually added or removed, not even by a server or by a remote
+		 * user, which prevents its (ab)use as a kiddie 'god mode' on such networks.
+		 * I'm sure if someone really wants to do that they can make a copy of this module
+		 * that does the job. It won't be me though!
+		 */
 		return MODEACTION_DENY;
 	}