Replace IS_* with member functions.

- All user types get an Is* function.
- Only local users are cast using the old function so only local
  users get an As* function.

1 files changed, 5 insertions, 3 deletions

diff --git a/modules/spanningtree/servercommand.h b/modules/spanningtree/servercommand.h
index af1761cb0..e4e2062cf 100644
--- a/modules/spanningtree/servercommand.h
+++ b/modules/spanningtree/servercommand.h
@@ -71,9 +71,10 @@ public:
 
 	CmdResult Handle(User* user, Params& parameters) override
 	{
-		RemoteUser* remoteuser = IS_REMOTE(user);
-		if (!remoteuser)
+		if (!user->IsRemote())
 			throw ProtocolException("Invalid source");
+
+		auto* remoteuser = static_cast<RemoteUser*>(user);
 		return static_cast<T*>(this)->HandleRemote(remoteuser, parameters);
 	}
 };
@@ -91,8 +92,9 @@ public:
 
 	CmdResult Handle(User* user, CommandBase::Params& parameters) override
 	{
-		if (!IS_SERVER(user))
+		if (!user->IsServer())
 			throw ProtocolException("Invalid source");
+
 		TreeServer* server = TreeServer::Get(user);
 		return static_cast<T*>(this)->HandleServer(server, parameters);
 	}