What changed in EDuke32, build by build

Re-add something I didn't intend to remove with r7297

Remove 'none' as an option for the frame limiter in the menu and fix a bug with the slider for the offset

MSVC build fix

Fix NETCODE_DISABLED builds. We should probably rename Net_GetPackets() to something else, because it does other required things.

Fix loading screen tile

Fix issue where the player's pistol isn't removed when they spawn on HURTRAIL/FLOORSLIME/FLOORPLASMA

macOS build fix. DONT_BUILD.

Slight cleanup to MV_GetVorbisCommentLoops(). The biggest change here is swapping calls to atol to our Batol macro, which is actually backed by strtol instead of atol.

Hendricks and I discussed "auto *" and we've come to the conclusion that it's bad form, so this corrects that.

Reduce vorbis read block size from 32k to MV_MIXBUFFERSIZE. Despite requiring more calls into libvorbis, this should be faster on slower CPUs in cases where more copies of the same .ogg sound are attempting to be played at once than are allowed, e.g large explosion scenes in Ion Maiden.

I don't see a reason for these operations to be separated

Fix function that some sadist formatted with 3 spaces

This fixes S_CheckSoundPlaying() and makes CON_IFSOUND work correctly in scripts that run during menus. These changes together fix the exchange between Duke and Lani at the end of episode 3 of Duke3D.

This makes the output from r_showfps 3, cl_showcoords 1, and what is printed after DNDEBUG all happily coexist. The display of g_moveActorsTime and g_moveWorldTime has also been moved from cl_showcoords to r_showfps.

Fix CON commands leaving useless extra values in the bytecode. They were only valid outside of executable blocks so it was never noticed.

Fix CON compiler line number accounting problem introduced with the optimization that transforms CON_SETVARVAR -> CON_SETVAR if the second parameter is a constant

Change A_AddToDeleteQueue() to call A_DeleteSprite() on sprites it removes from the queue directly, as sprites with statnums other than STAT_ACTOR and STAT_MISC are not deleted automatically when their xrepeat is 0. This fixes using CON_INSERTSPRITEQ with sprites of statnums other than STAT_ACTOR and STAT_MISC.

This is never going to work. When it was first programmed, it was done under the assumption that dividing an integer by a power of 2 was always the same as shifting it. It turns out that this isn't the case and I'm a dumbass. :D

Move the debug_break() call in the default case of VM_Execute() to after the call to VM_ScriptInfo().

Fix error building the transpal utility

After some refactoring, several CON error checks done at runtime stood out as things that could be determined at script compile time. They have been moved.

Attempting to compile a CON with something stupid like "mul var 0" or "div var 0" in it now throws an error

cmenu console command now opens the menu if it isn't already open

Comment out actor and player versions of gamevar manipulation opcodes as they aren't performant with our current switch-based method of processing instructions. This also tweaks the beginning of VM_Execute() to gain a little more performance.

WIP work on fixing CON_DIVVAR/CON_MULVAR optimization

Implement individual opcodes for all the gamevar if checks and the functions that do math on vars

Global var opcodes for common conditional statements. Decent performance gain!

Change goofy check for g_errorCnt and g_warningCnt in gamedef

Use separate instructions in the bytecode for manipulating global variables versus more complex variable types. The optimization that transforms CON_DIVVAR and CON_MULVAR into CON_SHIFTVARR and CON_SHIFTVARL plus CON_INV has been disabled for now until I can debug a problem with it.

Replace pointer with reference in VM_Execute()

CON cleanup/tweaks. This improves the game update time by about 5% in some of my test scenarios.

Use reference to clean up CON_QSPRINTF

CON cleanups

Fix setvar console command, and add a line of output so you know it did something

This just shuffles some things around and these aren't (or shouldn't be) functional changes.

Set AllowShortCaseLabelsOnASingleLine to false and AlignAfterOpenBracket to Align in _clang-format

Initialize rottile values to -1 with a loop like this instead, because doing it the C++ way increases the binary size by like 120KB

This is a cool CON compiler optimization that replaces all of the "varvar" commands and conditional checks in the compiled code with the "var" version if the value supplied to the function is a constant

I'm pretty sure this is a bug

Move CON_STARTTRACK in gamedef.cpp so it isn't in between the related CON_IFXXX stuff nearby

Include build.h from player.h to satisfy the need for MAXPLAYERS and compat.h stuff that build.h pulls in

Bump MAXANGVEL and MAXHORIZ