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

src/bllua4.cpp

@@ -2,12 +2,15 @@
 
 // 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"
@@ -61,7 +64,9 @@ bool init() {
 #endif
 
   // Expose Lua API to TS
-	BlAddFunction(NULL, NULL, "_bllua_luacall", bll_ts_luacall, "LuaCall(name, ...) - Call Lua function and return result", 2, 20);
+  BlAddFunction(NULL, NULL, "_bllua_luacall", bll_ts_luacall,
+                "LuaCall(name, ...) - Call Lua function and return result", 2,
+                20);
   BlEval(bll_fileTsEnv);
 
   // Load utilities
@@ -98,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
@@ -104,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)
@@ -128,6 +131,7 @@ function _bllua_requiresecure(name)
     return old_require(name)
   end
 end
+
 package = {
   seeall = old_package.seeall,
 }

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.lua

@@ -118,7 +118,8 @@ end
 bl._forceType = bl._forceType or {}
 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
@@ -143,9 +144,11 @@ local function valFromTs(val, name, name2) -- todo: ensure name and name2 are al
     '^(%-?[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 {
+  if x1S then
+    return {
       vector { tonumber(x1S), tonumber(y1S), tonumber(z1S) },
-		vector{tonumber(x2S),tonumber(y2S),tonumber(z2S)} } end
+      vector { tonumber(x2S), tonumber(y2S), tonumber(z2S) } }
+  end
   -- string
   return val
 end
@@ -162,15 +165,18 @@ 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()
 
@@ -201,8 +207,11 @@ local function tsIsFunction(name) return tsBool(_bllua_ts.call('isFunction', nam
 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
 end
 
 function bl.isObject(obj)
@@ -215,12 +224,15 @@ function bl.isObject(obj)
   end
   return tsIsObject(obj)
 end
+
 function bl.isFunction(a1, a2)
   if type(a1) ~= 'string' then
-		error('bl.isFunction: argument #1: expected string', 2) end
+    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
+      error('bl.isFunction: argument #2: expected string', 2)
+    end
     return tsIsFunctionNs(a1, a2)
   else
     return tsIsFunction(a1)
@@ -237,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 {
@@ -254,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
@@ -274,6 +291,7 @@ function bl.class(cname, inhname)
     end
   end
 end
+
 local function objectInheritedMetas(name)
   local inh = bl._objectUserMetas[name:lower()]
   return function()
@@ -288,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
@@ -328,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
@@ -339,11 +362,14 @@ 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
@@ -356,9 +382,11 @@ local tsObjectMeta = {
   -- 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()
@@ -366,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()
@@ -390,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
@@ -428,9 +465,11 @@ 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 end
+      return false
+    end
     return tsIsObject(t._tsObjectId)
   end,
 }
@@ -460,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 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 = {
@@ -554,12 +595,15 @@ function bl.call(func, ...)
   local argsS = arglistToTs(args)
   return _bllua_ts.call(func, unpack(argsS))
 end
+
 function bl.eval(code)
   return valFromTs(_bllua_ts.eval(code))
 end
+
 function bl.exec(file)
   return valFromTs(_bllua_ts.call('exec', file))
 end
+
 function bl.boolean(val)
   return val ~= nil and
       val ~= false and
@@ -567,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
@@ -576,14 +621,17 @@ function bl.object(id)
     error('bl.object: id must be a ts object, number, or string', 2)
   end
 end
+
 function bl.array(name, ...)
   local rest = { ... }
   return name .. table.concat(rest, '_')
 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
+    error('luacall: no global lua function named \'' .. fnameS .. '\'')
+  end
   -- todo: library fields and object methods
   local res = _G[fnameS](unpack(args))
   return valToTs(res)
@@ -615,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))
@@ -633,9 +682,11 @@ 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
   local arglist = '%a,%b,%c,%d,%e,%f,%g,%h,%i,%j,%k,%l,%m,%n,%o,%p'
   _bllua_ts.eval('function ' .. cmd .. '(' .. arglist .. '){' ..
@@ -646,6 +697,7 @@ function bl.addServerCmd(name, func)
   addCmd('servercmd' .. name, func)
   bl._forceType['servercmd' .. name .. ':1'] = 'object'
 end
+
 function bl.addClientCmd(name, func)
   name = name:lower()
   addCmd('clientcmd' .. name, func)
@@ -657,11 +709,13 @@ function bl.commandToServer(cmd, ...)
     _bllua_ts.call('addTaggedString', cmd),
     unpack(arglistToTs({ ... })))
 end
+
 function bl.commandToClient(cmd, ...)
   _bllua_ts.call('commandToClient',
     _bllua_ts.call('addTaggedString', cmd),
     unpack(arglistToTs({ ... })))
 end
+
 function bl.commandToAll(cmd, ...)
   _bllua_ts.call('commandToAll',
     _bllua_ts.call('addTaggedString', cmd),
@@ -689,14 +743,16 @@ local function deactivatePackage(pkg)
 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'
+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
@@ -707,6 +763,7 @@ function _bllua_process_hook_before(pkgS, nameS, ...)
     _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, { ... })
@@ -714,10 +771,12 @@ 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 ..
@@ -743,13 +802,17 @@ local function updateHook(pkg, name, hk)
 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 {}
@@ -758,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
+    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
+    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
+    error('bl.unhook: argument #3: time must be \'before\' or \'after\'', 2)
+  end
 
   if not name then
     if bl._hooks[pkg] then
@@ -791,7 +858,8 @@ 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
+          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]) and tableEmpty(bl._hooks[pkg]) then
           bl._hooks[pkg] = nil
@@ -811,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
@@ -828,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
@@ -870,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
@@ -886,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)
 
@@ -927,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
@@ -938,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
@@ -963,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,7 +84,9 @@ 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'
 }
@@ -98,7 +102,8 @@ local function io_open_absolute(fn, mode)
   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)
@@ -106,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
 
@@ -126,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)
@@ -144,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')
@@ -194,8 +206,11 @@ 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)
@@ -204,7 +219,8 @@ local function isValidCode(code)
 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/std.lua

@@ -1,4 +1,3 @@
-
 -- Basic functionality that should be standard in Lua
 
 
@@ -7,6 +6,7 @@
 function table.empty(t)
   return next(t) ~= nil
 end
+
 -- Apply a function to each key in a table
 function table.map(f, ...)
   local ts = { ... }
@@ -18,6 +18,7 @@ function table.map(f, ...)
   end
   return u
 end
+
 function table.map_list(f, ...)
   local ts = { ... }
   local u = {}
@@ -28,18 +29,21 @@ function table.map_list(f, ...)
   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
@@ -49,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 = { ... }
@@ -57,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
@@ -68,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
@@ -75,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 = {}
@@ -106,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
@@ -113,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)
@@ -186,7 +200,6 @@ function table.tostring(t)
   return tableToString(t, 0, {})
 end
 
-
 -- String
 
 -- Split string into table by separator
@@ -194,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
@@ -220,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 = {}
@@ -229,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
@@ -249,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 = {
   ['\\'] = '\\\\',
@@ -277,6 +299,7 @@ function string.escape(s, escapes)
   end
   return table.concat(t)
 end
+
 local defaultEscapeChar = '\\'
 local defaultUnescapes = {
   ['\\'] = '\\',
@@ -307,7 +330,6 @@ function string.unescape(s, escapeChar, unescapes)
   return table.concat(t)
 end
 
-
 -- IO
 
 io = io or {}
@@ -319,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')
@@ -328,19 +351,23 @@ 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))

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 }