From 2a5fae43b8238431b2847e63c6af79e4dfe6eb8f Mon Sep 17 00:00:00 2001
From: Freeman-AM <luc.marle@laposte.net>
Date: Sat, 11 Oct 2014 20:55:24 +0200
Subject: [PATCH] Plmenu: replace len hardcoded content by charsmax

---
 plugins/tfc/plmenu.sma | 160 ++++++++++++++++++++---------------------
 1 file changed, 80 insertions(+), 80 deletions(-)

diff --git a/plugins/tfc/plmenu.sma b/plugins/tfc/plmenu.sma
index 5ed9b16c..afa41b0c 100755
--- a/plugins/tfc/plmenu.sma
+++ b/plugins/tfc/plmenu.sma
@@ -61,8 +61,8 @@ public plugin_init()
 	g_coloredMenus = colored_menus()
 
 	new clcmds_ini_file[64]
-	get_configsdir(clcmds_ini_file, 63)
-	format(clcmds_ini_file, 63, "%s/clcmds.ini", clcmds_ini_file)
+	get_configsdir(clcmds_ini_file, charsmax(clcmds_ini_file))
+	format(clcmds_ini_file, charsmax(clcmds_ini_file), "%s/clcmds.ini", clcmds_ini_file)
 	load_settings(clcmds_ini_file)
 }
 
@@ -118,10 +118,10 @@ public actionBanMenu(id, key)
 			new player = g_menuPlayers[id][g_menuPosition[id] * 7 + key]
       
 			new name[MAX_NAME_LENGTH], name2[MAX_NAME_LENGTH], authid[32], authid2[32]
-			get_user_name(player, name2, 31)
-			get_user_authid(id, authid, 31)
-			get_user_authid(player, authid2, 31)
-			get_user_name(id, name, 31)
+			get_user_name(player, name2, charsmax(name2))
+			get_user_authid(id, authid, charsmax(authid))
+			get_user_authid(player, authid2, charsmax(authid2))
+			get_user_name(id, name, charsmax(name))
 			new userid2 = get_user_userid(player)
 
 			log_amx("Ban: ^"%s<%d><%s><>^" ban and kick ^"%s<%d><%s><>^" (minutes ^"%d^")", name, get_user_userid(id), authid, name2, userid2, authid2, g_menuSettings[id])
@@ -137,7 +137,7 @@ public actionBanMenu(id, key)
 					else
 					{
 						new tempTime[32];
-						formatex(tempTime,sizeof(tempTime)-1,"%d",g_menuSettings[id]);
+						formatex(tempTime,charsmax(tempTime),"%d",g_menuSettings[id]);
 						client_print(0, print_chat, "%L %s: %L %s %L", LANG_PLAYER, "ADMIN", name, LANG_PLAYER, "BAN", name2, LANG_PLAYER, "FOR_MIN", tempTime);
 					}
 				}
@@ -150,7 +150,7 @@ public actionBanMenu(id, key)
 					else
 					{
 						new tempTime[32];
-						formatex(tempTime,sizeof(tempTime)-1,"%d",g_menuSettings[id]);
+						formatex(tempTime,charsmax(tempTime),"%d",g_menuSettings[id]);
 						client_print(0, print_chat, "%L: %L %s %L", LANG_PLAYER, "ADMIN", LANG_PLAYER, "BAN", name2, LANG_PLAYER, "FOR_MIN", tempTime);
 					}
 				}
