Zandronum (TSPG Version)

Fork

Allow --version to work on other platforms, and make it default to stdout if passed without a value

- Fixed regression with vodoo doll players and hubs (fixes 2960).

Add '!!' in front of condition check methods.

changed sector_t::SavedLightLevel to short to match the type of sector_t::lightlevel

fixed more VC++ warnings

fixed more VC++ warnings

changed xscale, yscale in DStrifeStatusBar::DrawPopScreen back to int (this was a leftover of Skulltag's old way of scaling with floats)

fixed more VC++ warnings

- renamed g_lSavedTurnTicks to g_SavedTurnTicks and change its type to BYTE to match player_t::turnticks
- fixed VC++ warnings in cl_pred.cpp

fixed some VC++ warnings

- Fixed: the DamageFactor calculation was executed twice for armors (fixes 2978).

Fixed: ACS functions GetScreenWidth and GetScreenHeight did not work online, fixes 2847

Fixed: A_ScaleVelocity and A_ChangeVelocity did not work online, fixes 2972

- Added sv_survival_nomapresetondeath Zandronum dmflag to modify survival game mode where the map will not be reset on all players death - the players will be able to continue with fresh amount of lives. Dead players will carry over their inventory in accordance to "keep inventory" flags.
- Fixed: body of the player who loses all lives no longer disappears into thin air.
- Dead spectators will now preserve inventory so that they can be respawned with it.
- When alive players complete the map in survival, dead spectators will recover their inventory on the next map in accordance to SV_COOP_LOSE* flags

- fixed: The StripRight fix left some code from a previous fix in that no longer applies.

- Fixed online desync with A_FireBullet FBF_NOFLASH flag.

- fixed: FString::StripRight's space checking counter was broken and would cause deletion of the last valid character in the string.

- Fixed a certain number of server CVARs which weren't protected against the ConsoleCommand (fixes 2001).

- Fixed: V_ColorizeString didn't handle \x characters correctly.

- Fixed: the ClientMoveCommand cmd data wasn't initialized properly.

Spotted by Valgrind while investigating ticket 2960.

Simplified some bot code, which should also fix a compiler warning.

- Import GME fix to critical security bug.

Details at the following link: https://scarybeastsecurity.blogspot.com/2016/12/redux-compromising-linux-using-snes.html .

- Fixed the wrong handling of empty command line argument.

Fixed: FString::StripLeftRight() lost the right character when copying to a new buffer

- Also, some minor improvements to the strip functions to avoid doing
extra work.

- Fixed: the item drop physics requires full speed precision online (fixes 750 and 2522 for real).

- now that C++11 is allowed, these annoying 'narrowing conversion' warnings can finally be fixed...

- fixed: FString's 'Strip...' functions could crash on empty strings.

- fixed: The CollectSectorStack functions had a local sector_t variable used by gl_FakeFlat which caused a stack overflow. Using the global one defined at the top of the file is ok here.

Fixed: FString::ReallocBuffer could write to unallocated memory

- Previously, calling ReallocBuffer with a smaller buffer size than the
current one could overwrite unallocated memory. This required that the
string it was called on had more than one reference and therefore
required creating a new copy. The entire original string would be
copied, whether it fit in the new buffer or not.

- Shut up GCC aggressive optimizer warnings.

From what I can see, GCC would miscompile the involved loops, because the index variable is 'signed int' and the multiplication with an unsigned would cause signed overflow (undefined behavior). Change the index variable type to 'unsigned int' to expect unsigned overflow (conformant to standard).