Script updates and fixes

[ohlidalp]

Fixes #3195 - apparently I forgot to fix some scripts after I fixed the [X] button in 41060b8

I'm in the process of going through the scripts, making sure they work and they use all capabilities offered by the game. For example I added tabs to the road editor since those arrived in the meantime.

[CuriousMike56]

Tested all scripts in scripts/ directory except those used by the base game (base.as, default.as, AI.as, races.as)

The following scripts fail or throw warnings:

example_ogre_overlays.as

Guaranteed game crash almost every time. Log warnings:

example_ogre_overlays.as (30, 1): Info = Compiling void frameStep(float)
example_ogre_overlays.as (66, 73): Warning = The operand is implicitly converted to handle in order to compare them

---

example_terrn_raceConverter.as

Fails if no races are present on terrain, otherwise OK

addfunc (1, 42): Error = No matching symbol 'races'
An error occurred while trying to add a function ('void eventCallback(int key, int value) { races.eventCallback(key, value); }') to script module 'default.as(category:TERRAIN,unique ID:0)'.
Error in `getVariable()` - 'races' not found

---

example_autotest_scenario.as

Loads with warnings:

example_autotest_scenario.as (78, 1): Info = Compiling void frameStep(float)
example_autotest_scenario.as (145, 38): Warning = Signed/Unsigned mismatch
example_autotest_scenario.as (153, 31): Warning = Signed/Unsigned mismatch
Executing main() in example_autotest_scenario.as(category:CUSTOM,unique ID:1)
Automated test is running...
The script finished successfully.
Automated test step 0: Waiting for the game to fully load...

Doesn't advance past step 0, even with a vehicle spawned. Removing vehicle gives 'out of bounds' error:

Autotest: vehicle deleted, ID: 1
The script ended with exception; details below:
context.ExceptionLineNumber: 73
context.ExceptionString: Index out of bounds
context.ExceptionFunction.Declaration: void eventCallbackEx(scriptEvents, int, int, int, int, string, string, string, string)
context.ExceptionFunction.ModuleName: example_autotest_scenario.as(category:CUSTOM,unique ID:1)
context.ExceptionFunction.ScriptSectionName: example_autotest_scenario.as
context.ExceptionFunction.ObjectName: 

---

example_game_keyCodes.as

OK with warning:

example_game_keyCodes.as (24, 1): Info = Compiling void showKeyState(keyCodes, string)
example_game_keyCodes.as (34, 31): Warning = Signed/Unsigned mismatch

---

example_ogre_textureBlitting.as

example_ogre_textureBlitting.as (11, 1): Error = Identifier 'GridViewer' is not a data type in global namespace
example_ogre_textureBlitting.as (12, 1): Error = Identifier 'GridViewer' is not a data type in global namespace
example_ogre_textureBlitting.as (61, 24): Error = Identifier 'GridViewer' is not a data type in global namespace

---

example_ImGui_nodeHighlight.as

Spammed log with 'out of bounds' error first time, have yet to reproduce again (with or without vehicle spawned)

The script ended with exception; details below:
context.ExceptionLineNumber: 197
context.ExceptionString: Index out of bounds
context.ExceptionFunction.Declaration: void drawNodeHighlights(BeamClass@)
context.ExceptionFunction.ModuleName: example_ImGui_nodeHighlight.as(category:CUSTOM,unique ID:1)
context.ExceptionFunction.ScriptSectionName: example_ImGui_nodeHighlight.as
context.ExceptionFunction.ObjectName: 

---

All other scripts appear to load OK 👍

road_editor.as
demo_script.as
script_editor.as
overlay_editor.as
genericdoc_utils.as
gridviewer_utils.as
imgui_utils.as
scriptinfo_utils.as
example_terrn_railBuilder.as
example_actor_RigEditorAlpha.as
example_actor_shockTuning.as
example_angelscript_exceptionEvent.as
example_angelscript_getScriptDetails.as
example_angelscript_linecallback.as
example_game_camera.as
example_game_fetchUrlAsStringAsync.as
example_game_findResourceFileInfo.as
example_game_getManagedMaterials.as
example_game_getMousePointedMovableObjects.as
example_game_gridSpawn.as
example_game_setEventSimulatedValue.as
example_GenericDocument_uniEditor.as
example_html_parsing.as
example_ImGui_devDocLinksOffline.as
example_ImGui_hotkeyHighlight.as
example_ImGui_ImHyperlink.as
example_ImGui_tabs.as
example_ogre_characterPosing(skelAnims).as
example_ogre_inspector.as
example_ogre_ManualObject.as
example_ogre_meshedConcrete.as
example_ogre_shaderParams.as
example_ogre_terrnBatcher.as
example_ogre_vertexData.as
example_terrn_odefBrowser.as

[ohlidalp]

@CuriousMike56 Thanks for such an elaborate test. Everything should be fixed now.

Please also test if races/AI work normally because I've done changes across the entire scripting subsystem - I had to unify return codes of functions because it was an undocumented mess:

~ executeString() [script: not exported] - used to return 0 on success, 1 on internal error and negative number otherwise.
~ addFunction() [script: `game.addScriptFunction()`] - used to return 0 on success, 1 on internal error and negative number otherwise.
~ functionExists() [script: `game.scriptFunctionExists()`] - used to return function ID (always >0) on success, negative number on error.
~ deleteFunction() [script: `game.deleteScriptFunction()`] - used to return function ID (always >0) on success, negative number on error.
~ addVariable() [script: `game.addScriptVariable()` ] - used to return 0 on success, 1 on internal error and negative number otherwise.
~ variableExists() [script: `game.scriptVariableExists()`] - newly added, returns 0 on success  and negative number otherwise.
~ deleteVariable() [script: `game.deleteScriptVariable()` ] - used to return 0 on success, 1 on internal error and negative number otherwise.
~ getVariable() [script: `game.getScriptVariable()` ] - recently added, returns 0 on success  and negative number otherwise.

[CuriousMike56]

Well loading all scripts at once works now lol (previously crashed) 👍

Races and AI seem to be functioning as expected too ✅