Age | Commit message (Collapse) | Author |
|
The Windows API has an _snprintf function that is not the same as
Unix's snprintf(): if the string is truncated then no trailing NUL
character is appended. This makes the function unsafe. Define a
replacement/wrapper called M_snprintf that works the same but always
appends a trailing NUL, for safety on Windows and other OSes that
behave like this.
Do the same thing for vsnprintf(), and update HACKING to list
snprintf/vsnprintf as forbidden functions. This fixes #375;
thanks to Quasar for pointing out the different behavior of these
functions.
|
|
Use snprintf() or other functions in place of sprintf(). This is part
of fixing #371.
|
|
As part of this, add DIR_SEPARATOR_S as a string version of the
DIR_SEPARATOR macro. Change M_TempFile() to return a string allocated
on the C heap rather than the zone heap.
This is a first step towards fixing #371.
|
|
Vanilla Hexen makes you specify the demo name to play by giving
the plain lump name, eg. heretic -playdemo mydemo to load mydemo.lmp.
It doesn't work if you specify the extension or the full file path.
As a convenience and to match the behavior of Chocolate Doom, allow
paths and extensions.
Also rework the code for other games so that they're slightly more
consistent. This fixes #301.
|
|
Eliminate use of strcpy, strcat, strncpy, and use the new safe
alternatives.
|
|
Just as M_StringCopy behaves the same as strlcpy(), M_StringConcat
behaves the same as strlcat(). Use this in one location.
|
|
It's more readable to write "M_StringEndsWith(..." than doing a bunch of
pointer arithmetic, and this is a common pattern. Also add
M_StringStartsWith, M_StringJoin and M_StringCopy. The latter is a
safe alternative for strcpy() that works the same as OpenBSD's
strlcpy(). Use these functions in a few places where it makes sense.
|
|
commit 42faefce1fd03 added code to set the GUS patch path
automatically when the BFG Edition is installed, but this caused
problems because gusconf.c is not included as part of the build for
the setup tool. Use D_SetVariable() instead which accomplishes the
same thing without a hard dependency.
|
|
|
|
Initialize low-level sound startup (calls to I_InitSound) separately
from the high-level sound startup (S_Init). In particular, make sure
that SDL sound is initialized before the textscreen multiplayer
waiting screen is shown. This is an attempt to fix a bug with sound in
multiplayer games on Windows; calling SDL_QuitSubSystem(SDL_INIT_VIDEO)
(on closedown of the textscreen library) causes subsequent attempts to
initialize audio to fail.
Big thanks go to Alexandre-Xavier (AXDOOMER) for working out a fix for
this bug and James Haley (Quasar) for giving some technical background
about the lack of separation between SDL subsystems. This (hopefully)
fixes #270.
|
|
In #364 I recommended -1 as the default to avoid taking screenshots in
some situations where a key event is posted for key 0. It turns out
that wasn't such a great idea, as -1 is not parsed properly by the
config parser, and is shown as ??-1 in the setup tool. Default to zero
instead, and add guards to prevent screenshots being taken by mistake.
Thanks to Alexandre-Xavier for pointing this out.
|
|
Add -gameversion options for v1.666, v1.7, v1.8. Set demo and savegame
headers appropriately depending on version. Also add an enum entry for
Doom v1.2 but no code support for it yet.
|
|
A deathmatch player spawning facing directly west does not see the
normal teleport fog (or hear it). This is because of an angle that
should be unsigned but is actually signed. Import PrBoom's code to
emulate this bug. This fixes #186.
|
|
Print a warning if the user is attempting to record or playback a demo o...
|
|
|
|
connect to a network game using one of the BFG Edition IWADs.
|
|
|
|
|
|
Ultimate Doom's doom.exe will crash when run with an IWAD that doesn't
contain M_EPI4 (like the stock v1.9 IWAD).
Thanks to Alexandre Xavier for the bug report (fixes #344).
|
|
Lots of otherwise Vanilla-compatible WADs contain DEHACKED lumps.
Allow these to be loaded by adding a -dehlump command line parameter.
Thanks to Fabian Greffrath for the suggestion (fixes #349).
|
|
Use "ON" to mean "high detail", "OFF" for "low detail". The logic was
backwards.
|
|
|
|
1) Move the check for (bfgedition) right behind loading the IWAD, i.e.
before any PWADs are loaded that could probably provide a DMENUPIC lump.
2) Instead of checking for a missing TITLEPIC lump (which is only true
for the doom2.wad shipped with the BFG Edition) check for the presence
of DMENUPIC (which is exclusive to both classic IWADs shipped with the
BFG Edition). The M_GDHIGH lumps, however, are incompatibly modified
in *both* IWADs.
3) Move the check for the missing TITLEPIC lump to the place where it
becomes actually crucial, i.e. D_DoAdvanceDemo() and make it independent
of the (bfgedition) check. So, PWADs still have a chance to provide their
own TITLEPIC lump.
|
|
The BFG edition IWADs have their M_GDHIGH lumps changed to say
"Fullscreen:" instead of just "high". This breaks the options menu
which assumes a graphic that is not as wide.
In the same spirit as the title screen workaround, add a workaround
for this as well: use the message "on/off" graphics instead when we
are running using a BFG edition IWAD.
This fixes #341 (thanks Fabian Greffrath).
|
|
Freedoom has new IWAD names. Support these as fallbacks if no other
IWAD file can be found, but prefer FreeDM as it has Vanilla-
compatible levels. Show a warning on startup if running using the
main Freedoom IWADs.
|
|
Fabian Greffrath).
Subversion-branch: /branches/v2-branch
Subversion-revision: 2747
|
|
Subversion-branch: /branches/v2-branch
Subversion-revision: 2742
|
|
menu (thanks Alexandre Xavier).
Subversion-branch: /branches/v2-branch
Subversion-revision: 2723
|
|
startup.
Subversion-branch: /branches/v2-branch
Subversion-revision: 2699
|
|
Xavier).
Subversion-branch: /branches/v2-branch
Subversion-revision: 2635
|
|
keys, which cause the same behavior.
Subversion-branch: /branches/v2-branch
Subversion-revision: 2632
|
|
pause key is pressed in a menu with a scroll-bar (thanks Alexandre
Xavier).
Subversion-branch: /branches/v2-branch
Subversion-revision: 2620
|
|
the spinal loading screen.
Subversion-branch: /branches/v2-branch
Subversion-revision: 2585
|
|
Subversion-branch: /branches/v2-branch
Subversion-revision: 2583
|
|
Subversion-branch: /branches/v2-branch
Subversion-revision: 2582
|
|
Subversion-branch: /branches/v2-branch
Subversion-revision: 2555
|
|
Subversion-branch: /branches/v2-branch
Subversion-revision: 2553
|
|
Subversion-branch: /branches/v2-branch
Subversion-revision: 2550
|
|
Subversion-branch: /branches/v2-branch
Subversion-revision: 2537
|
|
parameter, to match statdump.exe (thanks Alexandre Xavier).
Subversion-branch: /branches/v2-branch
Subversion-revision: 2479
|
|
instances of this within the code (thanks Edward-san).
Subversion-branch: /branches/v2-branch
Subversion-revision: 2468
|
|
Subversion-branch: /branches/v2-branch
Subversion-revision: 2458
|
|
Subversion-branch: /branches/v2-branch
Subversion-revision: 2452
|
|
statdump.exe does with Vanilla Doom.
Subversion-branch: /branches/v2-branch
Subversion-revision: 2450
|
|
Subversion-branch: /branches/v2-branch
Subversion-revision: 2449
|
|
its player limit to match. Update GUI code to adjust waiting screen list
size appropriately.
Subversion-branch: /branches/v2-branch
Subversion-revision: 2442
|
|
defines for each game. Refactor "waiting screen" data code.
Subversion-branch: /branches/v2-branch
Subversion-revision: 2441
|
|
the splash screen before network startup.
Subversion-branch: /branches/v2-branch
Subversion-revision: 2435
|
|
checksums, plus the is_freedoom flag.
Subversion-branch: /branches/v2-branch
Subversion-revision: 2434
|
|
Subversion-branch: /branches/v2-branch
Subversion-revision: 2428
|