Inline CommandAPIBukkit#getKnownCommands since it is only being used once

willkroboth · willkroboth · commit b3078e4f26b7 · 2023-08-03T17:27:43.000-04:00
diff --git a/commandapi-platforms/commandapi-bukkit/commandapi-bukkit-core/src/main/java/dev/jorel/commandapi/CommandAPIBukkit.java b/commandapi-platforms/commandapi-bukkit/commandapi-bukkit-core/src/main/java/dev/jorel/commandapi/CommandAPIBukkit.java
@@ -488,7 +488,7 @@ public void unregister(String commandName, boolean force) {
 		if (!CommandAPI.canRegister() || force) {
 			// Bukkit is done with normal command stuff, so we have to modify their CommandMap ourselves
 			// If we're forcing, we'll also go here to make sure commands are really gone
-			Map<String, Command> knownCommands = getKnownCommands();
+			Map<String, Command> knownCommands = commandMapKnownCommands.get((SimpleCommandMap) paper.getCommandMap());
 			knownCommands.remove(commandName);
 			if (force) removeCommandNamespace(knownCommands, commandName);
 
@@ -531,10 +531,6 @@ private void removeCommandNamespace(Map<String, ?> map, String commandName) {
 		}
 	}
 
-	private Map<String, Command> getKnownCommands() {
-		return commandMapKnownCommands.get((SimpleCommandMap) paper.getCommandMap());
-	}
-
 	@Override
 	@Unimplemented(because = REQUIRES_MINECRAFT_SERVER)
 	public abstract CommandDispatcher<Source> getBrigadierDispatcher();
