Sound refactor backport

Closes #22

See merge request KartKrew/Kart-Public!325

Prevent multiple nodes fopen-ing the same file

See merge request KartKrew/Kart-Public!326

Makes it act similarly to before if only 1 node needs the file.

Instead of FILE pointer stored in the transfer, they go in their own array. The number of nodes accessing a file is kept track of, and it's only fully closed after everyone who needs it is done with it.

Means that will unfortunately do a lot more seeking, but multiple users can download the same file now without it being implementation specific.

Increase range of anti-grief cvar

Closes #20

See merge request KartKrew/Kart-Public!327

Consistently free G_BuildMapTitle

See merge request KartKrew/Kart-Public!323

James R. authored 2 years ago and toaster committed 2 years ago

Previously stopped only one, probably the first that
started playing.

Minor refactor: Handle *sep entirely within S_AdjustSoundParams, instead of having a component partly outside.

Fix gamepad axis brake behaviour

See merge request KartKrew/Kart-Public!324

- It was actually ACCELERATION axis input that was predominantly broken!
    - It was inexplicably slightly under double what it should be (98 instead of the digital value of 50), which meant brake analog input struggled to dampen it.
    - Analog brake deceleration also gets a slight buff, from 24 to 25 (previously off from the digital value of 25 due to integer division rounding)
- Remove a pointless, always-true condition for brake handling.
    - Checked whether accel was held down OR whether `cmd->forwardmove` was less than or equal 0...
    - But further up the same function, `cmd` was invariably overwritten with a blank `I_BaseTiccmd`!
    - Therefore, `cmd->forwardmove` would always equal 0, and the `forwardmove` subtraction would always occur.

Use SDL version of executable icon at runtime on macOS

See merge request KartKrew/Kart-Public!321

Fix game trying to load nothing when music and sound files are not present.

See merge request KartKrew/Kart-Public!322

So that it actually falls back to direct downloading

In cases of where the file mismatches from what the server expects, it will fall back to direct downloading of the file.

- Fixed off by one in Got_RequestFilePak that could cause correct, maximised-space-usage packets to be rejected
- More verbose printing for aborting send files
- More verbose printing for client request files (behind a define for troubleshooting)

Skips resync, clearticcmd, and sendtics, since all they're doing with no nodes in game is writing the same value to the same address again and again

There's enough confidence among krew that there's zero apparent downside to this, but made sure to do it as a seperate commit if we have to hit the emergency switch and revert

Same rules as thunder shield - the SPBis coming out again ASAP.

- If no clients at server start or after 10 seconds of GS_LEVEL, and no Netxcmd waiting to be digested, halt all SV_MakeTic.
- Currently #define'd out, but if we don't get to test it before 1.6 release, I fully encourage community build developers to enable this codepath and trial it on their servers.
- It's absolutely netsafe to only have enabled on the host's end, the only risk is that a dedicated server might not re-awaken when presented with certain stimuli.

Still cope compared to the MP addfile codepath, but not the n^2 time cope of before.

Tested with a full 255 WAD server, it's a LITTLE silly but I think it's inexplicably safe to ship!?

- Make all the I_Errors return false and print to the console instead.
- New prints for missing files if you can't fit it all into one packet.
- Make the startmessage warning less specific and direct you to the logfile, to accomodate all the different ways legacy downloads can fail.