Merge pull request #22 from omcho420/master

use of 'sizeof' argument

Author: Southclaws

a_npc.inc

@@ -288,7 +288,7 @@ native printf(const format[], {Float,_}:...);
 /// </remarks>
 /// <remarks>The values for the placeholders follow in the exact same order as parameters in the call. For example, <b><c>"I am %i years old"</c></b> - the <b><c>%i</c></b> will be replaced with an Integer variable, which is the person's age.</remarks>
 /// <remarks>You may optionally put a number between the <b><c>%</c></b> and the letter of the placeholder code. This number indicates the field width; if the size of the parameter to print at the position of the placeholder is smaller than the field width, the field is expanded with spaces. To cut the number of decimal places beeing shown of a float, you can add <b><c>.&lt;max number&gt;</c></b> between the <b><c>%</c></b> and the <b><c>f</c></b>. (example: <b><c>%.2f</c></b>)</remarks>
-native format(output[], len, const format[], {Float,_}:...);
+native format(output[], len = sizeof output, const format[], {Float,_}:...);
 
 /// <summary>Sets a 'timer' to call a function after some time. Can be set to repeat.</summary>
 /// <param name="funcname">Name of the function to call as a string. This must be a public function (forwarded). A null string here will crash the server</param>
@@ -512,7 +512,7 @@ native IsPlayerInRangeOfPoint(playerid, Float:range, Float:X, Float:Y, Float:Z);
 /// <seealso name="GetPlayerVersion"/>
 /// <remarks>A player's name can be up to 24 characters long (as of <b>0.3d R2</b>) by using <a href="#SetPlayerName">SetPlayerName</a>. This is defined in <c>a_samp.inc</c> as <b><c>MAX_PLAYER_NAME</c></b>. However, the client can only join with a nickname between 3 and 20 characters, otherwise the connection will be rejected and the player has to quit to choose a valid name.</remarks>
 /// <returns>The length of the player's name. <b><c>0</c></b> if player specified doesn't exist.</returns>
-native GetPlayerName(playerid, name[], len);
+native GetPlayerName(playerid, name[], len = sizeof name);
 
 /// <summary>Checks if a player is connected (if an ID is taken by a connected player).</summary>
 /// <param name="playerid">The ID of the player to check</param>

a_players.inc

@@ -611,7 +611,7 @@ native GetPlayerState(playerid);
 /// <seealso name="OnPlayerDisconnect"/>
 /// <remarks>This function does not work when used in <a href="#OnPlayerDisconnect">OnPlayerDisconnect</a> because the player is already disconnected. It will return an invalid IP (<b><c>255.255.255.255</c></b>). Save players' IPs under <a href="#OnPlayerConnect">OnPlayerConnect</a> if they need to be used under <a href="#OnPlayerDisconnect">OnPlayerDisconnect</a>. </remarks>
 /// <returns><b><c>1</c></b> on success and <b><c>0</c></b> on failure.</returns>
-native GetPlayerIp(playerid, name[], len);
+native GetPlayerIp(playerid, ip[], len = sizeof ip);
 
 /// <summary>Get the ping of a player. The ping measures the amount of time it takes for the server to 'ping' the client and for the client to send the message back.</summary>
 /// <param name="playerid">The ID of the player to get the ping of</param>
@@ -653,7 +653,7 @@ native GetPlayerKeys(playerid, &keys, &updown, &leftright);
 /// <seealso name="GetPlayerVersion"/>
 /// <remarks>A player's name can be up to 24 characters long (as of <b>0.3d R2</b>) by using <a href="#SetPlayerName">SetPlayerName</a>. This is defined in <c>a_samp.inc</c> as <b><c>MAX_PLAYER_NAME</c></b>. However, the client can only join with a nickname between 3 and 20 characters, otherwise the connection will be rejected and the player has to quit to choose a valid name.</remarks>
 /// <returns>The length of the player's name. <b><c>0</c></b> if player specified doesn't exist.</returns>
-native GetPlayerName(playerid, name[], len);
+native GetPlayerName(playerid, name[], len = sizeof name);
 
 /// <summary>Sets the game time for a player. If a player's clock is enabled (<a href="#TogglePlayerClock">TogglePlayerClock</a>) the time displayed by it will update automatically.</summary>
 /// <param name="playerid">The ID of the player to set the game time of</param>
@@ -1545,7 +1545,7 @@ native SetPVarString(playerid, const varname[], const string_value[]);
 /// <seealso name="DeletePVar"/>
 /// <remarks>If length of string is zero (value not set), string_return text will not be updated or set to anything and will remain with old data, neccesying that you clear the variable to blank value if <a href="#GetPVarString">GetPVarString</a> returns <b><c>0</c></b> if that behavior is undesired </remarks>
 /// <returns>The length of the string.</returns>
