Properly fix the tool switch insanity

How did I not think of this in the first place

garrysmod/gamemodes/sandbox/entities/weapons/gmod_tool/cl_init.lua

@@ -2,12 +2,9 @@
 local gmod_drawhelp = CreateClientConVar( "gmod_drawhelp", "1", true, false )
 local gmod_toolmode = CreateClientConVar( "gmod_toolmode", "rope", true, true )
 
-// HACK: We need a better way to do this
-local LastSwitchTime = 0
 cvars.AddChangeCallback( "gmod_toolmode", function( name, old, new )
-	if ( old == new || LastSwitchTime + .1 > CurTime() ) then return end
-	LastSwitchTime = CurTime()
-	spawnmenu.ActivateTool( new )
+	if ( old == new ) then return end
+	spawnmenu.ActivateTool( new, true )
 end, "gmod_toolmode_panel" )
 
 include( "shared.lua" )

garrysmod/lua/includes/modules/spawnmenu.lua

@@ -273,7 +273,7 @@ function ActivateToolPanel( id, cp )
 end
 
 -- While technically tool class names CAN be duplicate, it normally should never happen.
-function ActivateTool( strName )
+function ActivateTool( strName, noCommand )
 
 	-- I really don't like this triple loop
 	for tab, v in ipairs( g_ToolMenu ) do
@@ -282,7 +282,7 @@ function ActivateTool( strName )
 
 				if ( istable( item ) && item.ItemName && item.ItemName == strName ) then
 
-					if ( item.Command ) then
+					if ( !noCommand && item.Command ) then
 						RunConsoleCommand( unpack( string.Explode( " ", item.Command ) ) )
 					end