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 !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