-native GetPVarString(playerid, const varname[], string_return[], len);
+native GetPVarString(playerid, const varname[], string_return[], len = sizeof string_return);
 
 /// <summary>Set a float player variable's value.</summary>
 /// <param name="playerid">The ID of the player whose player variable will be set</param>
@@ -1607,7 +1607,7 @@ native GetPVarsUpperIndex(playerid);
 /// <seealso name="GetPVarInt"/>
 /// <seealso name="GetPVarFloat"/>
 /// <seealso name="GetPVarString"/>
-native GetPVarNameAtIndex(playerid, index, ret_varname[], ret_len);
+native GetPVarNameAtIndex(playerid, index, ret_varname[], ret_len = sizeof ret_varname);
 
 /// <summary>Gets the type (integer, float or string) of a player variable.</summary>
 /// <param name="playerid">The ID of the player whose player variable to get the type of</param>
@@ -1775,7 +1775,7 @@ native GetPlayerAnimationIndex(playerid); // return the index of any running app
 /// <seealso name="GetPlayerAnimationIndex"/>
 /// <remarks>This function was added in <b>SA-MP 0.3b</b> and will not work in earlier versions!</remarks>
 /// <returns><b><c>1</c></b> on success, <b><c>0</c></b> on failure.</returns>
-native GetAnimationName(index, animlib[], len1, animname[], len2); // get the animation lib/name for the index
+native GetAnimationName(index, animlib[], len1 = sizeof animlib, animname[], len2 = sizeof animname); // get the animation lib/name for the index
 
 /// <summary>Retrieves a player's current <a href="http://wiki.sa-mp.com/wiki/SpecialActions">special action</a>.</summary>
 /// <param name="playerid">The ID of the player to get the <a href="http://wiki.sa-mp.com/wiki/SpecialActions">special action</a> of</param>

a_samp.inc

@@ -385,7 +385,7 @@ native printf(const format[], {Float,_}:...);
 /// </remarks>
 /// <remarks>The values for the placeholders follow in the exact same order as parameters in the call. For example, <b><c>"I am %i years old"</c></b> - the <b><c>%i</c></b> will be replaced with an Integer variable, which is the person's age.</remarks>
 /// <remarks>You may optionally put a number between the <b><c>%</c></b> and the letter of the placeholder code. This number indicates the field width; if the size of the parameter to print at the position of the placeholder is smaller than the field width, the field is expanded with spaces. To cut the number of decimal places beeing shown of a float, you can add <b><c>.&lt;max number&gt;</c></b> between the <b><c>%</c></b> and the <b><c>f</c></b>. (example: <b><c>%.2f</c></b>)</remarks>
-native format(output[], len, const format[], {Float,_}:...);
+native format(output[], len = sizeof output, const format[], {Float,_}:...);
 
 /// <summary>This function sends a message to a specific player with a chosen color in the chat. The whole line in the chatbox will be in the set color unless color embedding is used (since <b><c>0.3c</c></b>).</summary>
 /// <param name="playerid">The ID of the player to display the message to</param>
@@ -653,7 +653,7 @@ native GetActorPoolSize();
 /// <remarks>This function was added in <b>SA-MP 0.3.7-R1</b> and will not work in earlier versions!</remarks>
 /// <remarks>The salt is appended to the end of the password, meaning password 'foo' and salt 'bar' would form 'foobar'. </remarks>
 /// <remarks>The salt should be random, unique for each player and at least as long as the hashed password. It is to be stored alongside the actual hash in the player's account. </remarks>
-native SHA256_PassHash(const password[], const salt[], ret_hash[], ret_hash_len); // SHA256 for password hashing
+native SHA256_PassHash(const password[], const salt[], ret_hash[], ret_hash_len = sizeof ret_hash_len); // SHA256 for password hashing
 
 // Server wide persistent variable system (SVars)
 
@@ -713,7 +713,7 @@ native SetSVarString(const varname[], const string_value[]);
 /// <seealso name="DeleteSVar"/>
 /// <remarks>This function was added in <b>SA-MP 0.3.7 R2</b> and will not work in earlier versions!</remarks>
 /// <returns>The length of the string.</returns>
-native GetSVarString(const varname[], string_return[], len);
+native GetSVarString(const varname[], string_return[], len = sizeof string_return);
 
 /// <summary>Set a float server variable.</summary>
 /// <param name="varname">The name of the server variable</param>
