I've decided to do away with the old method NewEngine used to check contents of strings.
Since the start of development, string checking has mainly been done through the use of checkString.bat.
When called, checkString.bat takes a string as an input, and changes the following variables based on the contents of the inputted string:
stringHasEqual stringHasSemiColon stringHasSpace stringHasQuotationMark stringHasTab stringHasSymbols stringHasLetters stringHasNumbers stringHasCustom stringIsSafe stringHasUnwantedChars
This system is terrible for many reasons.
For starters, it creates a lot of persistent variables which, for optimal memory usage, will need to manually be unloaded once they're no longer needed.
Another reason why this system sucks, is because it's incredibly ugly, have a look:
I've decided to move on from this script for good. From now on, anytime a script need to check string contents, It'll simply be hardcoded, such as this:
In addition to not being janky, hardcoding is (unsurprisingly) faster than calling to checkString.bat during data load.
I'm still converting to the new error logging system, which means looking over hundreds of exits and calls.