Changed Lua unsigned type check from error to warning

(This is just a temporary tolerance and will be reverted asap!)

Author: jushar

MTA10/mods/shared_logic/lua/CLuaFunctionDefs.h

@@ -32,6 +32,8 @@ class CRegisteredCommands;
 
 class CLuaFunctionDefs
 {
+    friend class CScriptArgReader;
+
 public:
     static void Initialize ( class CLuaManager* pLuaManager,
                               class CScriptDebugging* pScriptDebugging,

Shared/sdk/CScriptArgReader.h

@@ -9,8 +9,13 @@
 *  Multi Theft Auto is available from http://www.multitheftauto.com/
 *
 *****************************************************************************/
+#pragma once
 #include <limits>
 
+#ifdef MTA_CLIENT
+    #include "CScriptDebugging.h"
+#endif
+
 /////////////////////////////////////////////////////////////////////////
 //
 // CScriptArgReader
@@ -58,7 +63,7 @@ class CScriptArgReader
 
             if ( std::is_unsigned < T > () && number < -FLT_EPSILON )
             {
-                SetCustomError ( "Expected positive value, got negative", "Bad argument" );
+                SetCustomWarning ( "Expected positive value, got negative. This warning may be an error in future versions." );
                 return;
             }
 
@@ -90,7 +95,7 @@ class CScriptArgReader
 
             if ( std::is_unsigned < T > () && number < -FLT_EPSILON )
             {
-                SetCustomError ( "Expected positive value, got negative", "Bad argument" );
+                SetCustomWarning ( "Expected positive value, got negative. This warning may be an error in future versions." );
                 return;
             }
 
@@ -1121,11 +1126,24 @@ class CScriptArgReader
     //
     // HasErrors - Optional check if there are any unread arguments
     //
-    bool HasErrors ( bool bCheckUnusedArgs = false ) const
+    bool HasErrors ( bool bCheckUnusedArgs = false )
     {
         assert ( !IsReadFunctionPending () );
         if ( bCheckUnusedArgs && lua_type ( m_luaVM, m_iIndex ) != LUA_TNONE )
             return true;
+
+        // Output warning here (there's no better way to integrate it without huge code changes
+        if ( !m_bError && !m_strCustomWarning.empty () )
+        {
+            #ifdef MTA_CLIENT
+                CLuaFunctionDefs::m_pScriptDebugging->LogWarning ( m_luaVM, m_strCustomWarning );
+            #else
+                g_pGame->GetScriptDebugging ()->LogWarning ( m_luaVM, m_strCustomWarning );
+            #endif
+            
+            m_strCustomWarning.clear ();
+        }
+
         return m_bError;
     }
 
@@ -1216,6 +1234,14 @@ class CScriptArgReader
         return SString ( "%s @ '%s' [%s]", *m_strErrorCategory, lua_tostring ( m_luaVM, lua_upvalueindex ( 1 ) ), *GetErrorMessage () );
     }
 
+    //
+    // Set custom warning message
+    //
+    void SetCustomWarning ( const SString& message )
+    {
+        m_strCustomWarning = message;
+    }
+
     //
     // Skip n arguments
     //
@@ -1237,5 +1263,6 @@ class CScriptArgReader
     SString                 m_strErrorCategory;
     bool                    m_bHasCustomMessage;
     SString                 m_strCustomMessage;
+    SString                 m_strCustomWarning;
 
 };