@@ -772,7 +772,7 @@ native GetSVarsUpperIndex();
 /// <seealso name="GetSVarInt"/>
 /// <seealso name="GetSVarFloat"/>
 /// <seealso name="GetSVarString"/>
-native GetSVarNameAtIndex(index, ret_varname[], ret_len);
+native GetSVarNameAtIndex(index, ret_varname[], ret_len = sizeof ret_varname);
 
 /// <summary>Gets the type (integer, float or string) of a server variable.</summary>
 /// <param name="varname">The name of the server variable to get the type of</param>
@@ -1020,7 +1020,7 @@ native SetWorldTime(hour);
 ///   <b><c>1</c></b>: The function executed successfully.<p/>
 ///   <b><c>0</c></b>: The function failed to execute. The weapon specified does not exist.
 /// </returns>
-native GetWeaponName(weaponid, weapon[], len);
+native GetWeaponName(weaponid, weapon[], len = sizeof weapon);
 
 /// <param name="enable"><b><c>1</c></b> to enable, <b><c>0</c></b> to disable tire popping</param>
 /// <remarks>This function was removed in <b>SA-MP 0.3</b>. Tire popping is enabled by default. If you want to disable tire popping, you'll have to manually script it using <a href="#OnVehicleDamageStatusUpdate">OnVehicleDamageStatusUpdate</a>.</remarks>
@@ -1158,8 +1158,8 @@ native IsPlayerNPC(playerid);
 native AddCharModel(baseid, newid, const dffname[], const txdname[]);
 native AddSimpleModel(virtualworld, baseid, newid, const dffname[], const txdname[]);
 native AddSimpleModelTimed(virtualworld, baseid, newid, const dffname[], const txdname[], timeon, timeoff);
-native FindModelFileNameFromCRC(crc, retstr[], retstr_size);
-native FindTextureFileNameFromCRC(crc, retstr[], retstr_size);
+native FindModelFileNameFromCRC(crc, retstr[], retstr_size = sizeof retstr);
+native FindTextureFileNameFromCRC(crc, retstr[], retstr_size = sizeof retstr);
 native RedirectDownload(playerid, const url[]);
 
 // Admin
@@ -1223,7 +1223,7 @@ native SendRconCommand(const command[]);
 /// <seealso name="NetStats_GetIpPort"/>
 /// <remarks>This function was added in <b>SA-MP 0.3c R4</b> and will not work in earlier versions!</remarks>
 /// <remarks>This function may not return accurate data when used under <a href="#OnPlayerDisconnect">OnPlayerDisconnect</a> if the player has quit normally. It usually returns accurate data if the player has been kicked or has timed out.</remarks>
-native GetPlayerNetworkStats(playerid, retstr[], retstr_size);
+native GetPlayerNetworkStats(playerid, retstr[], retstr_size = sizeof retstr);
 
 /// <summary>Gets the server's network stats and stores them in a string.</summary>
 /// <param name="retstr">The string to store the network stats in, passed by reference</param>
@@ -1262,7 +1262,7 @@ native GetPlayerNetworkStats(playerid, retstr[], retstr_size);
 ///   </c>
 /// </remarks>
 /// <returns>This function always returns <b><c>1</c></b>.</returns>
-native GetNetworkStats(retstr[], retstr_size);
+native GetNetworkStats(retstr[], retstr_size = sizeof retstr);
 
 /// <summary>Returns the SA-MP client version, as reported by the player.</summary>
 /// <param name="playerid">The ID of the player to get the client version of</param>
@@ -1274,7 +1274,7 @@ native GetNetworkStats(retstr[], retstr_size);
 /// <remarks>This function was added in <b>SA-MP 0.3e</b> and will not work in earlier versions!</remarks>
 /// <remarks>The string the version gets stored in will be empty if playerid is an NPC.</remarks>
 /// <returns><b><c>1</c></b> on success and <b><c>0</c></b> on failure (if player specified doesn't exist).</returns>
-native GetPlayerVersion(playerid, version[], len); // Returns the SA-MP client revision as reported by the player
+native GetPlayerVersion(playerid, version[], len = sizeof version); // Returns the SA-MP client revision as reported by the player
 
 /// <summary>Blocks an IP address from further communication with the server for a set amount of time (with wildcards allowed). Players trying to connect to the server with a blocked IP address will receive the generic "You are banned from this server." message. Players that are online on the specified IP before the block will timeout after a few seconds and, upon reconnect, will receive the same message.</summary>
 /// <param name="ip_address">The IP to block</param>
@@ -1306,7 +1306,7 @@ native UnBlockIpAddress(const ip_address[]);
 /// <remarks>Type 'varlist' in the server console to display a list of available server variables and their types.</remarks>
 /// <remarks>When filterscripts or plugins is specified as the varname, this function only returns the name of the first specified filterscript or plugin. This is a bug.</remarks>
 /// <returns>The length of the returned string. <b><c>0</c></b> if the specified server variable is not a string or doesn't exist.</returns>
