Fix all warnings and errors in C++

.clang-format

@@ -0,0 +1,14 @@
+BasedOnStyle: LLVM
+SortIncludes: CaseSensitive
+IncludeBlocks: Preserve
+IncludeCategories:
+  - Regex: '<Windows\.h>'
+    Priority: 1
+  - Regex: '<Psapi\.h>'
+    Priority: 2
+  - Regex: '"BlHooks\\.hpp"'
+    Priority: 3
+  - Regex: '"BlFuncs\\.hpp"'
+    Priority: 4
+  - Regex: ".*"
+    Priority: 5

.editorconfig

@@ -0,0 +1,9 @@
+root = true
+
+[*]
+indent_style = space
+indent_size = 2
+end_of_line = lf
+charset = utf-8
+trim_trailing_whitespace = true
+insert_final_newline = true

.gitignore

@@ -1,2 +1,2 @@
-.*
-!.gitignore
+build/
+.cache/

.vscode/settings.json

@@ -0,0 +1,11 @@
+{
+  "Lua.diagnostics.globals": [
+    "_bllua_ts",
+    "_bllua_requiresecure",
+    "_bllua_on_unload"
+  ],
+  "Lua.runtime.version": "Lua 5.1",
+  "Lua.diagnostics.disable": ["lowercase-global", "undefined-global"],
+  "C_Cpp.default.compileCommands": "${workspaceFolder}/build/compile_commands.json",
+  "C_Cpp.default.compilerPath": "C:/msys64/mingw32/bin/g++.exe"
+}

CMakeLists.txt

@@ -0,0 +1,56 @@
+cmake_minimum_required(VERSION 3.10)
+
+project(BlockLua CXX)
+
+# Export compile_commands.json for VSCode IntelliSense
+set(CMAKE_EXPORT_COMPILE_COMMANDS ON)
+
+# Output directories to mirror compile.bat's build folder
+set(OUTPUT_DIR ${CMAKE_SOURCE_DIR}/build)
+set(CMAKE_RUNTIME_OUTPUT_DIRECTORY ${OUTPUT_DIR})
+set(CMAKE_LIBRARY_OUTPUT_DIRECTORY ${OUTPUT_DIR})
+set(CMAKE_ARCHIVE_OUTPUT_DIRECTORY ${OUTPUT_DIR})
+
+# Global compile options to mirror compile.bat
+add_compile_options(
+  -Wall
+  -Werror
+  -m32
+  -static-libgcc
+  -static-libstdc++
+)
+
+# Include paths
+include_directories(
+  ${CMAKE_SOURCE_DIR}/src
+  ${CMAKE_SOURCE_DIR}/inc/tsfuncs
+  ${CMAKE_SOURCE_DIR}/inc/lua
+)
+
+# Link directories (for -L.) and libraries from compile.bat
+link_directories(
+  ${CMAKE_SOURCE_DIR}
+)
+
+# Safe DLL
+add_library(BlockLua SHARED src/bllua4.cpp)
+# Ensure output name matches compile.bat
+set_target_properties(BlockLua PROPERTIES OUTPUT_NAME "BlockLua")
+# Linker flags and libraries
+if(MSVC)
+  # Not expected with mingw, but keep placeholder
+else()
+  target_link_libraries(BlockLua PRIVATE psapi lua5.1)
+endif()
+
+# Unsafe DLL (with BLLUA_UNSAFE definition)
+add_library(BlockLuaUnsafe SHARED src/bllua4.cpp)
+set_target_properties(BlockLuaUnsafe PROPERTIES OUTPUT_NAME "BlockLua-Unsafe")
+
+target_compile_definitions(BlockLuaUnsafe PRIVATE BLLUA_UNSAFE)
+
+if(MSVC)
+  # Not expected with mingw, but keep placeholder
+else()
+  target_link_libraries(BlockLuaUnsafe PRIVATE psapi lua5.1)
+endif()

compile.bat

@@ -0,0 +1,13 @@
+@echo off
+cd /d %~dp0
+
+set "PATH=C:\msys64\mingw32\bin;%PATH%"
+
+if not exist build mkdir build
+
+set buildargs=-Wall -Werror -m32 -shared -Isrc -Iinc/tsfuncs -Iinc/lua -lpsapi -L. -llua5.1 -static-libgcc -static-libstdc++
+
+echo on
+g++ src/bllua4.cpp %buildargs% -o build\BlockLua.dll
+g++ -DBLLUA_UNSAFE src/bllua4.cpp %buildargs% -o build\BlockLua-Unsafe.dll
+@echo off

compiling.md

@@ -0,0 +1,52 @@
+## Compiling on Windows with MSYS2 (32-bit)
+
+Follow these steps to build `BlockLua.dll` for 32-bit Windows using MSYS2's MinGW-w64 i686 toolchain.
+
+### 1) Install MSYS2
+
+- Download and install MSYS2 from `https://www.msys2.org/`.
+- After installing, open the "MSYS2 MSYS" terminal once and update the package database if prompted.
+
+### 2) Install required packages (i686 / 32-bit)
+
+Run this in the "MSYS2 MSYS" or "MSYS2 MinGW 32-bit" terminal:
+
+```bash
+pacman -Sy --needed mingw-w64-i686-toolchain mingw-w64-i686-binutils mingw-w64-i686-lua51
+```
+
+What these packages are for:
+
+- mingw-w64-i686-toolchain: 32-bit C/C++ compiler suite (g++, libstdc++, runtime libs) to build Windows binaries.
+- mingw-w64-i686-binutils: Linker and binary utilities (ld, as, objdump) used by the compiler and for optional inspection.
+- mingw-w64-i686-lua51: Lua 5.1 development files for 32-bit (import library). We use its import library to link; at runtime you can use the provided `lua5.1.dll`.
+
+### 3) Build (PowerShell, recommended)
+
+- Open PowerShell in the repo root.
+- Run the script:
+
+```powershell
+compile.bat
+```
+
+What the script does:
+
+- Temporarily prepends `C:\\msys64\\mingw32\\bin` to PATH so the 32-bit toolchain is used.
+- Compiles the project with `-m32` and links against `lua5.1`.
+- Produces `build\BlockLua.dll` and `build\BlockLua-Unsafe.dll`.
+
+### 4) Optional: Verify 32-bit output
+
+If you installed binutils, you can check the architecture:
+
+```powershell
+objdump -f build\BlockLua.dll | Select-String i386
+```
+
+You should see `architecture: i386` in the output.
+
+### Notes
+
+- Ensure you installed the i686 (32-bit) variants of the packages; x86_64 packages won’t work for a 32-bit build.
+- If the linker cannot find `-llua5.1`, confirm `mingw-w64-i686-lua51` is installed and you are using the `mingw32` toolchain (not `x86_64`).

inc/lua/lauxlib.h

@@ -4,17 +4,14 @@
 ** See Copyright Notice in lua.h
 */
 
-
 #ifndef lauxlib_h
 #define lauxlib_h
 
-
 #include <stddef.h>
 #include <stdio.h>
 
 #include "lua.h"
 
-
 /* extra error code for `luaL_load' */
 #define LUA_ERRFILE (LUA_ERRERR + 1)
 
@@ -69,7 +66,6 @@ LUALIB_API int (luaL_loadstring) (lua_State *L, const char *s);
 
 LUALIB_API lua_State *(luaL_newstate)(void);
 
-
 LUALIB_API const char *(luaL_gsub)(lua_State * L, const char *s, const char *p,
                                    const char *r);
 
