... which once used to be exclusively used for string helper functions.

Much nicer than to have the caller allocate it and then delete it if it's not needed.

Forwards compatible detours! If only thvorbis had thought that far...

Doesn't change anything yet, but prepares the ground for detour chaining.

(Note that calls to i...

Will be necessary for the upcoming detour chaining system.

Such a list of all function pointers across all loaded DLLs can be used for a lot more than what...

If the target process terminates using ExitProcess(), any active threads will have most likely a...

Well, if Windows gives us a way to avoid race conditions, we should make use of it, especially s...

This can be used to selectively disable certain binary hacks if they would crash a particular va...

More consistent with the general function naming.

This is to guarantee that any calls to GetTextExtent() return the correct widths.

Also, moving ...

This enables us to define a tabstop while simultaneously centering or right-aligning the text in...

I've been thinking for a long time about how to reduce the redundancy in all these wrappers, and...

Since SelectObject() doesn't crash when handing it a null pointer, we don't necessarily have to ...

I must have been terribly asleep when I wrote this. :(

Old code, and it turns out that this would indeed cause crashes if BP_file_loaded() was called a...

Well yeah, it *did* work everywhere, but the results were not exactly pretty, and barely legible...

They have never been used, will probably never be used (we have binary hacks, you know), and wit...

We've been using a __stdcall build of that library all the time anyway, and it turned out that w...

In a time without both global.js and the slightest idea of how popular the engine (not the patch...

This removes the final instance of code duplication between stack resolvers!

Previously, game-specific settings from patches at a lower position in the stack would overwrite...

Previously, the sequence of generic and build-specific files had to be hardcoded into every sing...

Memory leaks are dumped to the on-screen console just fine without them.

What the hell?!? Dialog patching in th06, th07 and parts of th08 was completely broken as a resu...

Should have done this before changing the CSV parser on the server!

Luckily, this is thcrap and there is still the possibility for nifty workarounds to deal with th...

If any of the first two string parameters include one, the game would otherwise get the wrong st...

Required for Danmakufu. And yes, it works, even through UPX. I can't believe it either.

Fixes a crash that previously occurred in this case. This is particularly likely to happen with ...

Third attempt at patching ruby annotations. At a cost of requiring 6 values per game, we finally...

This can be used to define general run configuration elements shared by all games.

... really, we should get rid of that run_cfg global.

... oh wait, it *is* easy to check whether that update is installed.

Now orthogonal to binhacks_apply().

This requires the list of exported functions to be stored in the run configuration beforehand - ...

And yes, we're going to use this for multi-address breakpoints, too.

"Hackpoints". What a brill...

Suddenly, even my most hated function in this code is looking so clean!

This gets rid of all the JSON dictionary lookups in breakpoint_process(), and makes everything a...

I should keep a tally of the instances where pointer arithmetic has actually made the code clear...

Another cause for DLL injection failure, and hopefully the last one. The LOAD_LIBRARY_SEARCH_DLL...

Time to get rid of this ancient wart.

Well, of COURSE we can't expect ZUN to apply BASIC PROGRAMMING PRINCIPLES to check the return va...

What a fucking mess this is.

One of the few last fixed char arrays, and it was susceptible to a buffer overflow. Hooray.

......

Avoids the misleading log message that was previously printed in that case.

Yup, we'd like to go multi-protocol.