-native GetServerVarAsString(const varname[], buffer[], len);
+native GetServerVarAsString(const varname[], buffer[], len = sizeof buffer);
 
 /// <summary>Get the integer value of a server variable.</summary>
 /// <param name="varname">The name of the integer variable to get the value of</param>
@@ -1337,7 +1337,7 @@ native GetServerVarAsBool(const varname[]);
 /// <remarks>When filterscripts or plugins are specified as the varname, this function only returns the name of the first specified filterscript or plugin.</remarks>
 /// <remarks>Using this function with anything other than a <b>string</b> (integer, boolean or float) will cause your server to crash. Using it with a nonexistent console variable will also cause your server to crash.</remarks>
 /// <returns>The length of the returned string. <b><c>0</c></b> if the specified console variable is not a string or doesn't exist.</returns>
-native GetConsoleVarAsString(const varname[], buffer[], len);
+native GetConsoleVarAsString(const varname[], buffer[], len = sizeof buffer);
 
 /// <summary>Get the integer value of a console variable.</summary>
 /// <param name="varname">The name of the integer variable to get the value of</param>
@@ -1528,7 +1528,7 @@ native NetStats_ConnectionStatus(playerid);
 /// <seealso name="NetStats_PacketLossPercent"/>
 /// <seealso name="NetStats_ConnectionStatus"/>
 /// <remarks>This function was added in <b>SA-MP 0.3z</b> and will not work in earlier versions!</remarks>
-native NetStats_GetIpPort(playerid, ip_port[], ip_port_len);
+native NetStats_GetIpPort(playerid, ip_port[], ip_port_len = sizeof ip_port);
 
 // Menu
 

a_sampdb.inc

@@ -70,7 +70,7 @@ native db_num_fields(DBResult:dbresult);
 /// <param name="maxlength">The max length of the field</param>
 /// <remarks>Using an <b>invalid handle</b> will crash your server! Get a <b>valid handle</b> by using <a href="#db_open">db_open</a>. But it's protected against <b><c>NULL</c></b> references</remarks>
 /// <returns>Returns <b><c>1</c></b>, if the function was successful, otherwise <b><c>0</c></b> if <b><c>DBResult:dbresult</c></b> is a <b><c>NULL</c></b> reference or the column index not available.</returns>
-native db_field_name(DBResult:dbresult, field, result[], maxlength);
+native db_field_name(DBResult:dbresult, field, result[], maxlength = sizeof result);
 
 /// <summary>Get the content of a field from <a href="#db_query">db_query</a>.</summary>
 /// <param name="dbresult">The result to get the data from</param>
@@ -79,7 +79,7 @@ native db_field_name(DBResult:dbresult, field, result[], maxlength);
 /// <param name="maxlength">The max length of the field</param>
 /// <remarks>Using an <b>invalid handle</b> will crash your server! Get a <b>valid handle</b> by using <a href="#db_open">db_open</a>. But it's protected against <b><c>NULL</c></b> references</remarks>
 /// <returns>Returns <b><c>1</c></b> if successful, otherwise <b><c>0</c></b> if <b><c>DBResult:dbresult</c></b> is a <b><c>NULL</c></b> reference or the column index not available.</returns>
-native db_get_field(DBResult:dbresult, field, result[], maxlength);
+native db_get_field(DBResult:dbresult, field, result[], maxlength = sizeof result);
 
 /// <summary>Get the content of a field as an integer from <a href="#db_query">db_query</a>.</summary>
 /// <param name="result">The result to get the data from</param>
@@ -102,7 +102,7 @@ native Float:db_get_field_float(DBResult:result, field = 0);
 /// <param name="maxlength">The max length of the field</param>
 /// <remarks>Using an <b>invalid handle</b> will crash your server! Get a <b>valid handle</b> by using <a href="#db_open">db_open</a>. But it's protected against <b><c>NULL</c></b> references</remarks>
 /// <returns>Returns <b><c>1</c></b> if successful, otherwise <b><c>0</c></b> if <b><c>DBResult:dbresult</c></b> is a <b><c>NULL</c></b> reference or the column index not available.</returns>
-native db_get_field_assoc(DBResult:dbresult, const field[], result[], maxlength);
+native db_get_field_assoc(DBResult:dbresult, const field[], result[], maxlength = sizeof result);
 
 /// <summary>Get the contents of field as an integer with specified name.</summary>
 /// <param name="result">The result to get the data from</param>