@@ -91,7 +87,6 @@ LUALIB_API void (luaL_pushmodule) (lua_State *L, const char *modname,
 LUALIB_API void *(luaL_testudata)(lua_State * L, int ud, const char *tname);
 LUALIB_API void(luaL_setmetatable)(lua_State *L, const char *tname);
 
-
 /*
 ** ===============================================================
 ** some useful macros
@@ -130,8 +125,6 @@ LUALIB_API void (luaL_setmetatable) (lua_State *L, const char *tname);
 ** =======================================================
 */
 
-
-
 typedef struct luaL_Buffer {
   char *p; /* current position in buffer */
   int lvl; /* number of strings in the stack (level) */
@@ -155,7 +148,6 @@ LUALIB_API void (luaL_addstring) (luaL_Buffer *B, const char *s);
 LUALIB_API void(luaL_addvalue)(luaL_Buffer *B);
 LUALIB_API void(luaL_pushresult)(luaL_Buffer *B);
 
-
 /* }====================================================== */
 
 #endif

inc/lua/lua.h

@@ -5,31 +5,26 @@
 ** See Copyright Notice at the end of this file
 */
 
-
 #ifndef lua_h
 #define lua_h
 
 #include <stdarg.h>
 #include <stddef.h>
 
-
 #include "luaconf.h"
 
-
 #define LUA_VERSION "Lua 5.1"
 #define LUA_RELEASE "Lua 5.1.4"
 #define LUA_VERSION_NUM 501
 #define LUA_COPYRIGHT "Copyright (C) 1994-2008 Lua.org, PUC-Rio"
 #define LUA_AUTHORS "R. Ierusalimschy, L. H. de Figueiredo & W. Celes"
 
-
 /* mark for precompiled code (`<esc>Lua') */
 #define LUA_SIGNATURE "\033Lua"
 
 /* option for multiple returns in `lua_pcall' and `lua_call' */
 #define LUA_MULTRET (-1)
 
-
 /*
 ** pseudo-indices
 */
@@ -38,7 +33,6 @@
 #define LUA_GLOBALSINDEX (-10002)
 #define lua_upvalueindex(i) (LUA_GLOBALSINDEX - (i))
 
-
 /* thread status */
 #define LUA_OK 0
 #define LUA_YIELD 1
@@ -47,12 +41,10 @@
 #define LUA_ERRMEM 4
 #define LUA_ERRERR 5
 
-
 typedef struct lua_State lua_State;
 
 typedef int (*lua_CFunction)(lua_State *L);
 
-
 /*
 ** functions that read/write blocks when loading/dumping Lua chunks
 */
@@ -60,13 +52,11 @@ typedef const char * (*lua_Reader) (lua_State *L, void *ud, size_t *sz);
 
 typedef int (*lua_Writer)(lua_State *L, const void *p, size_t sz, void *ud);
 
-
 /*
 ** prototype for memory-allocation functions
 */
 typedef void *(*lua_Alloc)(void *ud, void *ptr, size_t osize, size_t nsize);
 
-
 /*
 ** basic types
 */
@@ -82,12 +72,9 @@ typedef void * (*lua_Alloc) (void *ud, void *ptr, size_t osize, size_t nsize);
 #define LUA_TUSERDATA 7
 #define LUA_TTHREAD 8
 
-
-
 /* minimum Lua stack available to a C function */
 #define LUA_MINSTACK 20
 
-
 /*
 ** generic extra include file
 */
@@ -95,16 +82,12 @@ typedef void * (*lua_Alloc) (void *ud, void *ptr, size_t osize, size_t nsize);
 #include LUA_USER_H
 #endif
 
-
 /* type of numbers in Lua */
 typedef LUA_NUMBER lua_Number;
 
-
 /* type for integer functions */
 typedef LUA_INTEGER lua_Integer;
 
-
-
 /*
 ** state manipulation
 */
@@ -114,7 +97,6 @@ LUA_API lua_State *(lua_newthread) (lua_State *L);
 
 LUA_API lua_CFunction(lua_atpanic)(lua_State *L, lua_CFunction panicf);
 
-
 /*
 ** basic stack manipulation
 */
@@ -128,7 +110,6 @@ LUA_API int   (lua_checkstack) (lua_State *L, int sz);
 
 LUA_API void(lua_xmove)(lua_State *from, lua_State *to, int n);
 
-
 /*
 ** access functions (stack -> C)
 */
@@ -154,7 +135,6 @@ LUA_API void	       *(lua_touserdata) (lua_State *L, int idx);
 LUA_API lua_State *(lua_tothread)(lua_State * L, int idx);
 LUA_API const void *(lua_topointer)(lua_State * L, int idx);
 
-
 /*
 ** push functions (C -> stack)
 */
@@ -171,7 +151,6 @@ LUA_API void  (lua_pushboolean) (lua_State *L, int b);
 LUA_API void(lua_pushlightuserdata)(lua_State *L, void *p);
 LUA_API int(lua_pushthread)(lua_State *L);
 
-
 /*
 ** get functions (Lua -> stack)
 */
@@ -184,7 +163,6 @@ LUA_API void *(lua_newuserdata) (lua_State *L, size_t sz);
 LUA_API int(lua_getmetatable)(lua_State *L, int objindex);
 LUA_API void(lua_getfenv)(lua_State *L, int idx);
 
-
 /*
 ** set functions (stack -> Lua)
 */
@@ -195,7 +173,6 @@ LUA_API void  (lua_rawseti) (lua_State *L, int idx, int n);
 LUA_API int(lua_setmetatable)(lua_State *L, int objindex);
 LUA_API int(lua_setfenv)(lua_State *L, int idx);
 
-
 /*
 ** `load' and `call' functions (load and run Lua code)
 */
@@ -207,7 +184,6 @@ LUA_API int   (lua_load) (lua_State *L, lua_Reader reader, void *dt,
 
 LUA_API int(lua_dump)(lua_State *L, lua_Writer writer, void *data);
 
-
 /*
 ** coroutine functions
 */
@@ -231,7 +207,6 @@ LUA_API int  (lua_status) (lua_State *L);
 
 LUA_API int(lua_gc)(lua_State *L, int what, int data);
 
-
 /*
 ** miscellaneous functions
 */
@@ -245,8 +220,6 @@ LUA_API void  (lua_concat) (lua_State *L, int n);
 LUA_API lua_Alloc(lua_getallocf)(lua_State *L, void **ud);
 LUA_API void lua_setallocf(lua_State *L, lua_Alloc f, void *ud);
 
-
-
 /*
 ** ===============================================================
 ** some useful macros
@@ -280,8 +253,6 @@ LUA_API void lua_setallocf (lua_State *L, lua_Alloc f, void *ud);
 
 #define lua_tostring(L, i) lua_tolstring(L, (i), NULL)
 
-
-
 /*
 ** compatibility macros and functions
 */
@@ -295,18 +266,15 @@ LUA_API void lua_setallocf (lua_State *L, lua_Alloc f, void *ud);
 #define lua_Chunkreader lua_Reader
 #define lua_Chunkwriter lua_Writer
 
-
 /* hack */
 LUA_API void lua_setlevel(lua_State *from, lua_State *to);
 
-
 /*
 ** {======================================================================
 ** Debug API
 ** =======================================================================
 */
 
-
 /*
 ** Event codes
 */
@@ -316,7 +284,6 @@ LUA_API void lua_setlevel	(lua_State *from, lua_State *to);
 #define LUA_HOOKCOUNT 3
 #define LUA_HOOKTAILRET 4
 
-
 /*
 ** Event masks
 */
@@ -327,11 +294,9 @@ LUA_API void lua_setlevel	(lua_State *from, lua_State *to);
 
 typedef struct lua_Debug lua_Debug; /* activation record */
 
-
 /* Functions to be called by the debuger in specific events */
 typedef void (*lua_Hook)(lua_State *L, lua_Debug *ar);
 
-
 LUA_API int lua_getstack(lua_State *L, int level, lua_Debug *ar);
 LUA_API int lua_getinfo(lua_State *L, const char *what, lua_Debug *ar);
 LUA_API const char *lua_getlocal(lua_State *L, const lua_Debug *ar, int n);
@@ -356,7 +321,6 @@ LUA_API lua_Integer lua_tointegerx (lua_State *L, int idx, int *isnum);
 /* From Lua 5.3. */
 LUA_API int lua_isyieldable(lua_State *L);
 
-
 struct lua_Debug {
   int event;
   const char *name;           /* (n) */
@@ -374,7 +338,6 @@ struct lua_Debug {
 
 /* }====================================================================== */
 
-
 /******************************************************************************
  * Copyright (C) 1994-2008 Lua.org, PUC-Rio.  All rights reserved.
  *
@@ -398,5 +361,4 @@ struct lua_Debug {
  * SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
  ******************************************************************************/
 
-
 #endif

inc/lua/lua.hpp

@@ -1,9 +1,8 @@
 // C++ wrapper for LuaJIT header files.
 
 extern "C" {
-#include "lua.h"
 #include "lauxlib.h"
-#include "lualib.h"
+#include "lua.h"
 #include "luajit.h"
+#include "lualib.h"
 }
-

inc/lua/luaconf.h

@@ -20,10 +20,8 @@
 */
 #define LUA_LDIR "!\\lua\\"
 #define LUA_CDIR "!\\"
-#define LUA_PATH_DEFAULT \
-  ".\\?.lua;" LUA_LDIR"?.lua;" LUA_LDIR"?\\init.lua;"
-#define LUA_CPATH_DEFAULT \
-  ".\\?.dll;" LUA_CDIR"?.dll;" LUA_CDIR"loadall.dll"
+#define LUA_PATH_DEFAULT ".\\?.lua;" LUA_LDIR "?.lua;" LUA_LDIR "?\\init.lua;"
+#define LUA_CPATH_DEFAULT ".\\?.dll;" LUA_CDIR "?.dll;" LUA_CDIR "loadall.dll"
 #else
 /*
 ** Note to distribution maintainers: do NOT patch the following lines!
@@ -78,8 +76,8 @@
 #define LUA_EXECDIR "!"
 #define LUA_IGMARK "-"
 #define LUA_PATH_CONFIG                                                        \
-  LUA_DIRSEP "\n" LUA_PATHSEP "\n" LUA_PATH_MARK "\n" \
-  LUA_EXECDIR "\n" LUA_IGMARK "\n"
+  LUA_DIRSEP "\n" LUA_PATHSEP "\n" LUA_PATH_MARK "\n" LUA_EXECDIR              \
+             "\n" LUA_IGMARK "\n"
 
 /* Quoting in error messages. */
 #define LUA_QL(x) "'" x "'"
@@ -144,9 +142,16 @@
 #define lua_assert(x) assert(x)
 #endif
 #ifdef LUA_USE_APICHECK
-#define luai_apicheck(L, o)	{ (void)L; assert(o); }
+#define luai_apicheck(L, o)                                                    \
+  {                                                                            \
+    (void)L;                                                                   \
+    assert(o);                                                                 \
+  }
 #else
-#define luai_apicheck(L, o)	{ (void)L; }
+#define luai_apicheck(L, o)                                                    \
+  {                                                                            \
+    (void)L;                                                                   \
+  }
 #endif
 
 #endif

inc/lua/luajit.h

@@ -65,8 +65,8 @@ enum {
 LUA_API int luaJIT_setmode(lua_State *L, int idx, int mode);
 
 /* Low-overhead profiling API. */
-typedef void (*luaJIT_profile_callback)(void *data, lua_State *L,
-					int samples, int vmstate);
+typedef void (*luaJIT_profile_callback)(void *data, lua_State *L, int samples,
+                                        int vmstate);
 LUA_API void luaJIT_profile_start(lua_State *L, const char *mode,
                                   luaJIT_profile_callback cb, void *data);
 LUA_API void luaJIT_profile_stop(lua_State *L);

inc/tsfuncs/BlFuncs.cpp

@@ -2,15 +2,18 @@
 //////////////////////////////////////////////////
 // BlFuncs Version 1.0
 
-
 // Includes
-
-#include "BlHooks.hpp"
+#ifndef WIN32_LEAN_AND_MEAN
+#define WIN32_LEAN_AND_MEAN
+#endif
 #include "BlFuncs.hpp"
+#include <Windows.h>
+#include "BlHooks.hpp"
 
+#include <cstdarg>
+#include <cstring>
 #include <stdio.h>
 
-
 // Scanned structures
 
 ADDR tsf_mCacheSequence;
@@ -33,7 +36,6 @@ BlFunctionDefIntern(tsf_BlSimObject__getDataField    );
 BlFunctionDefIntern(tsf_BlSimObject__setDataField);
 BlFunctionDefIntern(tsf_BlCon__getReturnBuffer);
 
-
 // C->TS Args
 
 char *tsf_GetIntArg(signed int value) {
@@ -56,7 +58,6 @@ char* tsf_GetThisArg(ADDR obj) {
   return tsf_GetIntArg(*(signed int *)(obj + 32));
 }
 
-
 // Eval
 
 const char *tsf_Eval(const char *code) {
@@ -74,16 +75,17 @@ const char* tsf_Evalf(const char *fmt, ...) {
   return tsf_Eval((const char *)code);
 }
 
-
 // Objects
 
 ADDR tsf_FindObject(unsigned int id) {
   ADDR obj = *(ADDR *)(*(ADDR *)(tsf_gIdDictionary) + 4 * (id & 0xFFF));
-	if(!obj) return 0;
+  if (!obj)
+    return 0;
 
   while (obj && *(unsigned int *)(obj + 32) != id) {
     obj = *(ADDR *)(obj + 16);
-		if(!obj) return 0;
+    if (!obj)
+      return 0;
   }
 
   return obj;
@@ -112,10 +114,10 @@ ADDR tsf_LookupNamespace(const char* ns) {
   return tsf_LookupNamespace(ns, nullptr);
 }
 
-
 // Object Fields
 
-const char* tsf_GetDataField(ADDR simObject, const char* slotName, const char* array) {
+const char *tsf_GetDataField(ADDR simObject, const char *slotName,
+                             const char *array) {
   const char *ste_slotName;
   if (slotName) {
     ste_slotName = tsf_BlStringTable__insert(slotName, 0);
@@ -126,7 +128,8 @@ const char* tsf_GetDataField(ADDR simObject, const char* slotName, const char* a
   return tsf_BlSimObject__getDataField(simObject, ste_slotName, array);
 }
 
-void tsf_SetDataField(ADDR simObject, const char* slotName, const char* array, const char* value) {
+void tsf_SetDataField(ADDR simObject, const char *slotName, const char *array,
+                      const char *value) {
   const char *ste_slotName;
   if (slotName) {
     ste_slotName = tsf_BlStringTable__insert(slotName, 0);
@@ -137,7 +140,6 @@ void tsf_SetDataField(ADDR simObject, const char* slotName, const char* array, c
   tsf_BlSimObject__setDataField(simObject, ste_slotName, array, value);
 }
 
-
 // TS Global Variables
 
 const char *tsf_GetVar(const char *name) {
@@ -145,7 +147,8 @@ const char *tsf_GetVar(const char* name) {
 }
 
 void tsf_AddVarInternal(const char *name, signed int varType, void *data) {
-	tsf_BlDictionary__addVariable((ADDR *)tsf_gEvalState_globalVars, name, varType, data);
+  tsf_BlDictionary__addVariable((ADDR *)tsf_gEvalState_globalVars, name,
+                                varType, data);
 }
 
 void tsf_AddVar(const char *name, const char **data) {
@@ -161,10 +164,12 @@ void tsf_AddVar(const char* name, bool* data) {
   tsf_AddVarInternal(name, 6, data);
 }
 
-
 // TS->C Functions
 
-ADDR tsf_AddConsoleFuncInternal(const char* pname, const char* cname, const char* fname, signed int cbtype, const char* usage, signed int mina, signed int maxa) {
+ADDR tsf_AddConsoleFuncInternal(const char *pname, const char *cname,
+                                const char *fname, signed int cbtype,
+                                const char *usage, signed int mina,
+                                signed int maxa) {
   const char *ste_fname = tsf_BlStringTable__insert(fname, 0);
   ADDR ns = tsf_LookupNamespace(cname, pname);
   ADDR ent = tsf_BlNamespace__createLocalEntry(ns, ste_fname);
@@ -180,50 +185,92 @@ ADDR tsf_AddConsoleFuncInternal(const char* pname, const char* cname, const char
   return ent;
 }
 
-void tsf_AddConsoleFunc(const char* pname, const char* cname, const char* fname, tsf_StringCallback sc, const char* usage, signed int mina, signed int maxa) {
-	ADDR ent = tsf_AddConsoleFuncInternal(pname, cname, fname, 1, usage, mina, maxa);
+void tsf_AddConsoleFunc(const char *pname, const char *cname, const char *fname,
+                        tsf_StringCallback sc, const char *usage,
+                        signed int mina, signed int maxa) {
+  ADDR ent =
+      tsf_AddConsoleFuncInternal(pname, cname, fname, 1, usage, mina, maxa);
   *(tsf_StringCallback *)(ent + 40) = sc;
 }
-void tsf_AddConsoleFunc(const char* pname, const char* cname, const char* fname, tsf_IntCallback    ic, const char*  usage, signed int mina, signed int maxa) {
-	ADDR ent = tsf_AddConsoleFuncInternal(pname, cname, fname, 2, usage, mina, maxa);
+void tsf_AddConsoleFunc(const char *pname, const char *cname, const char *fname,
+                        tsf_IntCallback ic, const char *usage, signed int mina,
+                        signed int maxa) {
+  ADDR ent =
+      tsf_AddConsoleFuncInternal(pname, cname, fname, 2, usage, mina, maxa);
   *(tsf_IntCallback *)(ent + 40) = ic;
 }
-void tsf_AddConsoleFunc(const char* pname, const char* cname, const char* fname, tsf_FloatCallback  fc, const char*  usage, signed int mina, signed int maxa) {
-	ADDR ent = tsf_AddConsoleFuncInternal(pname, cname, fname, 3, usage, mina, maxa);
+void tsf_AddConsoleFunc(const char *pname, const char *cname, const char *fname,
+                        tsf_FloatCallback fc, const char *usage,
+                        signed int mina, signed int maxa) {
+  ADDR ent =
+      tsf_AddConsoleFuncInternal(pname, cname, fname, 3, usage, mina, maxa);
   *(tsf_FloatCallback *)(ent + 40) = fc;
 }
-void tsf_AddConsoleFunc(const char* pname, const char* cname, const char* fname, tsf_VoidCallback   vc, const char*  usage, signed int mina, signed int maxa) {
-	ADDR ent = tsf_AddConsoleFuncInternal(pname, cname, fname, 4, usage, mina, maxa);
+void tsf_AddConsoleFunc(const char *pname, const char *cname, const char *fname,
+                        tsf_VoidCallback vc, const char *usage, signed int mina,
+                        signed int maxa) {
+  ADDR ent =
+      tsf_AddConsoleFuncInternal(pname, cname, fname, 4, usage, mina, maxa);
   *(tsf_VoidCallback *)(ent + 40) = vc;
 }
-void tsf_AddConsoleFunc(const char* pname, const char* cname, const char* fname, tsf_BoolCallback   bc, const char*  usage, signed int mina, signed int maxa) {
-	ADDR ent = tsf_AddConsoleFuncInternal(pname, cname, fname, 5, usage, mina, maxa);
+void tsf_AddConsoleFunc(const char *pname, const char *cname, const char *fname,
+                        tsf_BoolCallback bc, const char *usage, signed int mina,
+                        signed int maxa) {
+  ADDR ent =
+      tsf_AddConsoleFuncInternal(pname, cname, fname, 5, usage, mina, maxa);
   *(tsf_BoolCallback *)(ent + 40) = bc;
 }
 
-
 // Initialization
 
 bool tsf_InitInternal() {
   BlScanFunctionText(tsf_BlStringTable__insert, "83 EC 0C 80 3D ? ? ? ? ?");
-	BlScanFunctionText(tsf_BlNamespace__find            , "55 8B EC 6A FF 68 ? ? ? ? 64 A1 ? ? ? ? 50 83 EC 0C 53 56 57 A1 ? ? ? ? 33 C5 50 8D 45 F4 64 A3 ? ? ? ? 8B DA 8B D1"              );
-	BlScanFunctionText(tsf_BlNamespace__createLocalEntry, "55 8B EC 6A FF 68 ? ? ? ? 64 A1 ? ? ? ? 50 83 EC 08 53 56 57 A1 ? ? ? ? 33 C5 50 8D 45 F4 64 A3 ? ? ? ? 89 4D F0"                 );
-	BlScanFunctionText(tsf_BlDataChunker__freeBlocks    , "55 8B EC 6A FF 68 ? ? ? ? 64 A1 ? ? ? ? 50 51 53 56 57 A1 ? ? ? ? 33 C5 50 8D 45 F4 64 A3 ? ? ? ? 8B D9 8B 33"                    );
-	BlScanFunctionText(tsf_BlCon__evaluate              , "55 8B EC 6A FF 68 ? ? ? ? 64 A1 ? ? ? ? 50 56 57 A1 ? ? ? ? 33 C5 50 8D 45 F4 64 A3 ? ? ? ? 8B 75 10"                             );
-	BlScanFunctionText(tsf_BlCon__executef              , "81 EC ? ? ? ? A1 ? ? ? ? 33 C4 89 84 24 ? ? ? ? 53 55 56 8B B4 24 ? ? ? ? 33 C9"                                                  );
-	BlScanFunctionText(tsf_BlCon__executefSimObj        , "81 EC ? ? ? ? A1 ? ? ? ? 33 C4 89 84 24 ? ? ? ? 53 56 8B B4 24 ? ? ? ? 33 C9"                                                     );
-	BlScanFunctionText(tsf_BlCon__getVariable           , "53 56 8B F1 57 85 F6 0F 84 ? ? ? ?"                                                                                               );
+  BlScanFunctionText(
+      tsf_BlNamespace__find,
+      "55 8B EC 6A FF 68 ? ? ? ? 64 A1 ? ? ? ? 50 83 EC 0C 53 56 57 A1 ? ? ? ? "
+      "33 C5 50 8D 45 F4 64 A3 ? ? ? ? 8B DA 8B D1");
+  BlScanFunctionText(
+      tsf_BlNamespace__createLocalEntry,
+      "55 8B EC 6A FF 68 ? ? ? ? 64 A1 ? ? ? ? 50 83 EC 08 53 56 57 A1 ? ? ? ? "
+      "33 C5 50 8D 45 F4 64 A3 ? ? ? ? 89 4D F0");
+  BlScanFunctionText(tsf_BlDataChunker__freeBlocks,
+                     "55 8B EC 6A FF 68 ? ? ? ? 64 A1 ? ? ? ? 50 51 53 56 57 "
+                     "A1 ? ? ? ? 33 C5 50 8D 45 F4 64 A3 ? ? ? ? 8B D9 8B 33");
+  BlScanFunctionText(tsf_BlCon__evaluate,
+                     "55 8B EC 6A FF 68 ? ? ? ? 64 A1 ? ? ? ? 50 56 57 A1 ? ? "
+                     "? ? 33 C5 50 8D 45 F4 64 A3 ? ? ? ? 8B 75 10");
+  BlScanFunctionText(tsf_BlCon__executef,
+                     "81 EC ? ? ? ? A1 ? ? ? ? 33 C4 89 84 24 ? ? ? ? 53 55 56 "
+                     "8B B4 24 ? ? ? ? 33 C9");
+  BlScanFunctionText(tsf_BlCon__executefSimObj,
+                     "81 EC ? ? ? ? A1 ? ? ? ? 33 C4 89 84 24 ? ? ? ? 53 56 8B "
+                     "B4 24 ? ? ? ? 33 C9");
+  BlScanFunctionText(tsf_BlCon__getVariable,
+                     "53 56 8B F1 57 85 F6 0F 84 ? ? ? ?");
   BlScanFunctionText(tsf_BlDictionary__addVariable, "8B 44 24 04 56 57 8B F9");
   BlScanFunctionText(tsf_BlSim__findObject_name, "57 8B F9 8A 17");
-	BlScanFunctionText(tsf_BlStringStack__getArgBuffer  , "55 8B EC 83 E4 F8 8B 0D ? ? ? ? A1 ? ? ? ? 56 57 8B 7D 08 8D 14 01 03 D7 3B 15 ? ? ? ? 72 2C 8B 0D"                               );
-	BlScanFunctionText(tsf_BlSimObject__getDataField    , "51 53 8B D9 55 56 8B 74 24 14"                                                                                                    );
-	BlScanFunctionText(tsf_BlSimObject__setDataField    , "81 EC ? ? ? ? A1 ? ? ? ? 33 C4 89 84 24 ? ? ? ? 8B 84 24 ? ? ? ? 53 8B D9 89 44 24 04"                                            );
+  BlScanFunctionText(tsf_BlStringStack__getArgBuffer,
+                     "55 8B EC 83 E4 F8 8B 0D ? ? ? ? A1 ? ? ? ? 56 57 8B 7D "
+                     "08 8D 14 01 03 D7 3B 15 ? ? ? ? 72 2C 8B 0D");
+  BlScanFunctionText(tsf_BlSimObject__getDataField,
+                     "51 53 8B D9 55 56 8B 74 24 14");
+  BlScanFunctionText(tsf_BlSimObject__setDataField,
+                     "81 EC ? ? ? ? A1 ? ? ? ? 33 C4 89 84 24 ? ? ? ? 8B 84 24 "
+                     "? ? ? ? 53 8B D9 89 44 24 04");
   BlScanFunctionText(tsf_BlCon__getReturnBuffer, "81 F9 ? ? ? ? 76 2B");
 
-	ADDR BlScanText   (tsf_mCacheSequenceLoc            , "FF 05 ? ? ? ? B9 ? ? ? ? 8B F8 E8 ? ? ? ? 8B 44 24 1C 89 47 18 8B 44 24 14"                                                       );
-	ADDR BlScanText   (tsf_mCacheAllocatorLoc           , "89 35 ? ? ? ? C7 06 ? ? ? ? A1 ? ? ? ? 68 ? ? ? ? C7 40 ? ? ? ? ? E8 ? ? ? ? 83 C4 04 8B 4D F4 64 89 0D ? ? ? ? 59 5E 8B E5 5D C3");
-	ADDR BlScanText   (tsf_gIdDictionaryLoc             , "89 15 ? ? ? ? E8 ? ? ? ? 8B F0 89 75 F0"                                                                                          );
-	ADDR BlScanText   (tsf_gEvalState_globalVarsLoc     , "B9 ? ? ? ? E8 ? ? ? ? 68 ? ? ? ? 6A 0A 68 ? ? ? ? B9 ? ? ? ? E8 ? ? ? ? E8 ? ? ? ?"                                               );
+  ADDR BlScanText(tsf_mCacheSequenceLoc,
+                  "FF 05 ? ? ? ? B9 ? ? ? ? 8B F8 E8 ? ? ? ? 8B 44 24 1C 89 47 "
+                  "18 8B 44 24 14");
+  ADDR BlScanText(
+      tsf_mCacheAllocatorLoc,
+      "89 35 ? ? ? ? C7 06 ? ? ? ? A1 ? ? ? ? 68 ? ? ? ? C7 40 ? ? ? ? ? E8 ? "
+      "? ? ? 83 C4 04 8B 4D F4 64 89 0D ? ? ? ? 59 5E 8B E5 5D C3");
+  ADDR BlScanText(tsf_gIdDictionaryLoc,
+                  "89 15 ? ? ? ? E8 ? ? ? ? 8B F0 89 75 F0");
+  ADDR BlScanText(tsf_gEvalState_globalVarsLoc,
+                  "B9 ? ? ? ? E8 ? ? ? ? 68 ? ? ? ? 6A 0A 68 ? ? ? ? B9 ? ? ? "
+                  "? E8 ? ? ? ? E8 ? ? ? ?");
 
   tsf_mCacheSequence = *(ADDR *)(tsf_mCacheSequenceLoc + 2);
   tsf_mCacheAllocator = *(ADDR *)(tsf_mCacheAllocatorLoc + 2);
@@ -233,6 +280,4 @@ bool tsf_InitInternal() {
   return true;
 }
 
-bool tsf_DeinitInternal() {
-	return true;
-}
+bool tsf_DeinitInternal() { return true; }

inc/tsfuncs/BlFuncs.hpp

@@ -5,10 +5,10 @@
 #ifndef _H_BLFUNCS
 #define _H_BLFUNCS
 
-// Require BlHooks to be included before this header
+// Ensure BlHooks is available (include it if not already included)
 #ifndef _H_BLHOOKS
-	#error "BlFuncs.hpp: You must include BlHooks.hpp first"
-#else
+#include "BlHooks.hpp"
+#endif
 
 typedef const char *(*tsf_StringCallback)(ADDR, signed int, const char *[]);
 typedef signed int (*tsf_IntCallback)(ADDR, signed int, const char *[]);
@@ -42,12 +42,21 @@ void tsf_AddVar(const char *, signed int *);
 void tsf_AddVar(const char *, float *);
 void tsf_AddVar(const char *, bool *);
 
-ADDR tsf_AddConsoleFuncInternal(const char *, const char *, const char *, signed int, const char *, signed int, signed int);
-void tsf_AddConsoleFunc(const char *, const char *, const char *, tsf_StringCallback, const char *, signed int, signed int);
-void tsf_AddConsoleFunc(const char *, const char *, const char *, tsf_IntCallback, const char *, signed int, signed int);
-void tsf_AddConsoleFunc(const char *, const char *, const char *, tsf_FloatCallback, const char *, signed int, signed int);
-void tsf_AddConsoleFunc(const char *, const char *, const char *, tsf_VoidCallback, const char *, signed int, signed int);
-void tsf_AddConsoleFunc(const char *, const char *, const char *, tsf_BoolCallback, const char *, signed int, signed int);
+ADDR tsf_AddConsoleFuncInternal(const char *, const char *, const char *,
+                                signed int, const char *, signed int,
+                                signed int);
+void tsf_AddConsoleFunc(const char *, const char *, const char *,
+                        tsf_StringCallback, const char *, signed int,
+                        signed int);
+void tsf_AddConsoleFunc(const char *, const char *, const char *,
+                        tsf_IntCallback, const char *, signed int, signed int);
+void tsf_AddConsoleFunc(const char *, const char *, const char *,
+                        tsf_FloatCallback, const char *, signed int,
+                        signed int);
+void tsf_AddConsoleFunc(const char *, const char *, const char *,
+                        tsf_VoidCallback, const char *, signed int, signed int);
+void tsf_AddConsoleFunc(const char *, const char *, const char *,
+                        tsf_BoolCallback, const char *, signed int, signed int);
 
 bool tsf_InitInternal();
 
@@ -56,21 +65,32 @@ extern ADDR tsf_mCacheAllocator;
 extern ADDR tsf_gIdDictionary;
 extern ADDR tsf_gEvalState_globalVars;
 
-BlFunctionDefExtern(const char *, __stdcall, tsf_BlStringTable__insert, const char *, bool);
-BlFunctionDefExtern(ADDR, __fastcall, tsf_BlNamespace__find, const char *, const char *);
-BlFunctionDefExtern(ADDR, __thiscall, tsf_BlNamespace__createLocalEntry, ADDR, const char *);
+BlFunctionDefExtern(const char *, __stdcall, tsf_BlStringTable__insert,
+                    const char *, bool);
+BlFunctionDefExtern(ADDR, __fastcall, tsf_BlNamespace__find, const char *,
+                    const char *);
+BlFunctionDefExtern(ADDR, __thiscall, tsf_BlNamespace__createLocalEntry, ADDR,
+                    const char *);
 BlFunctionDefExtern(void, __thiscall, tsf_BlDataChunker__freeBlocks, ADDR);
-BlFunctionDefExtern(const char *, , tsf_BlCon__evaluate, ADDR, signed int, const char **);
+BlFunctionDefExtern(const char *, , tsf_BlCon__evaluate, ADDR, signed int,
+                    const char **);
 BlFunctionDefExtern(const char *, , tsf_BlCon__executef, signed int, ...);
-BlFunctionDefExtern(const char *, , tsf_BlCon__executefSimObj, ADDR *, signed int, ...);
-BlFunctionDefExtern(const char *, __thiscall, tsf_BlCon__getVariable, const char *);
-BlFunctionDefExtern(void, __thiscall, tsf_BlDictionary__addVariable, ADDR *, const char *, signed int, void *);
-BlFunctionDefExtern(ADDR *, __thiscall, tsf_BlSim__findObject_name, const char *);
-BlFunctionDefExtern(char *, __stdcall, tsf_BlStringStack__getArgBuffer, unsigned int);
-BlFunctionDefExtern(const char *, __thiscall, tsf_BlSimObject__getDataField, ADDR, const char *, const char *);
-BlFunctionDefExtern(void, __thiscall, tsf_BlSimObject__setDataField, ADDR, const char *, const char *, const char *);
-BlFunctionDefExtern(char *, __fastcall, tsf_BlCon__getReturnBuffer, unsigned int);
-
+BlFunctionDefExtern(const char *, , tsf_BlCon__executefSimObj, ADDR *,
+                    signed int, ...);
+BlFunctionDefExtern(const char *, __thiscall, tsf_BlCon__getVariable,
+                    const char *);
+BlFunctionDefExtern(void, __thiscall, tsf_BlDictionary__addVariable, ADDR *,
+                    const char *, signed int, void *);
+BlFunctionDefExtern(ADDR *, __thiscall, tsf_BlSim__findObject_name,
+                    const char *);
+BlFunctionDefExtern(char *, __stdcall, tsf_BlStringStack__getArgBuffer,
+                    unsigned int);
+BlFunctionDefExtern(const char *, __thiscall, tsf_BlSimObject__getDataField,
+                    ADDR, const char *, const char *);
+BlFunctionDefExtern(void, __thiscall, tsf_BlSimObject__setDataField, ADDR,
+                    const char *, const char *, const char *);
+BlFunctionDefExtern(char *, __fastcall, tsf_BlCon__getReturnBuffer,
+                    unsigned int);
 
 // Function short names
 
@@ -94,16 +114,23 @@ BlFunctionDefExtern(char *, __fastcall, tsf_BlCon__getReturnBuffer, unsigned int
 
 #define BlAddFunction tsf_AddConsoleFunc
 
-#define __22ND_ARGUMENT(a1, a2, a3, a4, a5, a6, a7, a8, a9, a10, a11, a12, a13, a14, a15, a16, a17, a18, a19, a20, a21, a22, ...) a22
-#define __NUM_LIST(...) __22ND_ARGUMENT(dummy, ##__VA_ARGS__, 20, 19, 18, 17, 16, 15, 14, 13, 12, 11, 10, 9, 8, 7, 6, 5, 4, 3, 2, 1, 0)
-#define BlCall(...) \
-	tsf_BlCon__executef(__NUM_LIST(__VA_ARGS__), __VA_ARGS__)
+#define __22ND_ARGUMENT(a1, a2, a3, a4, a5, a6, a7, a8, a9, a10, a11, a12,     \
+                        a13, a14, a15, a16, a17, a18, a19, a20, a21, a22, ...) \
+  a22
+#define __NUM_LIST(...)                                                        \
+  __22ND_ARGUMENT(dummy, ##__VA_ARGS__, 20, 19, 18, 17, 16, 15, 14, 13, 12,    \
+                  11, 10, 9, 8, 7, 6, 5, 4, 3, 2, 1, 0)
+#define BlCall(...) tsf_BlCon__executef(__NUM_LIST(__VA_ARGS__), __VA_ARGS__)
 #define BlCallObj(obj, ...)                                                    \
   tsf_BlCon__executefSimObj((ADDR *)obj, __NUM_LIST(__VA_ARGS__), __VA_ARGS__)
 
-#define BlFuncsInit() if(!tsf_InitInternal()) { return false; }
-#define BlFuncsDeinit() if(!tsf_DeinitInternal()) { return false; }
-
+#define BlFuncsInit()                                                          \
+  if (!tsf_InitInternal()) {                                                   \
+    return false;                                                              \
+  }
+#define BlFuncsDeinit()                                                        \
+  if (!tsf_DeinitInternal()) {                                                 \
+    return false;                                                              \
+  }
 
 #endif
-#endif

inc/tsfuncs/BlHooks.cpp

@@ -2,21 +2,22 @@
 //////////////////////////////////////////////////
 // RedoBlHooks Version 3.0
 
-
 // Includes
 
+#ifndef WIN32_LEAN_AND_MEAN
+#define WIN32_LEAN_AND_MEAN
+#include <cstdlib>
+#endif
 #include <Windows.h>
 #include <Psapi.h>
 // #include <map>
 
 #include "BlHooks.hpp"
 
-
 // Scanned structures
 
 BlFunctionDefIntern(tsh_BlPrintf);
 
-
 // Sig Scanning
 
 ADDR ImageBase;
@@ -26,7 +27,8 @@ void tsh_i_InitScanner(){
   HMODULE module = GetModuleHandle(NULL);
   if (module) {
     MODULEINFO info;
-		GetModuleInformation(GetCurrentProcess(), module, &info, sizeof(MODULEINFO));
+    GetModuleInformation(GetCurrentProcess(), module, &info,
+                         sizeof(MODULEINFO));
     ImageBase = (ADDR)info.lpBaseOfDll;
     ImageSize = info.SizeOfImage;
   }
@@ -40,7 +42,8 @@ bool tsh_i_CompareData(BYTE *data, BYTE *pattern, char* mask){
   return (*mask) == 0;
 }
 
-ADDR tsh_i_FindPattern(ADDR imageBase, ADDR imageSize, BYTE *pattern, char *mask){
+ADDR tsh_i_FindPattern(ADDR imageBase, ADDR imageSize, BYTE *pattern,
+                       char *mask) {
   for (ADDR i = imageBase; i < imageBase + imageSize; i++) {
     if (tsh_i_CompareData((PBYTE)i, pattern, mask)) {
       return i;
@@ -87,12 +90,12 @@ void tsh_i_PatternTextToCode(char* text, char** opatt, char** omask) {
   *omask = mask;
 }
 
-
 // Public functions for sig scanning
 
 // Scan using code-style pattern
 ADDR tsh_ScanCode(char *pattern, char *mask) {
-	return tsh_i_FindPattern(ImageBase, ImageSize-strlen(mask), (BYTE*)pattern, mask);
+  return tsh_i_FindPattern(ImageBase, ImageSize - strlen(mask), (BYTE *)pattern,
+                           mask);
 }
 
 // Scan using a text-style pattern
@@ -109,7 +112,6 @@ ADDR tsh_ScanText(char* text) {
   return res;
 }
 
-
 // Call Patching and Hooking
 
 // Remove protection from address
@@ -117,7 +119,8 @@ ADDR tsh_ScanText(char* text) {
 
 void tsh_DeprotectAddress(ADDR length, ADDR location) {
   DWORD oldProtection;
-	VirtualProtect((void*)location, length, PAGE_EXECUTE_READWRITE, &oldProtection);
+  VirtualProtect((void *)location, length, PAGE_EXECUTE_READWRITE,
+                 &oldProtection);
   // tsh_DeprotectedAddresses[location] = {length, oldProtection};
 }
 
@@ -135,9 +138,7 @@ void tsh_ReplaceInt(ADDR addr, int rval) {
   tsh_PatchBytes(4, addr, (BYTE *)(&rval));
 }
 
-int tsh_i_CallOffset(ADDR instr, ADDR func) {
-	return func - (instr+4);
-}
+int tsh_i_CallOffset(ADDR instr, ADDR func) { return func - (instr + 4); }
 
 void tsh_i_ReplaceCall(ADDR instr, ADDR target) {
   tsh_ReplaceInt(instr, tsh_i_CallOffset(instr, target));
@@ -161,11 +162,13 @@ void tsh_UnhookFunction(ADDR victim, BYTE* origbytes){
   tsh_i_PatchCopy(victim, (ADDR)origbytes, 6); // restore old data
 }
 
-int tsh_PatchAllMatchesCode(ADDR len, char* patt, char* mask, char* replace, bool debugprint){
+int tsh_PatchAllMatchesCode(ADDR len, char *patt, char *mask, char *replace,
+                            bool debugprint) {
   int numpatched = 0;
   for (ADDR i = ImageBase; i < ImageBase + ImageSize - len; i++) {
     if (tsh_i_CompareData((BYTE *)i, (BYTE *)patt, mask)) {
-			if(debugprint) BlPrintf("RedoBlHooks: Patching call at %08x", i);
+      if (debugprint)
+        BlPrintf("RedoBlHooks: Patching call at %08x", i);
 
       numpatched++;
       tsh_DeprotectAddress(i, len);
@@ -177,7 +180,8 @@ int tsh_PatchAllMatchesCode(ADDR len, char* patt, char* mask, char* replace, boo
   return numpatched;
 }
 
-int tsh_PatchAllMatchesHex(ADDR len, char* text, char* replace, bool debugprint) {
+int tsh_PatchAllMatchesHex(ADDR len, char *text, char *replace,
+                           bool debugprint) {
   char *patt;
   char *mask;
   tsh_i_PatternTextToCode(text, &patt, &mask);
@@ -190,18 +194,16 @@ int tsh_PatchAllMatchesHex(ADDR len, char* text, char* replace, bool debugprint)
   return res;
 }
 
-
 // Initialization
 
 bool tsh_InitInternal() {
   tsh_i_InitScanner();
 
-	BlScanFunctionText(tsh_BlPrintf, "8D 44 24 08 33 D2 50 FF 74 24 08 33 C9 E8 ? ? ? ? 83 C4 08 C3");
+  BlScanFunctionText(
+      tsh_BlPrintf,
+      "8D 44 24 08 33 D2 50 FF 74 24 08 33 C9 E8 ? ? ? ? 83 C4 08 C3");
 
   return true;
 }
 
-bool tsh_DeinitInternal() {
-	return true;
-}
-
+bool tsh_DeinitInternal() { return true; }

inc/tsfuncs/BlHooks.hpp

@@ -5,13 +5,11 @@
 #ifndef _H_BLHOOKS
 #define _H_BLHOOKS
 
-
 // Typedefs
 
 typedef unsigned char BYTE;
 typedef unsigned int ADDR;
 
-
 // Prototypes
 
 bool tsh_InitInternal();
@@ -25,7 +23,6 @@ void tsh_PatchByte(ADDR, BYTE);
 void tsh_PatchBytes(unsigned int, ADDR, BYTE *);
 void tsh_PatchInt(ADDR, int);
 
-
 // Debug print settings
 
 #ifndef TSH_NO_DEBUG_PRINT
@@ -34,29 +31,32 @@ void tsh_PatchInt(ADDR, int);
 #define tsh_DEBUGPRINT true
 #endif
 
-
 // Function short names
 
 // Use in code when the def is not shared in a header
 #define BlFunctionDef(returnType, convention, name, ...)                       \
   typedef returnType(convention *tsh_##name##FnT)(__VA_ARGS__);                \
   tsh_##name##FnT name;
-// Use in header for shared function defs when a BlFunctionDefIntern exists in code
+// Use in header for shared function defs when a BlFunctionDefIntern exists in
+// code
 #define BlFunctionDefExtern(returnType, convention, name, ...)                 \
   typedef returnType(convention *tsh_##name##FnT)(__VA_ARGS__);                \
   extern tsh_##name##FnT name;
-// Use in code for shared function defs when a BlFunctionDefExtern exists in header
-#define BlFunctionDefIntern(name) \
-	tsh_##name##FnT name;
+// Use in code for shared function defs when a BlFunctionDefExtern exists in
+// header
+#define BlFunctionDefIntern(name) tsh_##name##FnT name;
 
-// Scan for and assign the pattern to the variable, or err and return if not found
+// Scan for and assign the pattern to the variable, or err and return if not
+// found
 #define BlScanFunctionCode(target, patt, mask)                                 \
   target = (tsh_##target##FnT)tsh_ScanCode((char *)patt, (char *)mask);        \
   if (!target) {                                                               \
     BlPrintf("RedoBlHooks | Cannot find function " #target "!");               \
     return false;                                                              \
   } else {                                                                     \
-		if(tsh_DEBUGPRINT) BlPrintf("RedoBlHooks | Found function "#target" at %08x", (int)target); \
+    if (tsh_DEBUGPRINT)                                                        \
+      BlPrintf("RedoBlHooks | Found function " #target " at %08x",             \
+               (int)target);                                                   \
   }
 #define BlScanFunctionText(target, text)                                       \
   target = (tsh_##target##FnT)tsh_ScanText((char *)text);                      \
@@ -64,7 +64,9 @@ void tsh_PatchInt(ADDR, int);
     BlPrintf("RedoBlHooks | Cannot find function " #target "!");               \
     return false;                                                              \
   } else {                                                                     \
-		if(tsh_DEBUGPRINT) BlPrintf("RedoBlHooks | Found function "#target" at %08x", (int)target); \
+    if (tsh_DEBUGPRINT)                                                        \
+      BlPrintf("RedoBlHooks | Found function " #target " at %08x",             \
+               (int)target);                                                   \
   }
 #define BlScanCode(target, patt, mask)                                         \
   target = tsh_ScanCode((char *)patt, (char *)mask);                           \
@@ -72,7 +74,8 @@ void tsh_PatchInt(ADDR, int);
     BlPrintf("RedoBlHooks | Cannot find pattern " #target "!");                \
     return false;                                                              \
   } else {                                                                     \
-		if(tsh_DEBUGPRINT) BlPrintf("RedoBlHooks | Found "#target" at %08x", (int)target); \
+    if (tsh_DEBUGPRINT)                                                        \
+      BlPrintf("RedoBlHooks | Found " #target " at %08x", (int)target);        \
   }
 #define BlScanText(target, text)                                               \
   target = tsh_ScanText((char *)text);                                         \
@@ -80,14 +83,20 @@ void tsh_PatchInt(ADDR, int);
     BlPrintf("RedoBlHooks | Cannot find " #target "!");                        \
     return false;                                                              \
   } else {                                                                     \
-		if(tsh_DEBUGPRINT) BlPrintf("RedoBlHooks | Found "#target" at %08x", (int)target); \
+    if (tsh_DEBUGPRINT)                                                        \
+      BlPrintf("RedoBlHooks | Found " #target " at %08x", (int)target);        \
   }
 
 // Use in code to define the data and functions for hooking a function
 #define BlFunctionHookDef(func)                                                \
   BYTE tsh_BlFunctionHook##func##Data[6];                                      \
-	void func##HookOn(){ tsh_HookFunction((ADDR)func, (ADDR)func##Hook, tsh_BlFunctionHook##func##Data); } \
-	void func##HookOff(){ tsh_UnhookFunction((ADDR)func, tsh_BlFunctionHook##func##Data); }
+  void func##HookOn() {                                                        \
+    tsh_HookFunction((ADDR)func, (ADDR)func##Hook,                             \
+                     tsh_BlFunctionHook##func##Data);                          \
+  }                                                                            \
+  void func##HookOff() {                                                       \
+    tsh_UnhookFunction((ADDR)func, tsh_BlFunctionHook##func##Data);            \
+  }
 // Use in code to initialize the hook once
 #define BlFunctionHookInit(func)                                               \
   tsh_DeprotectAddress(6, (ADDR)func);                                         \
@@ -95,28 +104,38 @@ void tsh_PatchInt(ADDR, int);
 
 // Replace all matches of the pattern with the given byte string
 #define BlPatchAllMatchesCode(len, patt, mask, repl)                           \
-	tsh_PatchAllMatchesCode((ADDR)len, (char*)patt, (char*)mask, (char*)repl, tsh_DEBUGPRINT);
+  tsh_PatchAllMatchesCode((ADDR)len, (char *)patt, (char *)mask, (char *)repl, \
+                          tsh_DEBUGPRINT);
 #define BlPatchAllMatchesText(len, text, repl)                                 \
-	tsh_PatchAllMatchesCode((ADDR)len, (char*)text, (char*)repl, tsh_DEBUGPRINT);
+  tsh_PatchAllMatchesCode((ADDR)len, (char *)text, (char *)repl,               \
+                          tsh_DEBUGPRINT);
 
 // Deprotect and replace one byte
-#define BlPatchByte(addr, byte) \
-	tsh_PatchByte((ADDR)addr, (BYTE)byte);
+#define BlPatchByte(addr, byte) tsh_PatchByte((ADDR)addr, (BYTE)byte);
 // Deprotect and replace a byte string
 #define BlPatchBytes(len, addr, repl)                                          \
   tsh_PatchBytes((ADDR)len, (ADDR)addr, (BYTE *)repl);
 
 // BlPrintf(char* format, ...)
-#define BlPrintf(...) if(tsh_BlPrintf) { tsh_BlPrintf(__VA_ARGS__); }
+#define BlPrintf(...)                                                          \
+  if (tsh_BlPrintf) {                                                          \
+    tsh_BlPrintf(__VA_ARGS__);                                                 \
+  }
 // BlHooksInit() -> bool: success
-#define BlHooksInit() if(!tsh_InitInternal()) { BlPrintf("BlHooksInit failed"); return false; }
+#define BlHooksInit()                                                          \
+  if (!tsh_InitInternal()) {                                                   \
+    BlPrintf("BlHooksInit failed");                                            \
+    return false;                                                              \
+  }
 // BlHooksDeinit() -> bool: success
-#define BlHooksDeinit() if(!tsh_DeinitInternal()) { BlPrintf("BlHooksDeinit failed"); return false; }
-
+#define BlHooksDeinit()                                                        \
+  if (!tsh_DeinitInternal()) {                                                 \
+    BlPrintf("BlHooksDeinit failed");                                          \
+    return false;                                                              \
+  }
 
 // Scanned structures
 
 BlFunctionDefExtern(void, , tsh_BlPrintf, const char *, ...);
 
-
 #endif

make.bat

@@ -0,0 +1,27 @@
+@echo off
+cd /d %~dp0
+
+REM Ensure MinGW32 toolchain is first in PATH (matches compile.bat)
+set "PATH=C:\msys64\mingw32\bin;%PATH%"
+
+REM Configure CMake (generate into build/)
+cmake -S . -B build -G "MinGW Makefiles"
+if errorlevel 1 goto :error
+
+REM Build (Release by default)
+cmake --build build --config Release -j
+if errorlevel 1 goto :error
+
+echo.
+echo Build completed.
+echo Outputs in .\build :
+echo   - BlockLua.dll
+
+echo   - BlockLua-Unsafe.dll
+
+exit /b 0
+
+:error
+echo.
+echo Build failed. See errors above.
+exit /b 1

readme.md

@@ -14,10 +14,10 @@ Lua scripting for Blockland
 ### From TorqueScript
 `'print('hello world')` - Execute Lua in the console by prepending a `'` (single quote)  
 `luaeval("code");` - Execute Lua code  
-`luacall("funcName", %args...);` - Call a Lua function (supports indexing tables and object methods)  
-`luaexec("fileName");` - Execute a Lua file. Path rules are the same as when executing .cs files, relative paths are allowed.  
-`luaget("varName");` - Read a Lua global variable (supports indexing tables)  
-`luaset("varName", %value);` - Write a Lua global variable (supports indexing tables)  
+`luacall("funcName", %args...);` - Call a Lua global function  
+`luaexec("fileName");` - Execute a Lua file. Path rules are the same as executing .cs files.  
+`luaget("varName");` - Read a Lua global variable  
+`luaset("varName", %value);` - Write a Lua global variable  
 
 ### From Lua
 `bl.eval('code')` - Eval TorqueScript code  
@@ -35,9 +35,7 @@ Lua scripting for Blockland
 `object.key = value` - Associate Lua data with a Torque object  
 `object:method(args)` - Call a Torque object method  
 `object[index]` - Access a member of a Torque set or group  
-`for child in object:members() do` - Iterate objects within of a Torque set or group. Indices start at 0 like in Torque.
-`bl.isObject(object, objectID, or 'objectName')` - Check if an object exists  
-`object:exists()` - Check if an object exists  
+`for childIndex, child in object:members() do` - Iterate objects within of a Torque set or group. Indices start at 0 like in Torque.
 
 ### Timing/Schedules
 `sched = bl.schedule(timeMs, function, args...)` - Schedule a Lua function to be called later, similar to schedule in Torque  
@@ -48,25 +46,15 @@ Lua scripting for Blockland
 `for object in bl.boxSearch(vector{centerX,y,z}, vector{sizeX,y,z}, 'objtype'/{'objtypes',...}) do` - Find all objects in the world of the specified type(s) whose bounding box overlaps with the specified box. See the Types section for a list of valid object types.  
 `for object in bl.radiusSearch(vector{centerX,y,z}, radius, 'objtype'/{'objtypes',...}) do` - Find all objects of the specified type(s) whose bounding box overlaps with the specified sphere. See the Types section for a list of valid object types.  
 
-### List of Object Classes (for raycasts and searches)
-`'all'` - Any object  
-`'player'` - Players or bots  
-`'item'` - Items  
-`'vehicle'` - Vehicles  
-`'projectile'` - Projectiles  
-`'brick'` - Bricks with raycasting enabled  
-`'brickalways'` - All bricks including those with raycasting disabled  
-Other types: `'static'`, `'environment'`, `'terrain'`, `'water'`, `'trigger'`, `'marker'`, `'gamebase'`, `'shapebase'`, `'camera'`, `'staticshape'`, `'vehicleblocker'`, `'explosion'`, `'corpse'`, `'debris'`, `'physicalzone'`, `'staticts'`, `'staticrendered'`, `'damagableitem'`  
-
 ### Server-Client Communication
-`bl.addServerCmd('commandName', function(client, args...) ... end)` - Register a /command on the server  
-`bl.addClientCmd('commandName', function(args...) ... end)` - Register a client command on the client  
-`bl.commandToServer('commandName', args...)` - As a client, execute a server command  
-`bl.commandToClient(client, 'commandName', args...)` - As the server, execute a client command on a specific client  
+`bl.addServerCmd('commandName', function(client, args...) yourCode end)` - Register a /command on the server  
+`bl.addClientCmd('commandName', function(args...) yourCode end)` - Register a client command on the client  
+`bl.commandToServer('commandName', args...)` - Execute a server command as a client  
+`bl.commandToClient('commandName', args...)` - As the server, execute a client command on a specific client  
 `bl.commandToAll('commandName', args...)` - As the server, execute a client command on all clients  
 
 ### Packages/Hooks
-`bl.hook('packageName', 'functionName', 'before'/'after', function(args) ... end)` - Hook a Torque function with a Lua function.  
+`bl.hook('packageName', 'functionName', 'before'/'after', function(args) yourCode end)` - Hook a Torque function with a Lua function.  
 `args` is an array containing the arguments provided to the function. If the hook is `before`, these can be modified before being passed to the parent function.  
 If `args._return` is set to anything other than nil by a `before` hook, the parent function will not be called, and the function will simply return that value. Also in this case, any `after` hook will not be executed.  
 In an `after` hook, `args._return` is set to the value returned by the parent function, and can be modified.  
@@ -113,11 +101,10 @@ When reading from outside ZIPs, binary files are fully supported.
 `bl.type('className::funcName', 'type')` - Register the return type of a Torque object method.  
 `bl.class('className')` - Register an existing Torque class to be used from Lua. Already done for all built-in classes.  
 `bl.class('className', 'parentClassName')` - Same as above, with inheritance  
-`bl.boolean(arg)` - Manually convert a Torque boolean (0 or 1) into a Lua boolean.  
-`bl.object(arg)` - Manually convert a Torque object reference (object ID or name) into a Lua object.  
-`bl.string(arg)` - Manually convert any automatically-converted Torque value back into a string. This is not as reliable as using `bl.type` to specify the type as a string beforehand.  
+`bl.boolean(thing)` - Manually convert a Torque boolean (0 or 1) into a Lua boolean.  
+`bl.object(thing)` - Manually convert a Torque object reference (object ID or name) into a Lua object.
 
-### Vector
+### Vectors
 `vec = vector{x,y,z}` - Create a vector. Can have any number of elements  
 `vec1 + vec2` - Add  
 `vec1 - vec2` - Subtract  
@@ -142,20 +129,15 @@ When reading from outside ZIPs, binary files are fully supported.
 `vec1:distance(vec2)` - Distance between two points  
 `vec2 = vec:copy()` - Clone a vector so its elements can be modified without affecting the original. Usually not needed - the builtin vector functions never modify vectors in-place.
 
-### Matrix
-WIP
-
 ### Extended Standard Lua Library
-`str[index]`  
-`str[{start,stop}]`  
+`string[index]`  
+`string[{start,stop}]`  
 `string.split(str, separator='' (splits into chars), noregex=false)`  
 `string.bytes(str)`  
 `string.trim(str, charsToTrim=' \t\r\n')`  
 `table.empty`  
 `table.map(func, ...)`  
-`table.mapk(func, ...)`  
 `table.map_list(func, ...)`  
-`table.mapi_list(func, ...)`  
 `table.swap(tbl)`  
 `table.reverse(list)`  
 `table.islist(list)`  
@@ -181,39 +163,41 @@ WIP
 ## Type Conversion
 When a TorqueScript function is called from Lua or vice-versa, the arguments and return value must be converted between the two languages' type systems.  
 TorqueScript stores no type information; all values in TorqueScript are strings. So it's necessary to make some inferences when converting values between the two languages.  
-
+### From TorqueScript to Lua
+- Any numeric value becomes a Lua `number`, except as specified with `bl.type`, which may convert a value into a `boolean` or a Torque object container.  
+- The empty string "" becomes `nil`  
+- A string containing three numbers separated by spaces becomes a `vector`  
+- A string containing six numbers separated by spaces becomes a table of two vectors  
+- Any other string is passed directly as a `string`
 ### From Lua to TorqueScript
 - `nil` becomes the empty string ""  
 - `true` and `false` become "1" and "0" respectively  
-- A Torque object container becomes its object ID  
+- Torque containers become their object ID  
 - A `vector` becomes a string containing three numbers separated by spaces  
-- A table of two `vector`s becomes a string containing six numbers separated by spaces  
-- (WIP) A `matrix` is converted into an axis-angle (a "transform"), a string containing seven numbers separated by spaces  
+- A table of two vectors becomes a string containing six numbers separated by spaces  
 - Any `string` is passed directly as a string  
 - Tables cannot be passed and will throw an error  
 
-### From TorqueScript to Lua
-- The empty string "" becomes `nil`  
-- Any numeric value becomes a Lua `number`, except as specified with `bl.type`, which may convert a value into a `boolean` or a Torque object container.  
-- A string containing two or three numbers separated by single spaces becomes a `vector`  
-- A string containing six numbers separated by single spaces becomes a table of two vectors, usually defining the corners a bounding box  
-- (WIP) A string containing seven numbers separated by single spaces is treated as an axis-angle (a "transform"), and is converted into a `matrix` representing the translation and rotation  
-- Any other string is passed directly as a `string`  
-  
-For scenarios where the automatic TorqueScript->Lua conversion rules are insufficient or incorrect, use `bl.type`.  
-To convert things by hand, use `bl.object`, `bl.boolean`, or `bl.string`.
-
 ## I/O and Safety
 All Lua code is sandboxed, and file access is confined to the default directories in the same way TorqueScript is.  
 BlockLua also has access to any C libraries installed in the `modules/lualib` folder, so be careful throwing things in there.  
 ### Unsafe Mode
 BlockLua can be built in Unsafe Mode by specifying the `-DBLLUA_UNSAFE` compiler flag. This removes the sandboxing of Lua code, allowing it to access any file and use any library, including ffi.  
-A more limited option is `-DBLLUA_ALLOWFFI`, which allows the use of the `ffi` library. This can still be exploited to grant all the same access as full unsafe mode.  
-Please do not publish add-ons that require either of these.  
+Please do not publish add-ons that require unsafe mode.  
+
+### List of Object Types
+`'all'` - Any object  
+`'player'` - Players or bots  
+`'item'` - Items  
+`'vehicle'` - Vehicles  
+`'projectile'` - Projectiles  
+`'brick'` - Bricks with raycasting enabled  
+`'brickalways'` - All bricks including those with raycasting disabled  
+Other types: `'static'`, `'environment'`, `'terrain'`, `'water'`, `'trigger'`, `'marker'`, `'gamebase'`, `'shapebase'`, `'camera'`, `'staticshape'`, `'vehicleblocker'`, `'explosion'`, `'corpse'`, `'debris'`, `'physicalzone'`, `'staticts'`, `'staticrendered'`, `'damagableitem'`  
 
 ## Compiling
 
 With any *32-bit* variant of GCC installed (such as MinGW or MSYS2), run the following command in the repo directory:  
-`g++ src/bllua4.cpp -o BlockLua.dll -m32 -shared -static-libgcc -Isrc -Iinc/tsfuncs -Iinc/lua -lpsapi -L. -llua5.1`  
+`g++ src/bllua4.cpp -o BlockLua.dll -m32 -shared -static-libgcc -Isrc -Iinc/tsfuncs -Iinc/lua -lpsapi -L. -llua5.1 src/bllua`  
 
 LuaJIT (lua5.1.dll) can be obtained from https://luajit.org/  

src/bllua4.cpp

@@ -1,13 +1,16 @@
-// BlockLua (bllua4): Advanced Lua interface for TorqueScript
+// BlockLua (bllua4): Simple Lua interface for TorqueScript
 
 // Includes
 
+#ifndef WIN32_LEAN_AND_MEAN
+#define WIN32_LEAN_AND_MEAN
+#endif
 #include <Windows.h>
 #include <Psapi.h>
 
-#include "lua.hpp"
-#include "BlHooks.cpp"
 #include "BlFuncs.cpp"
+#include "BlHooks.cpp"
+#include "lua.hpp"
 
 #include "luainterp.cpp"
 #include "lualibts.cpp"
@@ -28,7 +31,6 @@ INCLUDE_BIN(bll_fileLuaEnv    , "lua-env.lua");
 INCLUDE_BIN(bll_fileTsEnv, "ts-env.cs");
 INCLUDE_BIN(bll_fileLuaStd, "util/std.lua");
 INCLUDE_BIN(bll_fileLuaVector, "util/vector.lua");
-INCLUDE_BIN(bll_fileLuaMatrix     , "util/matrix.lua");
 INCLUDE_BIN(bll_fileLuaLibts, "util/libts-lua.lua");
 INCLUDE_BIN(bll_fileTsLibts, "util/libts-ts.cs");
 INCLUDE_BIN(bll_fileLuaLibbl, "util/libbl.lua");
@@ -57,26 +59,23 @@ bool init() {
 
   // Set up Lua environment
   BLL_LOAD_LUA(gL, bll_fileLuaEnv);
-	#ifdef BLLUA_ALLOWFFI
-		lua_pushboolean(gL, true);
-		lua_setglobal(gL, "_bllua_allowffi");
-	#endif
 #ifndef BLLUA_UNSAFE
   BLL_LOAD_LUA(gL, bll_fileLuaEnvSafe);
 #endif
 
-	// Load utilities in Lua
+  // Expose Lua API to TS
+  BlAddFunction(NULL, NULL, "_bllua_luacall", bll_ts_luacall,
+                "LuaCall(name, ...) - Call Lua function and return result", 2,
+                20);
+  BlEval(bll_fileTsEnv);
+
+  // Load utilities
   BLL_LOAD_LUA(gL, bll_fileLuaStd);
   BLL_LOAD_LUA(gL, bll_fileLuaVector);
-	BLL_LOAD_LUA(gL, bll_fileLuaMatrix);
   BLL_LOAD_LUA(gL, bll_fileLuaLibts);
+  BlEval(bll_fileTsLibts);
   BLL_LOAD_LUA(gL, bll_fileLuaLibbl);
   BLL_LOAD_LUA(gL, bll_fileLuaLibblTypes);
-	
-	// Expose Lua API to TS
-	BlAddFunction(NULL, NULL, "_bllua_luacall", bll_ts_luacall, "LuaCall(name, ...) - Call Lua function and return result", 2, 20);
-	BlEval(bll_fileTsEnv);
-	BlEval(bll_fileTsLibts);
   BlEval(bll_fileTsLibblSupport);
   BlEval(bll_fileLoadaddons);
 
@@ -89,7 +88,8 @@ bool init() {
 bool deinit() {
   BlPrintf("BlockLua: Unloading");
 
-	BlEval("$_bllua_active=0;deactivatePackage(_bllua_main);");
+  BlEval("deactivatePackage(_bllua_main);");
+  BlEval("$_bllua_active = 0;");
   bll_LuaEval(gL, "for _,f in pairs(_bllua_on_unload) do f() end");
 
   lua_close(gL);
@@ -103,8 +103,11 @@ bool deinit() {
 
 bool __stdcall DllMain(HINSTANCE hinstance, DWORD reason, void *reserved) {
   switch (reason) {
-		case DLL_PROCESS_ATTACH: return init();
-		case DLL_PROCESS_DETACH: return deinit();
-		default                : return true;
+  case DLL_PROCESS_ATTACH:
+    return init();
+  case DLL_PROCESS_DETACH:
+    return deinit();
+  default:
+    return true;
   }
 }

src/lua-env-safe.lua

@@ -4,7 +4,9 @@
 
 
 -- Utility: Convert a list of strings into a map of string->true
-local function tmap(t) local u = {}; for _, n in ipairs(t) do u[n] = true end; return u; end
+local function tmap(t)
+  local u = {}; for _, n in ipairs(t) do u[n] = true end; return u;
+end
 
 -- Save banned global variables for wrapping with safe functions
 local old_io = io
@@ -12,7 +14,6 @@ local old_require = require
 local old_os = os
 local old_debug = debug
 local old_package = package
-local old_allowffi = _bllua_allowffi
 
 -- Remove all global variables except a whitelist
 local ok_names = tmap {
@@ -38,10 +39,13 @@ end
 
 -- Sanitize file paths to point only to allowed files within the game directory
 -- List of allowed directories for reading/writing
--- modules/lualib is also allowed as read-only
 local allowed_dirs = tmap {
   'add-ons', 'base', 'config', 'saves', 'screenshots', 'shaders'
 }
+-- List of allowed directories for reading only
+local allowed_dirs_readonly = tmap {
+  'lualib'
+}
 -- List of disallowed file extensions - basically executable file extensions
 -- Note that even without this protection, exploiting would still require somehow
 --   getting a file within the allowed directories to autorun,
@@ -77,15 +81,14 @@ local function safe_path(fn, readonly)
   end
   -- allow only whitelisted dirs
   local dir = fn:match('^([^/]+)/')
-	if not (dir and (
-		allowed_dirs[dir:lower()] or
-		( readonly and fn:find('^modules/lualib/') ) ))
-	then
-		return nil, 'File is in disallowed directory '..(dir or 'nil')
+  if (not dir) or (
+        (not allowed_dirs[dir:lower()]) and
+        ((not readonly) or (not allowed_dirs_readonly[dir:lower()]))) then
+    return nil, 'filename is in disallowed directory ' .. (dir or 'nil')
   end
-	-- disallow blacklisted extensions
+  -- disallow blacklisted extensions or no extension
   local ext = fn:match('%.([^/%.]+)$')
-	if ext and disallowed_exts[ext:lower()] then
+  if (not ext) or (disallowed_exts[ext:lower()]) then
     return nil, 'Filename \'' .. fn .. '\' has disallowed extension \'' ..
         (ext or '') .. '\''
   end
@@ -103,6 +106,7 @@ function _bllua_io_open(fn, md)
     return nil, err
   end
 end
+
 -- Allow io.type (works on file handles returned by io.open)
 function _bllua_io_type(f)
   return old_io.type(f)
@@ -116,7 +120,6 @@ local disallowed_packages = tmap {
   'ffi', 'debug', 'package', 'io', 'os',
   '_bllua_ts',
 }
-if old_allowffi then disallowed_packages['ffi'] = nil end
 function _bllua_requiresecure(name)
   if name:find('[^a-zA-Z0-9_%-%.]') or name:find('%.%.') or
       name:find('^%.') or name:find('%.$') then
@@ -128,6 +131,7 @@ function _bllua_requiresecure(name)
     return old_require(name)
   end
 end
+
 package = {
   seeall = old_package.seeall,
 }
@@ -139,5 +143,4 @@ debug = {
   getfilename = old_debug.getfilename, -- defined in lua.env.lua
 }
 
-print = _bllua_ts.echo
-print('  Executed bllua-env-safe.lua')
+_bllua_ts.echo('  Executed bllua-env-safe.lua')

src/lua-env.lua

@@ -37,9 +37,4 @@ function _bllua_on_error(err)
   return table.concat(tracelines, '\n')
 end
 
--- overridden in lua-env-safe.lua (executed if not BLLUA_UNSAFE)
-_bllua_io_open = io.open
-_bllua_requiresecure = require
-
-print = _bllua_ts.echo
-print('  Executed bllua-env.lua')
+_bllua_ts.echo('  Executed bllua-env.lua')

src/luainterp.cpp

@@ -1,5 +1,10 @@
 
 // Handle errors with a Lua function, defined in lua-env.lua
+#include "luainterp.hpp"
+#include "BlHooks.hpp"
+#include "lauxlib.h"
+#include "lua.h"
+#include <cstring>
 int bll_error_handler(lua_State *L) {
   lua_getfield(L, LUA_GLOBALSINDEX, "_bllua_on_error");
   if (!lua_isfunction(L, -1)) {
@@ -48,21 +53,26 @@ bool bll_LuaEval(lua_State* L, const char* str) {
 
 // Convert a Lua stack entry into a string for providing to TS
 // Use static buffer to avoid excessive malloc
-#define BLL_ARG_COUNT 20
-#define BLL_ARG_MAX 8192
 char bll_arg_buffer[BLL_ARG_COUNT][BLL_ARG_MAX];
 bool bll_toarg(lua_State *L, char *buf, int i, bool err) {
   if (lua_isstring(L, i)) {
     const char *str = lua_tostring(L, i);
     if (strlen(str) >= BLL_ARG_MAX) {
-			if(err) luaL_error(L, "argument to TS is too long - max length is 8192");
+      if (err)
+        luaL_error(L, "argument to TS is too long - max length is 8192");
       return true;
     } else {
-			strcpy(buf, str);
+#ifdef _MSC_VER
+      strncpy_s(buf, BLL_ARG_MAX, str, _TRUNCATE);
+#else
+      strncpy(buf, str, BLL_ARG_MAX - 1);
+      buf[BLL_ARG_MAX - 1] = '\0';
+#endif
       return false;
     }
   } else {
-		if(err) luaL_error(L, "argument to TS must be a string");
+    if (err)
+      luaL_error(L, "argument to TS must be a string");
     return true;
   }
 }

src/luainterp.hpp

@@ -0,0 +1,16 @@
+// Shared declarations for Lua <-> TS argument handling
+#ifndef _H_LUAINTERP_SHARED
+#define _H_LUAINTERP_SHARED
+
+#include "lua.h"
+#define BLL_ARG_COUNT 20
+#define BLL_ARG_MAX 8192
+
+extern char bll_arg_buffer[BLL_ARG_COUNT][BLL_ARG_MAX];
+bool bll_toarg(lua_State *L, char *buf, int i, bool err);
+int bll_pcall(lua_State *L, int nargs, int nret);
+void bll_printError(lua_State *L, const char *operation, const char *item);
+
+extern lua_State *gL;
+
+#endif

src/lualibts.cpp

@@ -3,14 +3,24 @@
 // TS -> Lua API
 
 // Call a TS function from Lua, push the result to Lua stack
-int bll_TsCall(lua_State* L, const char* oname, const char* fname, int argc, int ofs) {
+#include "BlFuncs.hpp"
+#include "BlHooks.hpp"
+#include "lauxlib.h"
+#include "lua.h"
+#include "luainterp.hpp"
+int bll_TsCall(lua_State *L, const char *oname, const char *fname, int argc,
+               int ofs) {
   ADDR obj = (ADDR)NULL;
   if (oname) {
     obj = BlObject(oname);
-		if(!obj) { return luaL_error(L, "Lua->TS call: Object not found"); }
+    if (!obj) {
+      return luaL_error(L, "Lua->TS call: Object not found");
+    }
   }
 
-	if(argc > BLL_ARG_COUNT) { return luaL_error(L, "Lua->TS call: Too many arguments (Max is 19)"); }
+  if (argc > BLL_ARG_COUNT) {
+    return luaL_error(L, "Lua->TS call: Too many arguments (Max is 19)");
+  }
 
   char *argv[BLL_ARG_COUNT];
   for (int i = 0; i < argc; i++) {
@@ -23,52 +33,184 @@ int bll_TsCall(lua_State* L, const char* oname, const char* fname, int argc, int
   const char *res;
   if (obj) {
     switch (argc) {
-			case  0: res = BlCallObj(obj, fname); break; // no idea why this happens sometimes, it shouldnt be possible
-			case  1: res = BlCallObj(obj, fname); break;
-			case  2: res = BlCallObj(obj, fname, argv[0]); break;
-			case  3: res = BlCallObj(obj, fname, argv[0], argv[1]); break;
-			case  4: res = BlCallObj(obj, fname, argv[0], argv[1], argv[2]); break;
-			case  5: res = BlCallObj(obj, fname, argv[0], argv[1], argv[2], argv[3]); break;
-			case  6: res = BlCallObj(obj, fname, argv[0], argv[1], argv[2], argv[3], argv[4]); break;
-			case  7: res = BlCallObj(obj, fname, argv[0], argv[1], argv[2], argv[3], argv[4], argv[5]); break;
-			case  8: res = BlCallObj(obj, fname, argv[0], argv[1], argv[2], argv[3], argv[4], argv[5], argv[6]); break;
-			case  9: res = BlCallObj(obj, fname, argv[0], argv[1], argv[2], argv[3], argv[4], argv[5], argv[6], argv[7]); break;
-			case 10: res = BlCallObj(obj, fname, argv[0], argv[1], argv[2], argv[3], argv[4], argv[5], argv[6], argv[7], argv[8]); break;
-			case 11: res = BlCallObj(obj, fname, argv[0], argv[1], argv[2], argv[3], argv[4], argv[5], argv[6], argv[7], argv[8], argv[9]); break;
-			case 12: res = BlCallObj(obj, fname, argv[0], argv[1], argv[2], argv[3], argv[4], argv[5], argv[6], argv[7], argv[8], argv[9], argv[10]); break;
-			case 13: res = BlCallObj(obj, fname, argv[0], argv[1], argv[2], argv[3], argv[4], argv[5], argv[6], argv[7], argv[8], argv[9], argv[10], argv[11]); break;
-			case 14: res = BlCallObj(obj, fname, argv[0], argv[1], argv[2], argv[3], argv[4], argv[5], argv[6], argv[7], argv[8], argv[9], argv[10], argv[11], argv[12]); break;
-			case 15: res = BlCallObj(obj, fname, argv[0], argv[1], argv[2], argv[3], argv[4], argv[5], argv[6], argv[7], argv[8], argv[9], argv[10], argv[11], argv[12], argv[13]); break;
-			case 16: res = BlCallObj(obj, fname, argv[0], argv[1], argv[2], argv[3], argv[4], argv[5], argv[6], argv[7], argv[8], argv[9], argv[10], argv[11], argv[12], argv[13], argv[14]); break;
-			case 17: res = BlCallObj(obj, fname, argv[0], argv[1], argv[2], argv[3], argv[4], argv[5], argv[6], argv[7], argv[8], argv[9], argv[10], argv[11], argv[12], argv[13], argv[14], argv[15]); break;
-			case 18: res = BlCallObj(obj, fname, argv[0], argv[1], argv[2], argv[3], argv[4], argv[5], argv[6], argv[7], argv[8], argv[9], argv[10], argv[11], argv[12], argv[13], argv[14], argv[15], argv[16]); break;
-			case 19: res = BlCallObj(obj, fname, argv[0], argv[1], argv[2], argv[3], argv[4], argv[5], argv[6], argv[7], argv[8], argv[9], argv[10], argv[11], argv[12], argv[13], argv[14], argv[15], argv[16], argv[17]); break;
-			case 20: res = BlCallObj(obj, fname, argv[0], argv[1], argv[2], argv[3], argv[4], argv[5], argv[6], argv[7], argv[8], argv[9], argv[10], argv[11], argv[12], argv[13], argv[14], argv[15], argv[16], argv[17], argv[18]); break;
-			default: res = ""; luaL_error(L, "Lua->TS object call: Too many arguments (Max is 19)");
+    case 0:
+      res = BlCallObj(obj, fname);
+      break; // no idea why this happens sometimes, it shouldnt be possible
+    case 1:
+      res = BlCallObj(obj, fname);
+      break;
+    case 2:
+      res = BlCallObj(obj, fname, argv[0]);
+      break;
+    case 3:
+      res = BlCallObj(obj, fname, argv[0], argv[1]);
+      break;
+    case 4:
+      res = BlCallObj(obj, fname, argv[0], argv[1], argv[2]);
+      break;
+    case 5:
+      res = BlCallObj(obj, fname, argv[0], argv[1], argv[2], argv[3]);
+      break;
+    case 6:
+      res = BlCallObj(obj, fname, argv[0], argv[1], argv[2], argv[3], argv[4]);
+      break;
+    case 7:
+      res = BlCallObj(obj, fname, argv[0], argv[1], argv[2], argv[3], argv[4],
+                      argv[5]);
+      break;
+    case 8:
+      res = BlCallObj(obj, fname, argv[0], argv[1], argv[2], argv[3], argv[4],
+                      argv[5], argv[6]);
+      break;
+    case 9:
+      res = BlCallObj(obj, fname, argv[0], argv[1], argv[2], argv[3], argv[4],
+                      argv[5], argv[6], argv[7]);
+      break;
+    case 10:
+      res = BlCallObj(obj, fname, argv[0], argv[1], argv[2], argv[3], argv[4],
+                      argv[5], argv[6], argv[7], argv[8]);
+      break;
+    case 11:
+      res = BlCallObj(obj, fname, argv[0], argv[1], argv[2], argv[3], argv[4],
+                      argv[5], argv[6], argv[7], argv[8], argv[9]);
+      break;
+    case 12:
+      res = BlCallObj(obj, fname, argv[0], argv[1], argv[2], argv[3], argv[4],
+                      argv[5], argv[6], argv[7], argv[8], argv[9], argv[10]);
+      break;
+    case 13:
+      res = BlCallObj(obj, fname, argv[0], argv[1], argv[2], argv[3], argv[4],
+                      argv[5], argv[6], argv[7], argv[8], argv[9], argv[10],
+                      argv[11]);
+      break;
+    case 14:
+      res = BlCallObj(obj, fname, argv[0], argv[1], argv[2], argv[3], argv[4],
+                      argv[5], argv[6], argv[7], argv[8], argv[9], argv[10],
+                      argv[11], argv[12]);
+      break;
+    case 15:
+      res = BlCallObj(obj, fname, argv[0], argv[1], argv[2], argv[3], argv[4],
+                      argv[5], argv[6], argv[7], argv[8], argv[9], argv[10],
+                      argv[11], argv[12], argv[13]);
+      break;
+    case 16:
+      res = BlCallObj(obj, fname, argv[0], argv[1], argv[2], argv[3], argv[4],
+                      argv[5], argv[6], argv[7], argv[8], argv[9], argv[10],
+                      argv[11], argv[12], argv[13], argv[14]);
+      break;
+    case 17:
+      res = BlCallObj(obj, fname, argv[0], argv[1], argv[2], argv[3], argv[4],
+                      argv[5], argv[6], argv[7], argv[8], argv[9], argv[10],
+                      argv[11], argv[12], argv[13], argv[14], argv[15]);
+      break;
+    case 18:
+      res =
+          BlCallObj(obj, fname, argv[0], argv[1], argv[2], argv[3], argv[4],
+                    argv[5], argv[6], argv[7], argv[8], argv[9], argv[10],
+                    argv[11], argv[12], argv[13], argv[14], argv[15], argv[16]);
+      break;
+    case 19:
+      res = BlCallObj(obj, fname, argv[0], argv[1], argv[2], argv[3], argv[4],
+                      argv[5], argv[6], argv[7], argv[8], argv[9], argv[10],
+                      argv[11], argv[12], argv[13], argv[14], argv[15],
+                      argv[16], argv[17]);
+      break;
+    case 20:
+      res = BlCallObj(obj, fname, argv[0], argv[1], argv[2], argv[3], argv[4],
+                      argv[5], argv[6], argv[7], argv[8], argv[9], argv[10],
+                      argv[11], argv[12], argv[13], argv[14], argv[15],
+                      argv[16], argv[17], argv[18]);
+      break;
+    default:
+      res = "";
+      luaL_error(L, "Lua->TS object call: Too many arguments (Max is 19)");
     }
   } else {
     switch (argc) {
-			case  0: res = BlCall(fname); break;
-			case  1: res = BlCall(fname, argv[0]); break;
-			case  2: res = BlCall(fname, argv[0], argv[1]); break;
-			case  3: res = BlCall(fname, argv[0], argv[1], argv[2]); break;
-			case  4: res = BlCall(fname, argv[0], argv[1], argv[2], argv[3]); break;
-			case  5: res = BlCall(fname, argv[0], argv[1], argv[2], argv[3], argv[4]); break;
-			case  6: res = BlCall(fname, argv[0], argv[1], argv[2], argv[3], argv[4], argv[5]); break;
-			case  7: res = BlCall(fname, argv[0], argv[1], argv[2], argv[3], argv[4], argv[5], argv[6]); break;
-			case  8: res = BlCall(fname, argv[0], argv[1], argv[2], argv[3], argv[4], argv[5], argv[6], argv[7]); break;
-			case  9: res = BlCall(fname, argv[0], argv[1], argv[2], argv[3], argv[4], argv[5], argv[6], argv[7], argv[8]); break;
-			case 10: res = BlCall(fname, argv[0], argv[1], argv[2], argv[3], argv[4], argv[5], argv[6], argv[7], argv[8], argv[9]); break;
-			case 11: res = BlCall(fname, argv[0], argv[1], argv[2], argv[3], argv[4], argv[5], argv[6], argv[7], argv[8], argv[9], argv[10]); break;
-			case 12: res = BlCall(fname, argv[0], argv[1], argv[2], argv[3], argv[4], argv[5], argv[6], argv[7], argv[8], argv[9], argv[10], argv[11]); break;
-			case 13: res = BlCall(fname, argv[0], argv[1], argv[2], argv[3], argv[4], argv[5], argv[6], argv[7], argv[8], argv[9], argv[10], argv[11], argv[12]); break;
-			case 14: res = BlCall(fname, argv[0], argv[1], argv[2], argv[3], argv[4], argv[5], argv[6], argv[7], argv[8], argv[9], argv[10], argv[11], argv[12], argv[13]); break;
-			case 15: res = BlCall(fname, argv[0], argv[1], argv[2], argv[3], argv[4], argv[5], argv[6], argv[7], argv[8], argv[9], argv[10], argv[11], argv[12], argv[13], argv[14]); break;
-			case 16: res = BlCall(fname, argv[0], argv[1], argv[2], argv[3], argv[4], argv[5], argv[6], argv[7], argv[8], argv[9], argv[10], argv[11], argv[12], argv[13], argv[14], argv[15]); break;
-			case 17: res = BlCall(fname, argv[0], argv[1], argv[2], argv[3], argv[4], argv[5], argv[6], argv[7], argv[8], argv[9], argv[10], argv[11], argv[12], argv[13], argv[14], argv[15], argv[16]); break;
-			case 18: res = BlCall(fname, argv[0], argv[1], argv[2], argv[3], argv[4], argv[5], argv[6], argv[7], argv[8], argv[9], argv[10], argv[11], argv[12], argv[13], argv[14], argv[15], argv[16], argv[17]); break;
-			case 19: res = BlCall(fname, argv[0], argv[1], argv[2], argv[3], argv[4], argv[5], argv[6], argv[7], argv[8], argv[9], argv[10], argv[11], argv[12], argv[13], argv[14], argv[15], argv[16], argv[17], argv[18]); break;
-			default: res = ""; luaL_error(L, "Lua->TS call: Too many arguments (Max is 19)");
+    case 0:
+      res = BlCall(fname);
+      break;
+    case 1:
+      res = BlCall(fname, argv[0]);
+      break;
+    case 2:
+      res = BlCall(fname, argv[0], argv[1]);
+      break;
+    case 3:
+      res = BlCall(fname, argv[0], argv[1], argv[2]);
+      break;
+    case 4:
+      res = BlCall(fname, argv[0], argv[1], argv[2], argv[3]);
+      break;
+    case 5:
+      res = BlCall(fname, argv[0], argv[1], argv[2], argv[3], argv[4]);
+      break;
+    case 6:
+      res = BlCall(fname, argv[0], argv[1], argv[2], argv[3], argv[4], argv[5]);
+      break;
+    case 7:
+      res = BlCall(fname, argv[0], argv[1], argv[2], argv[3], argv[4], argv[5],
+                   argv[6]);
+      break;
+    case 8:
+      res = BlCall(fname, argv[0], argv[1], argv[2], argv[3], argv[4], argv[5],
+                   argv[6], argv[7]);
+      break;
+    case 9:
+      res = BlCall(fname, argv[0], argv[1], argv[2], argv[3], argv[4], argv[5],
+                   argv[6], argv[7], argv[8]);
+      break;
+    case 10:
+      res = BlCall(fname, argv[0], argv[1], argv[2], argv[3], argv[4], argv[5],
+                   argv[6], argv[7], argv[8], argv[9]);
+      break;
+    case 11:
+      res = BlCall(fname, argv[0], argv[1], argv[2], argv[3], argv[4], argv[5],
+                   argv[6], argv[7], argv[8], argv[9], argv[10]);
+      break;
+    case 12:
+      res = BlCall(fname, argv[0], argv[1], argv[2], argv[3], argv[4], argv[5],
+                   argv[6], argv[7], argv[8], argv[9], argv[10], argv[11]);
+      break;
+    case 13:
+      res = BlCall(fname, argv[0], argv[1], argv[2], argv[3], argv[4], argv[5],
+                   argv[6], argv[7], argv[8], argv[9], argv[10], argv[11],
+                   argv[12]);
+      break;
+    case 14:
+      res = BlCall(fname, argv[0], argv[1], argv[2], argv[3], argv[4], argv[5],
+                   argv[6], argv[7], argv[8], argv[9], argv[10], argv[11],
+                   argv[12], argv[13]);
+      break;
+    case 15:
+      res = BlCall(fname, argv[0], argv[1], argv[2], argv[3], argv[4], argv[5],
+                   argv[6], argv[7], argv[8], argv[9], argv[10], argv[11],
+                   argv[12], argv[13], argv[14]);
+      break;
+    case 16:
+      res = BlCall(fname, argv[0], argv[1], argv[2], argv[3], argv[4], argv[5],
+                   argv[6], argv[7], argv[8], argv[9], argv[10], argv[11],
+                   argv[12], argv[13], argv[14], argv[15]);
+      break;
+    case 17:
+      res = BlCall(fname, argv[0], argv[1], argv[2], argv[3], argv[4], argv[5],
+                   argv[6], argv[7], argv[8], argv[9], argv[10], argv[11],
+                   argv[12], argv[13], argv[14], argv[15], argv[16]);
+      break;
+    case 18:
+      res = BlCall(fname, argv[0], argv[1], argv[2], argv[3], argv[4], argv[5],
+                   argv[6], argv[7], argv[8], argv[9], argv[10], argv[11],
+                   argv[12], argv[13], argv[14], argv[15], argv[16], argv[17]);
+      break;
+    case 19:
+      res = BlCall(fname, argv[0], argv[1], argv[2], argv[3], argv[4], argv[5],
+                   argv[6], argv[7], argv[8], argv[9], argv[10], argv[11],
+                   argv[12], argv[13], argv[14], argv[15], argv[16], argv[17],
+                   argv[18]);
+      break;
+    default:
+      res = "";
+      luaL_error(L, "Lua->TS call: Too many arguments (Max is 19)");
     }
   }
 
@@ -79,7 +221,8 @@ int bll_TsCall(lua_State* L, const char* oname, const char* fname, int argc, int
 // Lua lib function: ts.call
 int bll_lua_tscall(lua_State *L) {
   int argc = lua_gettop(L) - 1; // number of arguments after function name
-	if(argc < 0) return luaL_error(L, "_bllua_ts.call: Must provide a function name");
+  if (argc < 0)
+    return luaL_error(L, "_bllua_ts.call: Must provide a function name");
 
   const char *fname = luaL_checkstring(L, 1);
 
@@ -87,8 +230,11 @@ int bll_lua_tscall(lua_State* L) {
 }
 // Lua lib function: ts.callobj
 int bll_lua_tscallobj(lua_State *L) {
-	int argc = lua_gettop(L)-2; // number of arguments after function name and object?
-	if(argc < 0) return luaL_error(L, "_bllua_ts.callobj: Must provide an object and function name");
+  int argc =
+      lua_gettop(L) - 2; // number of arguments after function name and object?
+  if (argc < 0)
+    return luaL_error(
+        L, "_bllua_ts.callobj: Must provide an object and function name");
 
   const char *oname = luaL_checkstring(L, 1);
   const char *fname = luaL_checkstring(L, 2);
@@ -110,7 +256,9 @@ int bll_lua_tsgetfield(lua_State* L) {
   const char *oname = luaL_checkstring(L, 1);
   const char *vname = luaL_checkstring(L, 2);
   ADDR obj = BlObject(oname);
-	if(!obj) { return luaL_error(L, "_bllua_ts.getfield: Object not found"); }
+  if (!obj) {
+    return luaL_error(L, "_bllua_ts.getfield: Object not found");
+  }
 
   const char *val = BlGetField(obj, vname, NULL);
   lua_pushstring(L, val);
@@ -122,7 +270,9 @@ int bll_lua_tssetfield(lua_State* L) {
   const char *vname = luaL_checkstring(L, 2);
   const char *val = luaL_checkstring(L, 3);
   ADDR obj = BlObject(oname);
-	if(!obj) { return luaL_error(L, "_bllua_ts.setfield: Object not found"); }
+  if (!obj) {
+    return luaL_error(L, "_bllua_ts.setfield: Object not found");
+  }
 
   BlSetField(obj, vname, NULL, val);
   return 0;
@@ -145,15 +295,9 @@ int bll_lua_tsecho(lua_State* L) {
 }
 
 const luaL_Reg bll_lua_reg[] = {
-	{"call"    , bll_lua_tscall    },
-	{"callobj" , bll_lua_tscallobj },
-	{"getvar"  , bll_lua_tsgetvar  },
-	{"getfield", bll_lua_tsgetfield},
-	{"setfield", bll_lua_tssetfield},
-	{"eval"    , bll_lua_tseval    },
-	{"echo"    , bll_lua_tsecho    },
-	{NULL, NULL},
+    {"call", bll_lua_tscall},         {"callobj", bll_lua_tscallobj},
+    {"getvar", bll_lua_tsgetvar},     {"getfield", bll_lua_tsgetfield},
+    {"setfield", bll_lua_tssetfield}, {"eval", bll_lua_tseval},
+    {"echo", bll_lua_tsecho},         {NULL, NULL},
 };
-void llibbl_init(lua_State* L) {
-	luaL_register(L, "_bllua_ts", bll_lua_reg);
-}
+void llibbl_init(lua_State *L) { luaL_register(L, "_bllua_ts", bll_lua_reg); }

src/tsliblua.cpp

@@ -1,5 +1,9 @@
 
-// Call a Lua function from TS, return true if success - result will be on Lua stack
+// Call a Lua function from TS, return true if success - result will be on Lua
+// stack
+#include "BlFuncs.hpp"
+#include "BlHooks.hpp"
+#include "luainterp.hpp"
 bool bll_LuaCall(const char *fname, int argc, const char *argv[]) {
   lua_getglobal(gL, fname);
   for (int i = 0; i < argc; i++) {
@@ -14,9 +18,12 @@ bool bll_LuaCall(const char* fname, int argc, const char* argv[]) {
 
 // TS lib function: luacall
 const char *bll_ts_luacall(ADDR obj, int argc, const char *argv[]) {
-	if(argc<2) return "";
+  if (argc < 2)
+    return "";
 
-	if(!bll_LuaCall(argv[1], argc-2, &argv[2])) { return ""; }
+  if (!bll_LuaCall(argv[1], argc - 2, &argv[2])) {
+    return "";
+  }
 
   char *retbuf = BlReturnBuffer(BLL_ARG_MAX);
   bll_toarg(gL, retbuf, -1, false); // provide returned value to ts

src/util/libbl-support.cs

@@ -42,26 +42,4 @@ package _bllua_objectDeletionHook {
 };
 activatePackage(_bllua_objectDeletionHook);
 
-// Public Lua library for TS
-function luacall(%func, %a,%b,%c,%d,%e,%f,%g,%h,%i,%j,%k,%l,%m,%n,%o,%p) {
-	if($_bllua_active)
-		return _bllua_luacall("_bllua_call", %func, %a,%b,%c,%d,%e,%f,%g,%h,%i,%j,%k,%l,%m,%n,%o,%p);
-}
-function luaexec(%fn) {
-	if($_bllua_active)
-		return _bllua_luacall("_bllua_exec", %fn);
-}
-function luaeval(%code) {
-	if($_bllua_active)
-		return _bllua_luacall("_bllua_eval", %code);
-}
-function luaget(%name) {
-	if($_bllua_active)
-		return _bllua_luacall("_bllua_getvar", %name);
-}
-function luaset(%name, %val) {
-	if($_bllua_active)
-		_bllua_luacall("_bllua_setvar", %name, %val);
-}
-
 echo("  Executed libbl-support.cs");

src/util/libbl.lua

@@ -2,36 +2,23 @@
 -- Main lua-side functionality of bllua,
 -- provided through the global table 'bl.'
 
+-- todo: set
+
 local _bllua_ts = ts
 
 bl = bl or {}
 
--- Config
-local tsMaxArgs = 16
-local tsArgsLocal = '%a,%b,%c,%d,%e,%f,%g,%h,%i,%j,%k,%l,%m,%n,%o,%p'
-local tsArgsGlobal = 
-	'$_bllua_hook_arg1,$_bllua_hook_arg2,$_bllua_hook_arg3,$_bllua_hook_arg4,'..
-	'$_bllua_hook_arg5,$_bllua_hook_arg6,$_bllua_hook_arg7,$_bllua_hook_arg8,'..
-	'$_bllua_hook_arg9,$_bllua_hook_arg10,$_bllua_hook_arg11,$_bllua_hook_arg12,'..
-	'$_bllua_hook_arg13,$_bllua_hook_arg14,$_bllua_hook_arg15,$_bllua_hook_arg16'
-
 -- Misc
-local function ipairsNilable(t)
-	local maxk = 0
-	for k,_ in pairs(t) do
-		if k>maxk then maxk = k end
+-- Apply a function to each element in a list, building a new list from the returns
+local function map(t, f)
+  local u = {}
+  for i, v in ipairs(t) do
+    u[i] = f(v)
   end
-	local i = 0
-	return function()
-		i = i+1
-		if i>maxk then return nil
-		else return i, t[i] end
+  return u
 end
-end
-
--- Validation
 local function isValidFuncName(name)
-	return type(name)=='string' and name:find('^[a-zA-Z_][a-zA-Z0-9_]*$')
+  return type(name) == 'string' and name:find('^[a-zA-Z0-9_]+$')
 end
 local function isValidFuncNameNs(name)
   return type(name) == 'string' and (
@@ -89,7 +76,7 @@ for k,v in pairs(tsTypesByName) do
   tsTypesByNum[v] = k
 end
 
--- Type conversion from Lua to TS
+-- Type conversion
 local toTsObject
 -- Convert a string from TS into a boolean
 -- Note: Nonempty nonnumeric strings evaluate to 1, unlike in TS
@@ -113,77 +100,36 @@ local function valToTs(val)
       -- box - > 6 numbers
       return table.concat(val[1], ' ') .. ' ' .. table.concat(val[2], ' ')
     else
-			error('valToTs: cannot pass Lua tables to TorqueScript.', 3)
+      error('valToTs: could not convert table', 3)
     end
   else
-		error('valToTs: could not convert value to TorqueScript: '..tostring(val), 3)
+    error('valToTs: could not convert ' .. type(val), 3)
   end
 end
-
--- Type conversion from TS to Lua
 local fromTsForceTypes = {
-	['boolean'] = function(val) return tsBool(val) end,
-	['object'] = function(val) return toTsObject(val) end, -- wrap because toTsObject not defined yet
-	['string'] = function(val) return val end,
+  ['boolean'] = tsBool,
+  ['object'] = function(val) toTsObject(val) end, -- toTsObject not defined yet
+  ['string'] = tostring,
 }
-local function forceValFromTs(val, typ)
-	local func = fromTsForceTypes[typ]
-	if not func then error('valFromTs: invalid force type \''..typ..'\'', 4) end
-	return func(val)
-end
-local function vectorFromTs(val)
-	local xS,yS,zS = val:match('^(%-?[0-9%.e]+) (%-?[0-9%.e]+) (%-?[0-9%.e]+)$')
-	if xS then return vector{tonumber(xS),tonumber(yS),tonumber(zS)}
-	else return nil end
-end
-local function boxFromTs(val)
-	local x1S,y1S,z1S,x2S,y2S,z2S = val:match(
-		'^(%-?[0-9%.e]+) (%-?[0-9%.e]+) (%-?[0-9%.e]+) '..
-		'(%-?[0-9%.e]+) (%-?[0-9%.e]+) (%-?[0-9%.e]+)$')
-	if x1S then return {
-		vector{tonumber(x1S),tonumber(y1S),tonumber(z1S)},
-		vector{tonumber(x2S),tonumber(y2S),tonumber(z2S)} }
-	else return nil end
-end
-local function multinumericFromTs(val)
-	local tsNumPat = '%-?[0-9]+%.?[0-9]*e?[0-9]*'
-	if val:find('^'..tsNumPat) then
-		local nums = {}
-		for _,part in ipairs(val:split(' ')) do
-			if part:find('^'..tsNumPat..'$') then
-				table.insert(nums, tonumber(part))
-			else
-				return nil
-			end
-		end
-		if #nums==2 or #nums==3 then
-			return vector(nums)
-		elseif #nums==6 then -- box
-			return {
-				vector{nums[1], nums[2], nums[3]},
-				vector{nums[4], nums[5], nums[6]} }
-		elseif #nums==7 then -- axis
-			return nil --return matrix(nums)
-		else
-			return nil
-		end
-	end
+local function convertValFromTs(val, typ)
+  return fromTsForceTypes[typ](val) or
+      error('valFromTs: invalid force type ' .. typ, 4)
 end
 bl._forceType = bl._forceType or {}
--- todo: ensure name and name2 are already lowercase
-local function valFromTs(val, name, name2)
+local function valFromTs(val, name, name2) -- todo: ensure name and name2 are already lowercase
   if type(val) ~= 'string' then
-		error('valFromTs: expected string, got '..type(val), 3) end
+    error('valFromTs: expected string, got ' .. type(val), 3)
+  end
   if name then
     name = name:lower()
     if bl._forceType[name] then
-			return forceValFromTs(val, bl._forceType[name])
+      return convertValFromTs(val, bl._forceType[name])
     end
   end
   if name2 then
     name2 = name2:lower()
     if bl._forceType[name2] then
-			return forceValFromTs(val, bl._forceType[name2])
+      return convertValFromTs(val, bl._forceType[name2])
     end
   end
   -- '' -> nil
@@ -191,42 +137,46 @@ local function valFromTs(val, name, name2)
   -- number
   local num = tonumber(val)
   if num then return num end
-	-- vector, box, or axis->matrix
-	local vec = multinumericFromTs(val)
-	if vec then return vec end
-	-- net string
-	if val:sub(1,1)=='\x01' then
-		return _bllua_ts.call('getTaggedString', val)
+  -- vector
+  local xS, yS, zS = val:match('^(%-?[0-9%.e]+) (%-?[0-9%.e]+) (%-?[0-9%.e]+)$')
+  if xS then return vector { tonumber(xS), tonumber(yS), tonumber(zS) } end
+  local x1S, y1S, z1S, x2S, y2S, z2S = val:match(
+    '^(%-?[0-9%.e]+) (%-?[0-9%.e]+) (%-?[0-9%.e]+) ' ..
+    '(%-?[0-9%.e]+) (%-?[0-9%.e]+) (%-?[0-9%.e]+)$')
+  -- box (2 vectors)
+  if x1S then
+    return {
+      vector { tonumber(x1S), tonumber(y1S), tonumber(z1S) },
+      vector { tonumber(x2S), tonumber(y2S), tonumber(z2S) } }
   end
   -- string
   return val
 end
 local function arglistFromTs(name, argsS)
   local args = {}
-	for i,arg in ipairsNilable(argsS) do
+  for i, arg in ipairs(argsS) do
     args[i] = valFromTs(arg, name .. ':' .. i)
   end
   return args
 end
 local function arglistToTs(args)
-	local argsS = {}
-	for i,v in ipairsNilable(args) do
-		table.insert(argsS, valToTs(v))
-	end
-	return argsS
+  return map(args, valToTs)
 end
 local function classFromForceTypeStr(name)
   local class, rest = name:match('^([a-zA-Z0-9_]+)(::.+)$')
   if not class then
-		class, rest = name:match('^([a-zA-Z0-9_]+)(%..+)$') end
+    class, rest = name:match('^([a-zA-Z0-9_]+)(%..+)$')
+  end
   return class, rest
 end
 local setForceType
 setForceType = function(ftname, typ)
   if typ ~= nil and not fromTsForceTypes[typ] then
-		error('bl.type: invalid type \''..typ..'\'', 2) end
+    error('bl.type: invalid type \'' .. typ .. '\'', 2)
+  end
   if not isValidFuncNameNsArgn(ftname) then
-		error('bl.type: invalid function or variable name \''..ftname..'\'', 2) end
+    error('bl.type: invalid function or variable name \'' .. ftname .. '\'', 2)
+  end
 
   ftname = ftname:lower()
 
@@ -245,66 +195,48 @@ setForceType = function(ftname, typ)
 end
 bl.type = setForceType
 
--- Type conversion TS->Lua backwards, convert back to string
-local function numFromTsTostring(val)
-	-- todo: as-good-as-possible scientific notation for numbers
-	return tostring(val)
-end
-local function valFromTsTostring(val)
-	if type(val)=='string' then
-		return val
-	elseif type(val)=='number' then
-		return numFromTsTostring(val)
-	elseif type(val)=='table' then
-		if val.__is_vector then
-			local strs = {}
-			for i=1,#val do strs[i] = numFromTsTostring(val[i]) end
-			return table.concat(strs, ' ')
-		elseif val.__is_matrix then
-			-- todo: matrix back to axis-angle string
-			error('bl.string: matrix not yet supported', 3)
-		end
-	end
-	error('bl.string: cannot convert \''..type(val)..'\'', 3)
-end
 
--- Getting info from TS about functions and objects
+-- Value detection
+
 local function isTsObject(t)
   return type(t) == 'table' and t._tsObjectId ~= nil
 end
+
 local function tsIsObject(name) return tsBool(_bllua_ts.call('isObject', name)) end
 local function tsIsFunction(name) return tsBool(_bllua_ts.call('isFunction', name)) end
 local function tsIsFunctionNs(ns, name) return tsBool(_bllua_ts.call('isFunction', ns, name)) end
 local function tsIsFunctionNsname(nsname)
   local ns, name = nsname:match('^([^:]+)::([^:]+)$')
-	if ns then return tsIsFunctionNs(ns, name)
-	else return tsIsFunction(nsname) end
+  if ns then
+    return tsIsFunctionNs(ns, name)
+  else
+    return tsIsFunction(nsname)
   end
--- sanity check to make sure objects that don't isObject are always marked as ._deleted
--- can be removed later
-local function assertTsObjectExists(obj)
-	local is = tsIsObject(obj._tsObjectId)
-	if not is then
-		print('Warning: TS object :exists or isobject from lua but no longer exists') end
-	return is
 end
+
 function bl.isObject(obj)
-	if type(obj)=='number' then -- object id
-		return tsIsObject(tostring(obj))
-	elseif type(obj)=='string' then -- object name
-		return tsIsObject(obj)
-	elseif isTsObject(obj) then -- lua object
-		if obj._deleted then
-			return false
-		else
-			return assertTsObjectExists(obj)
-		end
-	else
+  if isTsObject(obj) then
+    obj = obj._tsObjectId
+  elseif type(obj) == 'number' then
+    obj = tostring(obj)
+  elseif type(obj) ~= 'string' then
     error('bl.isObject: argument #1: expected torque object, number, or string', 2)
   end
+  return tsIsObject(obj)
+end
+
+function bl.isFunction(a1, a2)
+  if type(a1) ~= 'string' then
+    error('bl.isFunction: argument #1: expected string', 2)
+  end
+  if a2 then
+    if type(a2) ~= 'string' then
+      error('bl.isFunction: argument #2: expected string', 2)
+    end
+    return tsIsFunctionNs(a1, a2)
+  else
+    return tsIsFunction(a1)
   end
-function bl.isFunction(name)
-	return tsIsFunctionNsname(name)
 end
 
 -- Torque object pseudo-class
@@ -317,9 +249,11 @@ local tsClassMeta = {
 bl._objectUserMetas = bl._objectUserMetas or {}
 function bl.class(cname, inhname)
   if not (type(cname) == 'string' and isValidFuncName(cname)) then
-		error('bl.class: argument #1: invalid class name', 2) end
+    error('bl.class: argument #1: invalid class name', 2)
+  end
   if not (inhname == nil or (type(inhname) == 'string' and isValidFuncName(inhname))) then
-		error('bl.class: argument #2: inherit name must be a string or nil', 2) end
+    error('bl.class: argument #2: inherit name must be a string or nil', 2)
+  end
   cname = cname:lower()
 
   local met = bl._objectUserMetas[cname] or {
@@ -334,15 +268,18 @@ function bl.class(cname, inhname)
     inhname = inhname:lower()
 
     local inh = bl._objectUserMetas[inhname]
-		if not inh then error('bl.class: argument #2: \''..inhname..'\' is not the '..
-			'name of an existing class', 2) end
+    if not inh then
+      error('bl.class: argument #2: \'' .. inhname .. '\' is not the ' ..
+        'name of an existing class', 2)
+    end
 
     inh._children[cname] = true
 
     local inhI = met._inherit
     if inhI and inhI ~= inh then
       error('bl.class: argument #2: class already exists and ' ..
-				'inherits a different parent.', 2) end
+        'inherits a different parent.', 2)
+    end
     met._inherit = inh
 
     -- apply inherited method and field types
@@ -354,6 +291,7 @@ function bl.class(cname, inhname)
     end
   end
 end
+
 local function objectInheritedMetas(name)
   local inh = bl._objectUserMetas[name:lower()]
   return function()
@@ -368,15 +306,18 @@ local tsObjectMeta = {
   -- Return torque member function or value
   __index = function(t, name)
     if rawget(t, '_deleted') then
-			error('ts object index: object no longer exists', 2) end
+      error('ts object index: object no longer exists', 2)
+    end
     if type(name) ~= 'string' and type(name) ~= 'number' then
-			error('ts object index: index must be a string or number', 2) end
+      error('ts object index: index must be a string or number', 2)
+    end
     if getmetatable(t)[name] then
       return getmetatable(t)[name]
     elseif type(name) == 'number' then
       if not tsIsFunctionNs(rawget(t, '_tsNamespace'), 'getObject') then
         error('ts object __index: index is number, but object does not have ' ..
-					'getObject method', 2) end
+          'getObject method', 2)
+      end
       return toTsObject(_bllua_ts.callobj(t._tsObjectId, 'getObject',
         tostring(name)))
     else
@@ -387,19 +328,17 @@ local tsObjectMeta = {
           tsIsFunctionNs(rawget(t, '_tsNamespace'), name) or
           tsIsFunctionNs(rawget(t, '_tsName'), name)
       then
-				return function(t2, ...)
-					if t2==nil or type(t2)~='table' or not t2._tsObjectId then
-						error('ts object method: be sure to use :func() not .func()', 2) end
-					local argsS = arglistToTs({...})
-					local res =
-						_bllua_ts.callobj(t2._tsObjectId, name, unpack(argsS))
-					return valFromTs(res,
-						t2._tsName and t2._tsName..'::'..name,
-						t2._tsNamespace..'::'..name)
+        return function(t, ...)
+          local args = { ... }
+          local argsS = arglistToTs(args)
+          return valFromTs(
+            _bllua_ts.callobj(rawget(t, '_tsObjectId'), name, unpack(argsS)),
+            rawget(t, '_tsName') and rawget(t, '_tsName') .. '::' .. name,
+            rawget(t, '_tsNamespace') .. '::' .. name)
         end
       else
-				local res = _bllua_ts.getfield(rawget(t,'_tsObjectId'), name)
-				return valFromTs(res,
+        return valFromTs(
+          _bllua_ts.getfield(rawget(t, '_tsObjectId'), name),
           rawget(t, '_tsName') and rawget(t, '_tsName') .. '.' .. name,
           rawget(t, '_tsNamespace') .. '.' .. name)
       end
@@ -410,9 +349,11 @@ local tsObjectMeta = {
   -- Use :set() to set Torque data
   __newindex = function(t, name, val)
     if rawget(t, '_deleted') then
-			error('ts object newindex: object no longer exists', 2) end
+      error('ts object newindex: object no longer exists', 2)
+    end
     if type(name) ~= 'string' then
-			error('ts object newindex: index must be a string', 2) end
+      error('ts object newindex: index must be a string', 2)
+    end
     rawset(t, name, val)
     -- create strong reference since it's now storing lua data
     bl._objectsStrong[rawget(t, '_tsObjectId')] = t
@@ -421,27 +362,31 @@ local tsObjectMeta = {
   -- Use to set torque data
   set = function(t, name, val)
     if t == nil or type(t) ~= 'table' or not t._tsObjectId then
-			error('ts object method: be sure to use :func() not .func()', 2) end
+      error('ts object method: be sure to use :func() not .func()', 2)
+    end
     if t._deleted then
-			error('ts object method: object no longer exists', 2) end
+      error('ts object method: object no longer exists', 2)
+    end
     if type(name) ~= 'string' then
-			error('ts object :set(): index must be a string', 2) end
+      error('ts object :set(): index must be a string', 2)
+    end
     _bllua_ts.setfield(t._tsObjectId, name, valToTs(val))
   end,
   -- __tostring: Called when printing
   -- Display a nice info string
   __tostring = function(t)
     return 'torque:' .. t._tsNamespace .. ':' .. t._tsObjectId ..
-			(t._tsName~='' and ('('..t._tsName..')') or '')..
-			(t._deleted and '(deleted)' or '')
+        (t._tsName ~= '' and ('(' .. t._tsName .. ')') or '')
   end,
   -- #object
   -- If the object has a getCount method, return its count
   __len = function(t)
     if t._deleted then
-			error('ts object __len: object no longer exists', 2) end
+      error('ts object __len: object no longer exists', 2)
+    end
     if not tsIsFunctionNs(t._tsNamespace, 'getCount') then
-			error('ts object __len: object has no getCount method', 2) end
+      error('ts object __len: object has no getCount method', 2)
+    end
     return tonumber(_bllua_ts.callobj(t._tsObjectId, 'getCount'))
   end,
   -- object:members()
@@ -449,14 +394,17 @@ local tsObjectMeta = {
   -- for index, object in group:members() do ... end
   members = function(t)
     if t == nil then
-			error('ts object method: be sure to use :func() not .func()', 2) end
+      error('ts object method: be sure to use :func() not .func()', 2)
+    end
     if t._deleted then
-			error('ts object method: object no longer exists', 2) end
+      error('ts object method: object no longer exists', 2)
+    end
     if not (
           tsIsFunctionNs(t._tsNamespace, 'getCount') and
           tsIsFunctionNs(t._tsNamespace, 'getObject')) then
       error('ts object :members() - ' ..
-				'Object does not have getCount and getObject methods', 2) end
+        'Object does not have getCount and getObject methods', 2)
+    end
     local count = tonumber(_bllua_ts.callobj(t._tsObjectId, 'getCount'))
     local idx = 0
     return function()
@@ -464,8 +412,7 @@ local tsObjectMeta = {
         local obj = toTsObject(_bllua_ts.callobj(t._tsObjectId,
           'getObject', tostring(idx)))
         idx = idx + 1
-				--return idx-1, obj
-				return obj
+        return idx - 1, obj
       else
         return nil
       end
@@ -474,23 +421,29 @@ local tsObjectMeta = {
   -- Wrap some Torque functions for performance and error checking
   getName = function(t)
     if t == nil then
-			error('ts object method: be sure to use :func() not .func()', 2) end
+      error('ts object method: be sure to use :func() not .func()', 2)
+    end
     if t._deleted then
-			error('ts object method: object no longer exists', 2) end
+      error('ts object method: object no longer exists', 2)
+    end
     return t._tsName
   end,
   getId = function(t)
     if t == nil then
-			error('ts object method: be sure to use :func() not .func()', 2) end
+      error('ts object method: be sure to use :func() not .func()', 2)
+    end
     if t._deleted then
-			error('ts object method: object no longer exists', 2) end
+      error('ts object method: object no longer exists', 2)
+    end
     return tonumber(t._tsObjectId)
   end,
   getType = function(t)
     if t == nil then
-			error('ts object method: be sure to use :func() not .func()', 2) end
+      error('ts object method: be sure to use :func() not .func()', 2)
+    end
     if t._deleted then
-			error('ts object method: object no longer exists', 2) end
+      error('ts object method: object no longer exists', 2)
+    end
     return tsTypesByNum[_bllua_ts.callobj(t._tsObjectId, 'getType')]
   end,
   ---- Schedule method for objects
@@ -512,12 +465,12 @@ local tsObjectMeta = {
   --end,
   exists = function(t)
     if t == nil then
-			error('ts object method: be sure to use :func() not .func()', 2) end
+      error('ts object method: be sure to use :func() not .func()', 2)
+    end
     if t._deleted then
       return false
-		else
-			return assertTsObjectExists(t)
     end
+    return tsIsObject(t._tsObjectId)
   end,
 }
 -- Weak-values table for caching Torque object references
@@ -546,13 +499,15 @@ end
 -- Return a Torque object for the object ID string, or create one if none exists
 toTsObject = function(idiS)
   if type(idiS) ~= 'string' then
-		error('toTsObject: input must be a string', 2) end
+    error('toTsObject: input must be a string', 2)
+  end
   idiS = idiS:lower()
   if bl._objectsWeak[idiS] then return bl._objectsWeak[idiS] end
 
-	if not tsIsObject(idiS) then
+  if not tsBool(_bllua_ts.call('isObject', idiS)) then
     --error('toTsObject: object \''..idiS..'\' does not exist', 2) end
-		return nil end
+    return nil
+  end
 
   local className = _bllua_ts.callobj(idiS, 'getClassName')
   local obj = {
@@ -572,13 +527,14 @@ end
 local function safeNamespaceName(name)
   return tostring(name:gsub(':', '_'))
 end
+local nscallArgStr = '%a,%b,%c,%d,%e,%f,%g,%h'
 bl._cachedNamespaceCalls = {}
 local function tsNamespacedCallTfname(name)
   local tfname = bl._cachedNamespaceCalls[name]
   if not tfname then
     tfname = '_bllua_nscall_' .. safeNamespaceName(name)
-		local tfcode = 'function '..tfname..'('..tsArgsLocal..'){'..
-			name..'('..tsArgsLocal..');}'
+    local tfcode = 'function ' .. tfname .. '(' .. nscallArgStr .. '){' ..
+        name .. '(' .. nscallArgStr .. ');}'
     _bllua_ts.eval(tfcode)
     bl._cachedNamespaceCalls[name] = tfname
   end
@@ -586,9 +542,9 @@ local function tsNamespacedCallTfname(name)
 end
 local function tsCallGen(name)
   return function(...)
-		local argsS = arglistToTs({...})
-		local res = _bllua_ts.call(name, unpack(argsS))
-		return valFromTs(res, name)
+    local args = { ... }
+    local argsS = arglistToTs(args)
+    return valFromTs(_bllua_ts.call(name, unpack(argsS)), name)
   end
 end
 
@@ -614,16 +570,14 @@ local tsMeta = {
         if not rest:find('::') and tsIsFunctionNs(ns, rest) then
           return tsCallGen(tsNamespacedCallTfname(name))
         else
-					local res = _bllua_ts.getvar(name)
-					return valFromTs(res, name)
+          return valFromTs(_bllua_ts.getvar(name), name)
         end
       elseif tsIsFunction(name) then
         return tsCallGen(name)
       elseif tsIsObject(name) then
         return toTsObject(name)
       else
-				local res = _bllua_ts.getvar(name)
-				return valFromTs(res, name)
+        return valFromTs(_bllua_ts.getvar(name), name)
       end
     end
   end,
@@ -641,18 +595,15 @@ function bl.call(func, ...)
   local argsS = arglistToTs(args)
   return _bllua_ts.call(func, unpack(argsS))
 end
+
 function bl.eval(code)
-	local res = _bllua_ts.eval(code)
-	return valFromTs(res)
+  return valFromTs(_bllua_ts.eval(code))
 end
+
 function bl.exec(file)
-	local res = _bllua_ts.call('exec', file)
-	return valFromTs(res)
-end
-function bl.array(name, ...)
-	local rest = {...}
-	return name..table.concat(rest, '_')
+  return valFromTs(_bllua_ts.call('exec', file))
 end
+
 function bl.boolean(val)
   return val ~= nil and
       val ~= false and
@@ -660,6 +611,7 @@ function bl.boolean(val)
       --val~='0' and
       val ~= 0
 end
+
 function bl.object(id)
   if type(id) == 'table' and id._tsObjectId then
     return id
@@ -669,63 +621,21 @@ function bl.object(id)
     error('bl.object: id must be a ts object, number, or string', 2)
   end
 end
-function bl.string(val)
-	return valFromTsTostring(val)
+
+function bl.array(name, ...)
+  local rest = { ... }
+  return name .. table.concat(rest, '_')
 end
 
--- Lua calling from TS
-local luaLookup
-luaLookup = function(tbl, name, set, val)
-	if name:find('%.') then
-		local first, rest = name:match('^([^%.:]+)%.(.+)$')
-		if not isValidFuncName(first) then
-			error('luaLookup: invalid name \''..tostring(first)..'\'', 3) end
-		if tbl[first]==nil then
-			if set then tbl[first] = {}
-			else return nil end
+function _bllua_call(fnameS, ...)
+  local args = arglistFromTs('lua:' .. fnameS:lower(), { ... }) -- todo: allow this though bl.type
+  if not _G[fnameS] then
+    error('luacall: no global lua function named \'' .. fnameS .. '\'')
   end
-		return luaLookup(tbl[first], rest, set, val)
-	elseif name:find(':') then
-		local first, rest = name:match('^([^%.:]+):(.*)$')
-		if rest:find('[%.:]') then
-			error('luacall: cannot have : or . after :', 3) end
-		if not isValidFuncName(first) then
-			error('luacall: invalid name \''..tostring(first)..'\'', 3) end
-		if not isValidFuncName(rest) then
-			error('luacall: invalid method name \''..tostring(first)..'\'', 3) end
-		if not tbl[first] then
-			error('luacall: no object named \''..rest..'\'', 3) end
-		if tbl[first][rest]==nil then
-			error('luacall: no method named \''..rest..'\'', 3) end
-		return function(...)
-			tbl[first][rest](tbl[first], ...)
-		end
-	else
-		if set then
-			tbl[name] = val
-		else
-			return tbl[name]
-		end
-	end
-end
--- Todo: similar deep access for luaget and luaset
-function _bllua_call(fname, ...)
-	local args = arglistFromTs(fname:lower(), {...}) -- todo: separate lua from ts func names?
-	local func = luaLookup(_G, fname)
-	if not func then
-		error('luacall: no global in lua named \''..fname..'\'', 2) end
-	local res = func(unpack(args))
+  -- todo: library fields and object methods
+  local res = _G[fnameS](unpack(args))
   return valToTs(res)
 end
-function _bllua_getvar(vname)
-	return valToTs(luaLookup(_G, vname))
-end
-function _bllua_setvar(vname, valS)
-	return valToTs(luaLookup(_G, vname, true, valFromTs(valS, vname))) -- todo: separate lua from ts var names?
-end
-function _bllua_eval(code) return loadstring(code)() end
-function _bllua_exec(fn) return dofile(fn, 2) end
-
 
 -- bl.schedule: Use TS's schedule function to schedule lua calls
 -- bl.schedule(time, function, args...)
@@ -753,6 +663,7 @@ function bl.schedule(time, cb, ...)
   bl._scheduleTable[id] = sch
   return sch
 end
+
 function _bllua_schedule_callback(idS)
   local id = tonumber(idS) or
       error('_bllua_schedule_callback: invalid id: ' .. tostring(idS))
@@ -771,18 +682,22 @@ function _bllua_process_cmd(cmdS, ...)
   if not func then error('_bllua_process_cmd: no cmd named \'' .. cmd .. '\'') end
   func(unpack(args)) --pcall(func, unpack(args))
 end
+
 local function addCmd(cmd, func)
   if not isValidFuncName(cmd) then
-		error('addCmd: invalid function name \''..tostring(cmd)..'\'') end
+    error('addCmd: invalid function name \'' .. tostring(cmd) .. '\'')
+  end
   bl._cmds[cmd] = func
-	_bllua_ts.eval('function '..cmd..'('..tsArgsLocal..'){'..
-		'_bllua_luacall(_bllua_process_cmd,"'..cmd..'",'..tsArgsLocal..');}')
+  local arglist = '%a,%b,%c,%d,%e,%f,%g,%h,%i,%j,%k,%l,%m,%n,%o,%p'
+  _bllua_ts.eval('function ' .. cmd .. '(' .. arglist .. '){' ..
+    '_bllua_luacall(_bllua_process_cmd,"' .. cmd .. '",' .. arglist .. ');}')
 end
 function bl.addServerCmd(name, func)
   name = name:lower()
   addCmd('servercmd' .. name, func)
   bl._forceType['servercmd' .. name .. ':1'] = 'object'
 end
+
 function bl.addClientCmd(name, func)
   name = name:lower()
   addCmd('clientcmd' .. name, func)
@@ -794,12 +709,13 @@ function bl.commandToServer(cmd, ...)
     _bllua_ts.call('addTaggedString', cmd),
     unpack(arglistToTs({ ... })))
 end
-function bl.commandToClient(client, cmd, ...)
+
+function bl.commandToClient(cmd, ...)
   _bllua_ts.call('commandToClient',
-		valToTs(client),
     _bllua_ts.call('addTaggedString', cmd),
     unpack(arglistToTs({ ... })))
 end
+
 function bl.commandToAll(cmd, ...)
   _bllua_ts.call('commandToAll',
     _bllua_ts.call('addTaggedString', cmd),
@@ -825,23 +741,29 @@ local function deactivatePackage(pkg)
     _bllua_ts.call('deactivatePackage', pkg)
   end
 end
+local hookNargs = 8
+local hookArglistLocal = '%a,%b,%c,%d,%e,%f,%g,%h'
+local hookArglistGlobal =
+'$_bllua_hook_arg1,$_bllua_hook_arg2,$_bllua_hook_arg3,$_bllua_hook_arg4,$_bllua_hook_arg5,$_bllua_hook_arg6,$_bllua_hook_arg7,$_bllua_hook_arg8'
 bl._hooks = bl._hooks or {}
 function _bllua_process_hook_before(pkgS, nameS, ...)
   local args = arglistFromTs(nameS, { ... })
   local func = bl._hooks[pkgS] and bl._hooks[pkgS][nameS] and
       bl._hooks[pkgS][nameS].before
   if not func then
-		error('_bllua_process_hook_before: no hook for '..pkgs..':'..nameS) end
+    error('_bllua_process_hook_before: no hook for ' .. pkgs .. ':' .. nameS)
+  end
   _bllua_ts.setvar('_bllua_hook_abort', '0')
   func(args) --pcall(func, args)
   if args._return then
     _bllua_ts.setvar('_bllua_hook_abort', '1')
     _bllua_ts.setvar('_bllua_hook_return', valToTs(args._return))
   end
-	for i=1,tsMaxArgs do
+  for i = 1, hookNargs do
     _bllua_ts.setvar('_bllua_hook_arg' .. i, valToTs(args[i]))
   end
 end
+
 function _bllua_process_hook_after(pkgS, nameS, resultS, ...)
   nameS = nameS:lower()
   local args = arglistFromTs(nameS, { ... })
@@ -849,15 +771,17 @@ function _bllua_process_hook_after(pkgS, nameS, resultS, ...)
   local func = bl._hooks[pkgS] and bl._hooks[pkgS][nameS] and
       bl._hooks[pkgS][nameS].after
   if not func then
-		error('_bllua_process_hook_after: no hook for '..pkgs..':'..nameS) end
+    error('_bllua_process_hook_after: no hook for ' .. pkgs .. ':' .. nameS)
+  end
   func(args) --pcall(func, args)
   return valToTs(args._return)
 end
+
 local function updateHook(pkg, name, hk)
   local beforeCode = hk.before and
       ('_bllua_luacall("_bllua_process_hook_before", "' .. pkg .. '","' .. name ..
-			'",'..tsArgsLocal..');') or ''
-	local arglist = (hk.before and tsArgsGlobal or tsArgsLocal)
+        '",' .. hookArglistLocal .. ');') or ''
+  local arglist = (hk.before and hookArglistGlobal or hookArglistLocal)
   local parentCode =
       tsIsFunctionNsname(name) and -- only call parent if it exists
       (hk.before and
@@ -870,22 +794,25 @@ local function updateHook(pkg, name, hk)
         arglist .. ');') or ''
   local code =
       'package ' .. pkg .. '{' ..
-			'function '..name..'('..tsArgsLocal..'){'..
+      'function ' .. name .. '(' .. hookArglistLocal .. '){' ..
       beforeCode .. parentCode .. afterCode ..
       '}' ..
       '};'
-	print('bl.hook eval output: [['..code..']]')
   _bllua_ts.eval(code)
 end
 function bl.hook(pkg, name, time, func)
   if not isValidFuncName(pkg) then
-		error('bl.hook: argument #1: invalid package name \''..tostring(pkg)..'\'', 2) end
+    error('bl.hook: argument #1: invalid package name \'' .. tostring(pkg) .. '\'', 2)
+  end
   if not isValidFuncNameNs(name) then
-		error('bl.hook: argument #2: invalid function name \''..tostring(name)..'\'', 2) end
+    error('bl.hook: argument #2: invalid function name \'' .. tostring(name) .. '\'', 2)
+  end
   if time ~= 'before' and time ~= 'after' then
-		error('bl.hook: argument #3: time must be \'before\' or \'after\'', 2) end
+    error('bl.hook: argument #3: time must be \'before\' or \'after\'', 2)
+  end
   if type(func) ~= 'function' then
-		error('bl.hook: argument #4: expected a function', 2) end
+    error('bl.hook: argument #4: expected a function', 2)
+  end
 
   bl._hooks[pkg]             = bl._hooks[pkg] or {}
   bl._hooks[pkg][name]       = bl._hooks[pkg][name] or {}
@@ -894,16 +821,20 @@ function bl.hook(pkg, name, time, func)
   updateHook(pkg, name, bl._hooks[pkg][name])
   activatePackage(pkg)
 end
+
 local function tableEmpty(t)
   return next(t) ~= nil
 end
 function bl.unhook(pkg, name, time)
   if not isValidFuncName(pkg) then
-		error('bl.unhook: argument #1: invalid package name \''..tostring(pkg)..'\'', 2) end
-	if name and not isValidFuncNameNs(name) then
-		error('bl.unhook: argument #2: invalid function name \''..tostring(name)..'\'', 2) end
-	if time and time~='before' and time~='after' then
-		error('bl.unhook: argument #3: time must be \'before\' or \'after\'', 2) end
+    error('bl.unhook: argument #1: invalid package name \'' .. tostring(pkg) .. '\'', 2)
+  end
+  if not isValidFuncNameNs(name) then
+    error('bl.unhook: argument #2: invalid function name \'' .. tostring(name) .. '\'', 2)
+  end
+  if time ~= 'before' and time ~= 'after' then
+    error('bl.unhook: argument #3: time must be \'before\' or \'after\'', 2)
+  end
 
   if not name then
     if bl._hooks[pkg] then
@@ -927,15 +858,13 @@ function bl.unhook(pkg, name, time)
         updateHook(pkg, name, {})
       else
         if time ~= 'before' and time ~= 'after' then
-					error('bl.unhook: argument #3: time must be nil, \'before\', or \'after\'', 2) end
-				bl._hooks[pkg][name][time] = nil
-				if tableEmpty(bl._hooks[pkg][name]) then
-					bl._hooks[pkg][name] = nil
+          error('bl.unhook: argument #3: time must be nil, \'before\', or \'after\'', 2)
         end
-				if tableEmpty(bl._hooks[pkg]) then
+        bl._hooks[pkg][name][time] = nil
+        if tableEmpty(bl._hooks[pkg][name]) and tableEmpty(bl._hooks[pkg]) then
           bl._hooks[pkg] = nil
-					updateHook(pkg, name, {})
           deactivatePackage(pkg)
+          updateHook(pkg, name, {})
         else
           updateHook(pkg, name, bl._hooks[pkg][name])
         end
@@ -950,14 +879,16 @@ end
 -- Container search/raycast
 local function vecToTs(v)
   if not isTsVector(v) then
-		error('vecToTs: argument is not a vector', 3) end
+    error('vecToTs: argument is not a vector', 3)
+  end
   return table.concat(v, ' ')
 end
 local function maskToTs(mask)
   if type(mask) == 'string' then
     local val = tsTypesByName[mask:lower()]
     if not val then
-			error('maskToTs: invalid mask \''..mask..'\'', 3) end
+      error('maskToTs: invalid mask \'' .. mask .. '\'', 3)
+    end
     return tostring(val)
   elseif type(mask) == 'table' then
     local tval = 0
@@ -967,7 +898,8 @@ local function maskToTs(mask)
         local val = tsTypesByName[v:lower()]
         if not val then
           error('maskToTs: invalid mask \'' .. v ..
-						'\' at index '..i..' in mask list', 3) end
+            '\' at index ' .. i .. ' in mask list', 3)
+        end
         tval = tval + val
         seen[v] = true
       end
@@ -991,7 +923,7 @@ function bl.raycast(start, stop, mask, ignores)
   local stopS = vecToTs(start)
   local maskS = maskToTs(mask)
   local ignoresS = {}
-	for _,v in ipairsNilable(ignores) do
+  for _, v in ipairs(ignores) do
     table.insert(ignoresS, objToTs(v))
   end
 
@@ -1009,6 +941,7 @@ function bl.raycast(start, stop, mask, ignores)
     return hit, pos, norm
   end
 end
+
 local function tsContainerSearchIterator()
   local retS = _bllua_ts.call('containerSearchNext')
   if retS == '0' then
@@ -1025,10 +958,12 @@ function bl.boxSearch(pos, size, mask)
   _bllua_ts.call('initContainerBoxSearch', posS, sizeS, maskS)
   return tsContainerSearchIterator
 end
+
 function bl.radiusSearch(pos, radius, mask)
   local posS = vecToTs(pos)
   if type(radius) ~= 'number' then
-		error('bl.radiusSearch: argument #2: radius must be a number', 2) end
+    error('bl.radiusSearch: argument #2: radius must be a number', 2)
+  end
   local radiusS = tostring(radius)
   local maskS = maskToTs(mask)
 
@@ -1040,7 +975,7 @@ end
 local maxTsArgLen = 8192
 local function valsToString(vals)
   local strs = {}
-	for i,v in ipairsNilable(vals) do
+  for i, v in ipairs(vals) do
     local tstr = table.tostring(v)
     if #tstr > maxTsArgLen then
       tstr = tostring(v)
@@ -1066,7 +1001,8 @@ local function createTsObj(keyword, class, name, inherit, props)
   if props then
     for k, v in pairs(props) do
       if not isValidFuncName(k) then
-				error('bl.new/bl.datablock: invalid property name \''..k..'\'') end
+        error('bl.new/bl.datablock: invalid property name \'' .. k .. '\'')
+      end
       table.insert(propsT, k .. '="' .. valToTs(v) .. '";')
     end
   end
@@ -1077,7 +1013,8 @@ local function createTsObj(keyword, class, name, inherit, props)
     table.concat(propsT) .. '};')
   local obj = toTsObject(objS)
   if not obj then
-		error('bl.new/bl.datablock: failed to create object', 3) end
+    error('bl.new/bl.datablock: failed to create object', 3)
+  end
 
   return obj
 end
@@ -1102,13 +1039,15 @@ local function parseTsDecl(decl)
         (inherit == nil or isValidFuncName(inherit))) then
     error('bl.new/bl.datablock: invalid decl \'' .. decl .. '\'\n' ..
       'must be of the format: \'className\', \'className name\', ' ..
-			'\'className :inherit\', or \'className name:inherit\'', 3) end
+      '\'className :inherit\', or \'className name:inherit\'', 3)
+  end
   return class, name, inherit
 end
 function bl.new(decl, props)
   local class, name, inherit = parseTsDecl(decl)
   return createTsObj('new', class, name, inherit, props)
 end
+
 function bl.datablock(decl, props)
   local class, name, inherit = parseTsDecl(decl)
   if not name then error('bl.datablock: must specify a name', 2) end

src/util/libts-lua.lua

@@ -1,4 +1,3 @@
-
 -- This Lua code provides some built-in utilities for writing Lua add-ons
 -- It is eval'd automatically once BLLua3 has loaded the TS API and environment
 -- It only has access to the sandboxed lua environment, just like user code.
@@ -7,7 +6,10 @@ ts = _bllua_ts
 
 -- Provide limited OS functions
 os = os or {}
+---@diagnostic disable-next-line: duplicate-set-field
 function os.time() return math.floor(tonumber(_bllua_ts.call('getSimTime')) / 1000) end
+
+---@diagnostic disable-next-line: duplicate-set-field
 function os.clock() return tonumber(_bllua_ts.call('getSimTime')) / 1000 end
 
 -- Virtual file class, emulating a file object as returned by io.open
@@ -82,24 +84,26 @@ local function new_file_obj(fn)
   return file
 end
 
-local function tflip(t) local u = {}; for _, n in ipairs(t) do u[n] = true end; return u; end
+local function tflip(t)
+  local u = {}; for _, n in ipairs(t) do u[n] = true end; return u;
+end
 local allowed_zip_dirs = tflip {
   'add-ons', 'base', 'config', 'saves', 'screenshots', 'shaders'
 }
 local function io_open_absolute(fn, mode)
   -- if file exists, use original mode
   local res, err = _bllua_io_open(fn, mode)
-	if res then return res
-	elseif err and not err:find('No such file or directory$') then return nil, err end
+  if res then return res end
 
   -- otherwise, if TS sees file but Lua doesn't, it must be in a zip, so use TS reader
   local dir = fn:match('^[^/]+')
-	if not allowed_zip_dirs[dir:lower()] then return nil, 'Zip is not in one of the allowed directories' end
+  if not allowed_zip_dirs[dir:lower()] then return nil, 'File is not in one of the allowed directories' end
   local exist = _bllua_ts.call('isFile', fn) == '1'
   if not exist then return nil, err end
 
   if mode ~= nil and mode ~= 'r' and mode ~= 'rb' then
-		return nil, 'Files in zips can only be opened in read mode' end
+    return nil, 'Files in zips can only be opened in read mode'
+  end
 
   -- return a temp lua file object with the data
   local fi = new_file_obj(fn)
@@ -107,6 +111,7 @@ local function io_open_absolute(fn, mode)
 end
 
 io = io or {}
+---@diagnostic disable-next-line: duplicate-set-field
 function io.open(fn, mode, errn)
   errn = errn or 1
 
@@ -127,13 +132,19 @@ function io.open(fn, mode, errn)
     return fi, err, fn
   end
 end
+
+---@diagnostic disable-next-line: duplicate-set-field
 function io.lines(fn)
-	local fi, err, fn2 = io.open(fn, nil, 2)
-	if not fi then error('Error opening file \''..fn2..'\': '..err, 2) end
+  local fi, err, fn2 = io.open(fn, 'r', 2)
+  if not fi then error('Error opening file \'' .. tostring(fn2) .. '\': ' .. tostring(err), 2) end
   return fi:lines()
 end
+
+---@diagnostic disable-next-line: duplicate-set-field
 function io.type(f)
+  ---@diagnostic disable-next-line: undefined-field
   if type(f) == 'table' and f._is_file then
+    ---@diagnostic disable-next-line: undefined-field
     return f._is_open and 'file' or 'closed file'
   else
     return _bllua_io_type(f)
@@ -145,7 +156,7 @@ function dofile(fn, errn)
   errn = errn or 1
 
   local fi, err, fn2 = io.open(fn, 'r', errn + 1)
-	if not fi then error('Error executing file \''..fn2..'\': '..err, errn+1) end
+  if not fi then error('Error executing file \'' .. tostring(fn2) .. '\': ' .. tostring(err), errn + 1) end
 
   print('Executing ' .. fn2)
   local text = fi:read('*a')
@@ -193,13 +204,23 @@ function require(mod)
   return _bllua_requiresecure(mod)
 end
 
+-- Exposure to TS
+function _bllua_getvar(name) return _G[name] end
+
+function _bllua_setvar(name, val) _G[name] = val end
+
+function _bllua_eval(code) return loadstring(code)() end
+
+function _bllua_exec(fn) return dofile(fn, 2) end
+
 local function isValidCode(code)
   local f, e = loadstring(code)
   return f ~= nil
 end
 function _bllua_smarteval(code)
   if (not code:find('^print%(')) and isValidCode('print(' .. code .. ')') then
-		code = 'print('..code..')' end
+    code = 'print(' .. code .. ')'
+  end
   local f, e = loadstring(code)
   if f then
     return f()

src/util/libts-ts.cs

@@ -49,4 +49,26 @@ function _bllua_set_var(%name, %val) {
 	return "";
 }
 
+// Public Lua library for TS
+function luacall(%func, %a,%b,%c,%d,%e,%f,%g,%h,%i,%j,%k,%l,%m,%n,%o,%p) {
+	if($_bllua_active)
+		return _bllua_luacall("_bllua_call", %func, %a,%b,%c,%d,%e,%f,%g,%h,%i,%j,%k,%l,%m,%n,%o,%p);
+}
+function luaexec(%fn) {
+	if($_bllua_active)
+		return _bllua_luacall("_bllua_exec", %fn);
+}
+function luaeval(%code) {
+	if($_bllua_active)
+		return _bllua_luacall("_bllua_eval", %code);
+}
+function luaget(%name) {
+	if($_bllua_active)
+		return _bllua_luacall("_bllua_getvar", %name);
+}
+function luaset(%name, %val) {
+	if($_bllua_active)
+		_bllua_luacall("_bllua_setvar", %name, %val);
+}
+
 echo("  Executed libts-ts.cs");

src/util/matrix.lua

@@ -1,7 +0,0 @@
-
--- todo
--- Matrix class with math operators
-
-
-
-print('  Executed matrix.lua')

src/util/std.lua

@@ -1,33 +1,24 @@
-
 -- Basic functionality that should be standard in Lua
 
 
 -- Table / List
 -- Whether a table contains no keys
 function table.empty(t)
-	return next(t)==nil
+  return next(t) ~= nil
 end
+
 -- Apply a function to each key in a table
 function table.map(f, ...)
   local ts = { ... }
   local u = {}
   for k, _ in pairs(ts[1]) do
     local args = {}
-		for j=1,#ts do args[j] = ts[j][k] end
-		u[k] = f(unpack(args))
-	end
-	return u
-end
-function table.mapk(f, ...)
-	local ts = {...}
-	local u = {}
-	for k,_ in pairs(ts[1]) do
-		local args = {}
-		for j=1,#ts do args[j] = ts[j][k] end
-		u[k] = f(k, unpack(args))
+    for j = 1, #ts do args[j] = ts[j][i] end
+    u[i] = f(unpack(args))
   end
   return u
 end
+
 function table.map_list(f, ...)
   local ts = { ... }
   local u = {}
@@ -38,28 +29,21 @@ function table.map_list(f, ...)
   end
   return u
 end
-function table.mapi_list(f, ...)
-	local ts = {...}
-	local u = {}
-	for i=1,#ts[1] do
-		local args = {}
-		for j=1,#ts do args[j] = ts[j][i] end
-		u[i] = f(i, unpack(args))
-	end
-	return u
-end
+
 -- Swap keys/values
 function table.swap(t)
   local u = {}
   for k, v in pairs(t) do u[v] = k end
   return u
 end
+
 -- Reverse a list
 function table.reverse(l)
   local m = {}
   for i = 1, #l do m[#l - i + 1] = l[i] end
   return m
 end
+
 -- Whether a table is a list/array (has only monotonic integer keys)
 function table.islist(t)
   local n = 0
@@ -69,6 +53,7 @@ function table.islist(t)
   end
   return n == #t
 end
+
 -- Append contents of other tables to first table
 function table.append(t, ...)
   local a = { ... }
@@ -77,6 +62,7 @@ function table.append(t, ...)
   end
   return t
 end
+
 -- Create a new table containing all keys from any number of tables
 -- latter tables in the arg list override prior ones
 -- overlaps, NOT appends, integer keys
@@ -88,6 +74,7 @@ function table.join(...)
   end
   return w
 end
+
 -- Whether a table contains a certain value in any key
 function table.contains(t, s)
   for _, v in pairs(t) do
@@ -95,29 +82,34 @@ function table.contains(t,s)
   end
   return false
 end
+
 function table.contains_list(t, s)
   for _, v in ipairs(t) do
     if v == s then return true end
   end
   return false
 end
+
 -- Copy a table to another table
 function table.copy(t)
   local u = {}
   for k, v in pairs(t) do u[k] = v end
   return u
 end
+
 function table.copy_list(l)
   local m = {}
   for i, v in ipairs(l) do m[i] = v end
   return m
 end
+
 -- Sort a table in a new copy
 function table.sortcopy(t, f)
   local u = table.copy_list(t)
   table.sort(u, f)
   return u
 end
+
 -- Remove a value from a table
 function table.removevalue(t, r)
   local rem = {}
@@ -126,6 +118,7 @@ function table.removevalue(t, r)
   end
   for _, k in ipairs(rem) do t[k] = nil end
 end
+
 function table.removevalue_list(t, r)
   for i = #t, 1, -1 do
     if t[i] == r then
@@ -133,6 +126,7 @@ function table.removevalue_list(t, r)
     end
   end
 end
+
 -- Export tables into formatted executable strings
 local function tabs(tabLevel)
   return ('  '):rep(tabLevel)
@@ -199,14 +193,13 @@ valueToString = function(v, tabLevel, seen)
     return tostring(v)
   else
     --error('table.tostring: table contains a '..t..' value, cannot serialize')
-		return 'nil --[[ '..tostring(v)..' ]]'
+    return 'nil --[[ cannot serialize ' .. t .. ': ' .. tostring(v) .. ' ]]'
   end
 end
 function table.tostring(t)
   return tableToString(t, 0, {})
 end
 
-
 -- String
 
 -- Split string into table by separator
@@ -214,7 +207,8 @@ end
 -- if regex is not true, sep is treated as a regex pattern
 function string.split(str, sep, noregex)
   if type(str) ~= 'string' then
-		error('string.split: argument #1: expected string, got '..type(str), 2) end
+    error('string.split: argument #1: expected string, got ' .. type(str), 2)
+  end
   if sep == nil or sep == '' then
     local t = {}
     local ns = #str
@@ -240,6 +234,7 @@ function string.split(str, sep, noregex)
       'string.split: argument #2: expected string or nil, got ' .. type(sep), 2)
   end
 end
+
 -- Split string to a list of char bytes
 function string.bytes(s)
   local b = {}
@@ -249,11 +244,13 @@ function string.bytes(s)
   end
   return b
 end
+
 -- Trim leading and trailing whitespace
 function string.trim(s, ws)
   ws = ws or ' \t\r\n'
   return s:gsub('^[' .. ws .. ']+', ''):gsub('[' .. ws .. ']+$', '') .. ''
 end
+
 -- String slicing and searching using [] operator
 local str_meta = getmetatable('')
 local str_meta_index_old = str_meta.__index
@@ -269,15 +266,20 @@ function str_meta.__index(s,k)
     return string.sub(s, a, b)
   end
 end
+
 -- String iterator
 function string.chars(s)
   local i = 0
   return function()
     i = i + 1
-		if i<=#s then return s:sub(i,i)
-		else return nil end
+    if i <= #s then
+      return s:sub(i, i)
+    else
+      return nil
     end
   end
+end
+
 -- Escape sequences
 local defaultEscapes = {
   ['\\'] = '\\\\',
@@ -297,6 +299,7 @@ function string.escape(s, escapes)
   end
   return table.concat(t)
 end
+
 local defaultEscapeChar = '\\'
 local defaultUnescapes = {
   ['\\'] = '\\',
@@ -327,7 +330,6 @@ function string.unescape(s, escapeChar, unescapes)
   return table.concat(t)
 end
 
-
 -- IO
 
 io = io or {}
@@ -339,6 +341,7 @@ function io.readall(filename)
   fi:close()
   return s
 end
+
 -- Write data to file all at once, return true if success / false,err if failure
 function io.writeall(filename, data)
   local fi, err = io.open(filename, 'wb')
@@ -348,23 +351,24 @@ function io.writeall(filename, data)
   return true, nil
 end
 
-
 -- Math
 
 -- Round
 function math.round(x)
   return math.floor(x + 0.5)
 end
+
 -- Mod that accounts for floating point inaccuracy
 function math.mod(a, b)
   local m = a % b
-	if m==0 or math.abs(m)<1e-15 or math.abs(m-b)<1e-15 then return 0
-	else return m end
+  if m == 0 or math.abs(m) < 1e-15 or math.abs(m - b) < 1e-15 then
+    return 0
+  else
+    return m
   end
+end
+
 -- Clamp value between min and max
 function math.clamp(v, n, x)
   return math.min(x, math.max(v, n))
 end
-
-
-print('  Executed std.lua')

src/util/vector.lua

@@ -1,18 +1,19 @@
-
 -- Vector math class with operators
 local vector_meta
 local vector_new
 local function vector_check(v, n, name, argn)
   if not v.__is_vector then
     error('vector ' .. name .. ': argument #' .. (argn or 1)
-			..': expected vector, got '..type(v), n+1) end
+      .. ': expected vector, got ' .. type(v), n + 1)
+  end
 end
 local function vector_checksamelen(v1, v2, name)
   vector_check(v1, 3, name, 1)
   vector_check(v2, 3, name, 2)
   if #v1 ~= #v2 then
     error('vector ' .. name .. ': vector lengths do not match (lengths are '
-			..#v1..' and '..#v2..')', 3) end
+      .. #v1 .. ' and ' .. #v2 .. ')', 3)
+  end
   return #v1
 end
 local function vector_checklen(v1, v2, name, len)
@@ -20,7 +21,8 @@ local function vector_checklen(v1, v2, name, len)
   vector_check(v2, 3, name, 2)
   if #v1 ~= len or #v2 ~= len then
     error('vector ' .. name .. ': vector lengths are not ' .. len .. ' (lengths are '
-			..#v1..' and '..#v2..')', 3) end
+      .. #v1 .. ' and ' .. #v2 .. ')', 3)
+  end
 end
 local function vector_opnnn(name, op)
   return function(v1, v2)
@@ -69,15 +71,21 @@ local vector_indices = {x = 1, y = 2, z = 3, w = 4, r = 1, g = 2, b = 3, a = 4}
 local vector_meta = {
   __is_vector     = true,
   __index         = function(t, k)
-		if tonumber(k) then return rawget(t, k)
-		elseif vector_indices[k] then return rawget(t, vector_indices[k])
-		else return getmetatable(t)[k]
+    if tonumber(k) then
+      return rawget(t, k)
+    elseif vector_indices[k] then
+      return rawget(t, vector_indices[k])
+    else
+      return getmetatable(t)[k]
     end
   end,
   __newindex      = function(t, k, v)
-		if tonumber(k) then rawset(t, k, v)
-		elseif vector_indices[k] then rawset(t, vector_indices[k], v)
-		else return
+    if tonumber(k) then
+      rawset(t, k, v)
+    elseif vector_indices[k] then
+      rawset(t, vector_indices[k], v)
+    else
+      return
     end
   end,
   __add           = vector_opnnn('add', function(x1, x2) return x1 + x2 end),
@@ -115,8 +123,11 @@ local vector_meta = {
     local len = #v1
     local v3 = {}
     for i = 1, len do
-			if length==0 then v3[i] = 0
-			else v3[i] = v1[i]/length end
+      if length == 0 then
+        v3[i] = 0
+      else
+        v3[i] = v1[i] / length
+      end
     end
     return vector_new(v3)
   end,
@@ -152,20 +163,28 @@ local vector_meta = {
   rotateByAngleId = function(v1, r)
     --vector_check(v1, 2, 'rotate')
     if type(r) ~= 'number' or r % 1 ~= 0 then
-			error('vector rotateByAngleId: invalid rotation '..tostring(r), 2) end
+      error('vector rotateByAngleId: invalid rotation ' .. tostring(r), 2)
+    end
     r = r % 4
     local v2
-		if     r==0 then v2 = vector_new{  v1[1],  v1[2], v1[3] }
-		elseif r==1 then v2 = vector_new{  v1[2], -v1[1], v1[3] }
-		elseif r==2 then v2 = vector_new{ -v1[1], -v1[2], v1[3] }
-		elseif r==3 then v2 = vector_new{ -v1[2],  v1[1], v1[3] }
-		else error('vector rotateByAngleId: invalid rotation '..r, 2) end
+    if r == 0 then
+      v2 = vector_new { v1[1], v1[2], v1[3] }
+    elseif r == 1 then
+      v2 = vector_new { v1[2], -v1[1], v1[3] }
+    elseif r == 2 then
+      v2 = vector_new { -v1[1], -v1[2], v1[3] }
+    elseif r == 3 then
+      v2 = vector_new { -v1[2], v1[1], v1[3] }
+    else
+      error('vector rotateByAngleId: invalid rotation ' .. r, 2)
+    end
     return v2
   end,
   rotateZ         = function(v, r)
     --vector_check(v, 2, 'rotate2d')
     if type(r) ~= 'number' then
-			error('vector rotateZ: invalid rotation '..tostring(r), 2) end
+      error('vector rotateZ: invalid rotation ' .. tostring(r), 2)
+    end
     local len = math.sqrt(v[1] ^ 2 + v[2] ^ 2)
     local ang = math.atan2(v[2], v[1]) + r
     local v2 = vector_new { math.cos(ang) * len, math.sin(ang) * len }
@@ -216,7 +235,4 @@ vector_new = function(vi)
 end
 
 vector = vector_new
-
-print('  Executed vector.lua')
-
 return vector_new