@@ -159,7 +159,7 @@ public actionBanMenu(id, key)
 			if (equal("4294967295", authid2))
 			{
 				new ipa[32]
-				get_user_ip(player, ipa, 31, 1)
+				get_user_ip(player, ipa, charsmax(ipa), 1)
 				server_cmd("addip %d %s;writeip", g_menuSettings[id], ipa)
 				if( g_tempBans )
 				{
@@ -200,7 +200,7 @@ displayBanMenu(id, pos)
 	if (start >= g_menuPlayersNum[id])
 		start = pos = g_menuPosition[id] = 0
 
-	new len = format(menuBody, 511, g_coloredMenus ? "\y%L\R%d/%d^n\w^n" : "%L %d/%d^n^n", id, "BAN_MENU", pos + 1, (g_menuPlayersNum[id] / 7 + ((g_menuPlayersNum[id] % 7) ? 1 : 0)))
+	new len = format(menuBody, charsmax(menuBody), g_coloredMenus ? "\y%L\R%d/%d^n\w^n" : "%L %d/%d^n^n", id, "BAN_MENU", pos + 1, (g_menuPlayersNum[id] / 7 + ((g_menuPlayersNum[id] % 7) ? 1 : 0)))
 	new end = start + 7
 	new keys = MENU_KEY_0|MENU_KEY_8
 
@@ -210,38 +210,38 @@ displayBanMenu(id, pos)
 	for (new a = start; a < end; ++a)
 	{
 		i = g_menuPlayers[id][a]
-		get_user_name(i, name, 31)
+		get_user_name(i, name, charsmax(name))
 
 		if (is_user_bot(i) || access(i, ADMIN_IMMUNITY))
 		{
 			++b
 			
 			if (g_coloredMenus)
-				len += format(menuBody[len], 511-len, "\d%d. %s^n\w", b, name)
+				len += format(menuBody[len], charsmax(menuBody)-len, "\d%d. %s^n\w", b, name)
 			else
-				len += format(menuBody[len], 511-len, "#. %s^n", name)
+				len += format(menuBody[len], charsmax(menuBody)-len, "#. %s^n", name)
 		} else {
 			keys |= (1<<b)
 				
 			if (is_user_admin(i))
-				len += format(menuBody[len], 511-len, g_coloredMenus ? "%d. %s \r*^n\w" : "%d. %s *^n", ++b, name)
+				len += format(menuBody[len], charsmax(menuBody)-len, g_coloredMenus ? "%d. %s \r*^n\w" : "%d. %s *^n", ++b, name)
 			else
-				len += format(menuBody[len], 511-len, "%d. %s^n", ++b, name)
+				len += format(menuBody[len], charsmax(menuBody)-len, "%d. %s^n", ++b, name)
 		}
 	}
 
 	if (g_menuSettings[id])
-		len += format(menuBody[len], 511-len, "^n8. %L^n", id, "BAN_FOR_MIN", g_menuSettings[id])
+		len += format(menuBody[len], charsmax(menuBody)-len, "^n8. %L^n", id, "BAN_FOR_MIN", g_menuSettings[id])
 	else
-		len += format(menuBody[len], 511-len, "^n8. %L^n", id, "BAN_PERM")
+		len += format(menuBody[len], charsmax(menuBody)-len, "^n8. %L^n", id, "BAN_PERM")
 
 	if (end != g_menuPlayersNum[id])
 	{
-		format(menuBody[len], 511-len, "^n9. %L...^n0. %L", id, "MORE", id, pos ? "BACK" : "EXIT")
+		format(menuBody[len], charsmax(menuBody)-len, "^n9. %L...^n0. %L", id, "MORE", id, pos ? "BACK" : "EXIT")
 		keys |= MENU_KEY_9
 	}
 	else
-		format(menuBody[len], 511-len, "^n0. %L", id, pos ? "BACK" : "EXIT")
+		format(menuBody[len], charsmax(menuBody)-len, "^n0. %L", id, pos ? "BACK" : "EXIT")
 
 	show_menu(id, keys, menuBody, -1, "Ban Menu")
 }
@@ -278,7 +278,7 @@ public actionSlapMenu(id,key) {
       new player = g_menuPlayers[id][g_menuPosition[id] * 7 + key]
       
       new name2[MAX_NAME_LENGTH]
-      get_user_name(player,name2,31)
+      get_user_name(player,name2,charsmax(name2))
       
       if (!is_user_alive(player)) {
         client_print(id,print_chat,"%L",id,"CANT_PERF_DEAD",name2)
@@ -288,9 +288,9 @@ public actionSlapMenu(id,key) {
             
       new authid[32],authid2[32], name[MAX_NAME_LENGTH]
 
-      get_user_authid(id,authid,31)
-      get_user_authid(player,authid2,31)
-      get_user_name(id,name,31)
+      get_user_authid(id,authid,charsmax(authid))
+      get_user_authid(player,authid2,charsmax(authid2))
+      get_user_name(id,name,charsmax(name))
         
       if ( g_menuOption[id] ) {
         log_amx("Cmd: ^"%s<%d><%s><>^" slap with %d damage ^"%s<%d><%s><>^"", 
@@ -336,7 +336,7 @@ displaySlapMenu(id, pos)
 	if (start >= g_menuPlayersNum[id])
 		start = pos = g_menuPosition[id] = 0
 
-	new len = format(menuBody, 511, g_coloredMenus ? "\y%L\R%d/%d^n\w^n" : "%L %d/%d^n^n", id, "SLAP_SLAY_MENU", pos + 1, (g_menuPlayersNum[id] / 7 + ((g_menuPlayersNum[id] % 7) ? 1 : 0)))
+	new len = format(menuBody, charsmax(menuBody), g_coloredMenus ? "\y%L\R%d/%d^n\w^n" : "%L %d/%d^n^n", id, "SLAP_SLAY_MENU", pos + 1, (g_menuPlayersNum[id] / 7 + ((g_menuPlayersNum[id] % 7) ? 1 : 0)))
 	new end = start + 7
 	new keys = MENU_KEY_0|MENU_KEY_8
 
@@ -346,7 +346,7 @@ displaySlapMenu(id, pos)
 	for (new a = start; a < end; ++a)
 	{
 		i = g_menuPlayers[id][a]
-		get_user_name(i, name, 31)
+		get_user_name(i, name, charsmax(name))
 		new iteam = get_user_team(i)
 
 		if (!is_user_alive(i) || access(i, ADMIN_IMMUNITY))
@@ -354,31 +354,31 @@ displaySlapMenu(id, pos)
 			++b
 		
 			if (g_coloredMenus)
-				len += format(menuBody[len], 511-len, "\d%d. %s\R%s^n\w", b, name, g_teamNames[iteam])
+				len += format(menuBody[len], charsmax(menuBody)-len, "\d%d. %s\R%s^n\w", b, name, g_teamNames[iteam])
 			else
-				len += format(menuBody[len], 511-len, "#. %s   %s^n", name, g_teamNames[iteam])		
+				len += format(menuBody[len], charsmax(menuBody)-len, "#. %s   %s^n", name, g_teamNames[iteam])		
 		} else {
 			keys |= (1<<b)
 				
 			if (is_user_admin(i))
-				len += format(menuBody[len], 511-len, g_coloredMenus ? "%d. %s \r*\y\R%s^n\w" : "%d. %s *   %s^n", ++b, name, g_teamNames[iteam])
+				len += format(menuBody[len], charsmax(menuBody)-len, g_coloredMenus ? "%d. %s \r*\y\R%s^n\w" : "%d. %s *   %s^n", ++b, name, g_teamNames[iteam])
 			else
-				len += format(menuBody[len], 511-len, g_coloredMenus ? "%d. %s\y\R%s^n\w" : "%d. %s   %s^n", ++b, name, g_teamNames[iteam])
+				len += format(menuBody[len], charsmax(menuBody)-len, g_coloredMenus ? "%d. %s\y\R%s^n\w" : "%d. %s   %s^n", ++b, name, g_teamNames[iteam])
 		}
 	}
 
 	if (g_menuOption[id])
-		len += format(menuBody[len], 511-len, "^n8. %L^n", id, "SLAP_WITH_DMG", g_menuSettings[id])
+		len += format(menuBody[len], charsmax(menuBody)-len, "^n8. %L^n", id, "SLAP_WITH_DMG", g_menuSettings[id])
 	else
-		len += format(menuBody[len], 511-len, "^n8. %L^n", id, "SLAY")
+		len += format(menuBody[len], charsmax(menuBody)-len, "^n8. %L^n", id, "SLAY")
 
 	if (end != g_menuPlayersNum[id])
 	{
-		format(menuBody[len], 511-len, "^n9. %L...^n0. %L", id, "MORE", id, pos ? "BACK" : "EXIT")
+		format(menuBody[len], charsmax(menuBody)-len, "^n9. %L...^n0. %L", id, "MORE", id, pos ? "BACK" : "EXIT")
 		keys |= MENU_KEY_9
 	}
 	else
-		format(menuBody[len], 511-len, "^n0. %L", id, pos ? "BACK" : "EXIT")
+		format(menuBody[len], charsmax(menuBody)-len, "^n0. %L", id, pos ? "BACK" : "EXIT")
 
 	show_menu(id, keys, menuBody, -1, "Slap/Slay Menu")
 }
@@ -407,10 +407,10 @@ public actionKickMenu(id,key)
       new player = g_menuPlayers[id][g_menuPosition[id] * 8 + key]
 
       new authid[32],authid2[32], name[MAX_NAME_LENGTH], name2[MAX_NAME_LENGTH]
-      get_user_authid(id,authid,31)
-      get_user_authid(player,authid2,31)
-      get_user_name(id,name,31)
-      get_user_name(player,name2,31)      
+      get_user_authid(id,authid,charsmax(authid))
+      get_user_authid(player,authid2,charsmax(authid2))
+      get_user_name(id,name,charsmax(name))
+      get_user_name(player,name2,charsmax(name2))      
       new userid2 = get_user_userid(player)
 
       log_amx("Kick: ^"%s<%d><%s><>^" kick ^"%s<%d><%s><>^"", 
@@ -446,7 +446,7 @@ displayKickMenu(id, pos)
 	if (start >= g_menuPlayersNum[id])
 		start = pos = g_menuPosition[id] = 0
 
-	new len = format(menuBody, 511, g_coloredMenus ? "\y%L\R%d/%d^n\w^n" : "%L %d/%d^n^n", id, "KICK_MENU", pos + 1, (g_menuPlayersNum[id] / 8 + ((g_menuPlayersNum[id] % 8) ? 1 : 0)))
+	new len = format(menuBody, charsmax(menuBody), g_coloredMenus ? "\y%L\R%d/%d^n\w^n" : "%L %d/%d^n^n", id, "KICK_MENU", pos + 1, (g_menuPlayersNum[id] / 8 + ((g_menuPlayersNum[id] % 8) ? 1 : 0)))
 	new end = start + 8
 	new keys = MENU_KEY_0
 
@@ -456,33 +456,33 @@ displayKickMenu(id, pos)
 	for (new a = start; a < end; ++a)
 	{
 		i = g_menuPlayers[id][a]
-		get_user_name(i, name, 31)
+		get_user_name(i, name, charsmax(name))
 
 		if (access(i, ADMIN_IMMUNITY))
 		{
 			++b
 		
 			if (g_coloredMenus)
-				len += format(menuBody[len], 511-len, "\d%d. %s^n\w", b, name)
+				len += format(menuBody[len], charsmax(menuBody)-len, "\d%d. %s^n\w", b, name)
 			else
-				len += format(menuBody[len], 511-len, "#. %s^n", name)
+				len += format(menuBody[len], charsmax(menuBody)-len, "#. %s^n", name)
 		} else {
 			keys |= (1<<b)
 				
 			if (is_user_admin(i))
-				len += format(menuBody[len], 511-len, g_coloredMenus ? "%d. %s \r*^n\w" : "%d. %s *^n", ++b, name)
+				len += format(menuBody[len], charsmax(menuBody)-len, g_coloredMenus ? "%d. %s \r*^n\w" : "%d. %s *^n", ++b, name)
 			else
-				len += format(menuBody[len], 511-len, "%d. %s^n", ++b, name)
+				len += format(menuBody[len], charsmax(menuBody)-len, "%d. %s^n", ++b, name)
 		}
 	}
 
 	if (end != g_menuPlayersNum[id])
 	{
-		format(menuBody[len], 511-len, "^n9. %L...^n0. %L", id, "MORE", id, pos ? "BACK" : "EXIT")
+		format(menuBody[len], charsmax(menuBody)-len, "^n9. %L...^n0. %L", id, "MORE", id, pos ? "BACK" : "EXIT")
 		keys |= MENU_KEY_9
 	}
 	else
-		format(menuBody[len], 511-len, "^n0. %L", id, pos ? "BACK" : "EXIT")
+		format(menuBody[len], charsmax(menuBody)-len, "^n0. %L", id, pos ? "BACK" : "EXIT")
 
 	show_menu(id, keys, menuBody, -1, "Kick Menu")
 }
@@ -510,10 +510,10 @@ public actionTeamMenu(id,key) {
     default: {
       new player = g_menuPlayers[id][g_menuPosition[id] * 7 + key]
       new authid[32],authid2[32], name[MAX_NAME_LENGTH], name2[MAX_NAME_LENGTH]
-      get_user_name(player,name2,31)
-      get_user_authid(id,authid,31)
-      get_user_authid(player,authid2,31)
-      get_user_name(id,name,31)
+      get_user_name(player,name2,charsmax(name2))
+      get_user_authid(id,authid,charsmax(authid))
+      get_user_authid(player,authid2,charsmax(authid2))
+      get_user_name(id,name,charsmax(name))
 
       log_amx("Cmd: ^"%s<%d><%s><>^" transfer ^"%s<%d><%s><>^" (team ^"%s^")", 
           name,get_user_userid(id),authid, name2,get_user_userid(player),authid2, g_teamNames[g_menuOption[id]] )
@@ -522,8 +522,8 @@ public actionTeamMenu(id,key) {
         case 1: client_print(0, print_chat, "%L", LANG_PLAYER, "ADMIN_TRANSF_1", name2, g_teamNames[g_menuOption[id]])
       }
 
-      new szCmd[2]
-      format(szCmd,2,"%d",g_menuOption[id])
+      new szCmd[3]
+      format(szCmd,charsmax(szCmd),"%d",g_menuOption[id])
       tfc_userkill(player)
       if (g_menuOption[id] == 5)
       {
@@ -555,7 +555,7 @@ displayTeamMenu(id, pos)
 	if (start >= g_menuPlayersNum[id])
 		start = pos = g_menuPosition[id] = 0
 
-	new len = format(menuBody, 511, g_coloredMenus ? "\y%L\R%d/%d^n\w^n" : "%L %d/%d^n^n", id, "TEAM_MENU", pos + 1, (g_menuPlayersNum[id] / 7 + ((g_menuPlayersNum[id] % 7) ? 1 : 0)))
+	new len = format(menuBody, charsmax(menuBody), g_coloredMenus ? "\y%L\R%d/%d^n\w^n" : "%L %d/%d^n^n", id, "TEAM_MENU", pos + 1, (g_menuPlayersNum[id] / 7 + ((g_menuPlayersNum[id] % 7) ? 1 : 0)))
 	new end = start + 7
 	new keys = MENU_KEY_0|MENU_KEY_8
 
@@ -565,7 +565,7 @@ displayTeamMenu(id, pos)
 	for (new a = start; a < end; ++a)
 	{
 		i = g_menuPlayers[id][a]
-		get_user_name(i, name, 31)
+		get_user_name(i, name, charsmax(name))
 		iteam = get_user_team(i)
 
 		if ((iteam == g_menuOption[id]) || access(i, ADMIN_IMMUNITY))
@@ -573,28 +573,28 @@ displayTeamMenu(id, pos)
 			++b
 			
 			if (g_coloredMenus)
-				len += format(menuBody[len], 511-len, "\d%d. %s\R%s^n\w", b, name, g_teamNames[iteam])
+				len += format(menuBody[len], charsmax(menuBody)-len, "\d%d. %s\R%s^n\w", b, name, g_teamNames[iteam])
 			else
-				len += format(menuBody[len], 511-len, "#. %s   %s^n", name, g_teamNames[iteam])		
+				len += format(menuBody[len], charsmax(menuBody)-len, "#. %s   %s^n", name, g_teamNames[iteam])		
 		} else {
 			keys |= (1<<b)
 				
 			if (is_user_admin(i))
-				len += format(menuBody[len], 511-len, g_coloredMenus ? "%d. %s \r*\y\R%s^n\w" : "%d. %s *   %s^n", ++b, name, g_teamNames[iteam])
+				len += format(menuBody[len], charsmax(menuBody)-len, g_coloredMenus ? "%d. %s \r*\y\R%s^n\w" : "%d. %s *   %s^n", ++b, name, g_teamNames[iteam])
 			else
-				len += format(menuBody[len], 511-len, g_coloredMenus ? "%d. %s\y\R%s^n\w" : "%d. %s   %s^n", ++b, name, g_teamNames[iteam])
+				len += format(menuBody[len], charsmax(menuBody)-len, g_coloredMenus ? "%d. %s\y\R%s^n\w" : "%d. %s   %s^n", ++b, name, g_teamNames[iteam])
 		}
 	}
 
-	len += format(menuBody[len], 511-len, "^n8. %L^n", id, "TRANSF_TO", g_teamNames[g_menuOption[id]])
+	len += format(menuBody[len], charsmax(menuBody)-len, "^n8. %L^n", id, "TRANSF_TO", g_teamNames[g_menuOption[id]])
 
 	if (end != g_menuPlayersNum[id])
 	{
-		format(menuBody[len], 511-len, "^n9. %L...^n0. %L", id, "MORE", id, pos ? "BACK" : "EXIT")
+		format(menuBody[len], charsmax(menuBody)-len, "^n9. %L...^n0. %L", id, "MORE", id, pos ? "BACK" : "EXIT")
 		keys |= MENU_KEY_9
 	}
 	else
-		format(menuBody[len], 511-len, "^n0. %L", id, pos ? "BACK" : "EXIT")
+		format(menuBody[len], charsmax(menuBody)-len, "^n0. %L", id, pos ? "BACK" : "EXIT")
 
 	show_menu(id, keys, menuBody, -1, "Team Menu")
 }
@@ -627,13 +627,13 @@ public actionClcmdMenu(id,key) {
       new flags = g_clcmdMisc[g_menuSelect[id][g_menuOption[id]]][1]
       if (is_user_connected(player)) {
         new command[64], authid[32], name[MAX_NAME_LENGTH], userid[32]
-        copy(command,63,g_clcmdCmd[g_menuSelect[id][g_menuOption[id]]])
-        get_user_authid(player,authid,31)
-        get_user_name(player,name,31)
-        num_to_str(get_user_userid(player),userid,31)
-        replace(command,63,"%userid%",userid)
-        replace(command,63,"%authid%",authid)
-        replace(command,63,"%name%",name)
+        copy(command,charsmax(command),g_clcmdCmd[g_menuSelect[id][g_menuOption[id]]])
+        get_user_authid(player,authid,charsmax(authid))
+        get_user_name(player,name,charsmax(name))
+        num_to_str(get_user_userid(player),userid,charsmax(userid))
+        replace(command,charsmax(command),"%userid%",userid)
+        replace(command,charsmax(command),"%authid%",authid)
+        replace(command,charsmax(command),"%name%",name)
         if (flags & 1) {
           server_cmd("%s", command)
           server_exec()
@@ -665,7 +665,7 @@ displayClcmdMenu(id, pos)
 	if (start >= g_menuPlayersNum[id])
 		start = pos = g_menuPosition[id] = 0
 
-	new len = format(menuBody, 511, g_coloredMenus ? "\y%L\R%d/%d^n\w^n" : "%L %d/%d^n^n", id, "CL_CMD_MENU", pos + 1, (g_menuPlayersNum[id] / 7 + ((g_menuPlayersNum[id] % 7) ? 1 : 0)))
+	new len = format(menuBody, charsmax(menuBody), g_coloredMenus ? "\y%L\R%d/%d^n\w^n" : "%L %d/%d^n^n", id, "CL_CMD_MENU", pos + 1, (g_menuPlayersNum[id] / 7 + ((g_menuPlayersNum[id] % 7) ? 1 : 0)))
 	new end = start + 7
 	new keys = MENU_KEY_0|MENU_KEY_8
 
@@ -675,38 +675,38 @@ displayClcmdMenu(id, pos)
 	for (new a = start; a < end; ++a)
 	{
 		i = g_menuPlayers[id][a]
-		get_user_name(i, name, 31)
+		get_user_name(i, name, charsmax(name))
 
 		if (!g_menuSelectNum[id] || access(i, ADMIN_IMMUNITY))
 		{
 			++b
 			
 			if (g_coloredMenus)
-				len += format(menuBody[len], 511-len, "\d%d. %s^n\w", b, name)
+				len += format(menuBody[len], charsmax(menuBody)-len, "\d%d. %s^n\w", b, name)
 			else
-				len += format(menuBody[len], 511-len, "#. %s^n", name)		
+				len += format(menuBody[len], charsmax(menuBody)-len, "#. %s^n", name)		
 		} else {
 			keys |= (1<<b)
 				
 			if (is_user_admin(i))
-				len += format(menuBody[len], 511-len, g_coloredMenus ? "%d. %s \r*^n\w" : "%d. %s *^n", ++b, name)
+				len += format(menuBody[len], charsmax(menuBody)-len, g_coloredMenus ? "%d. %s \r*^n\w" : "%d. %s *^n", ++b, name)
 			else
-				len += format(menuBody[len], 511-len, "%d. %s^n", ++b, name)
+				len += format(menuBody[len], charsmax(menuBody)-len, "%d. %s^n", ++b, name)
 		}
 	}
 
 	if (g_menuSelectNum[id])
-		len += format(menuBody[len], 511-len, "^n8. %s^n", g_clcmdName[g_menuSelect[id][g_menuOption[id]]])
+		len += format(menuBody[len], charsmax(menuBody)-len, "^n8. %s^n", g_clcmdName[g_menuSelect[id][g_menuOption[id]]])
 	else
-		len += format(menuBody[len], 511-len, "^n8. %L^n", id, "NO_CMDS")
+		len += format(menuBody[len], charsmax(menuBody)-len, "^n8. %L^n", id, "NO_CMDS")
 
 	if (end != g_menuPlayersNum[id])
 	{
-		format(menuBody[len], 511-len, "^n9. %L...^n0. %L", id, "MORE", id, pos ? "BACK" : "EXIT")
+		format(menuBody[len], charsmax(menuBody)-len, "^n9. %L...^n0. %L", id, "MORE", id, pos ? "BACK" : "EXIT")
 		keys |= MENU_KEY_9
 	}
 	else
-		format(menuBody[len], 511-len, "^n0. %L", id, pos ? "BACK" : "EXIT")
+		format(menuBody[len], charsmax(menuBody)-len, "^n0. %L", id, pos ? "BACK" : "EXIT")
 
 	show_menu(id, keys, menuBody, -1, "Client Cmds Menu")
 }
@@ -739,13 +739,13 @@ load_settings(szFilename[])
 	new text[256], szFlags[32], szAccess[32]
 	new a, pos = 0
 
-	while (g_clcmdNum < MAX_CLCMDS && read_file(szFilename, pos++, text, 255, a))
+	while (g_clcmdNum < MAX_CLCMDS && read_file(szFilename, pos++, text, charsmax(text), a))
 	{
 		if (text[0] == ';') continue
 
-		if (parse(text, g_clcmdName[g_clcmdNum], 31, g_clcmdCmd[g_clcmdNum], 63, szFlags, 31, szAccess, 31) > 3)
+		if (parse(text, g_clcmdName[g_clcmdNum], charsmax(g_clcmdName[]), g_clcmdCmd[g_clcmdNum], charsmax(g_clcmdCmd[]), szFlags, charsmax(szFlags), szAccess, charsmax(szAccess)) > 3)
 		{
-			while (replace(g_clcmdCmd[g_clcmdNum], 63, "\'", "^""))
+			while (replace(g_clcmdCmd[g_clcmdNum], charsmax(g_clcmdCmd[]), "\'", "^""))
 			{
 				// do nothing
 			}