Age | Commit message (Collapse) | Author |
|
This fixes the following two warnings
cast from pointer to integer of different size
cast to pointer from integer of different size
by either changing some ints that are supposed to hold pointers
to intptr_t type or by intermediate casting to (intptr_t).
|
|
When generating string representations of network addresses, include
the UDP port number if it isn't the standard port number. This is
necessary because the string version of the address is used by the
setup tool when filling in the address field; if a non-standard port
is used then it needs to be included.
Also fix byte swapping on the address portion in the same function.
Thanks to Alexandre-Xavier for the bug report on #469.
|
|
The config file API previously relied on binding config variables
using M_BindVariable() which took a void pointer. It occurred to me
that if used on a boolean variable, this would be erroneous, but the
void pointer would make it impossible to tell. Split this into
separate M_Bind{Foo}Variable() functions based on type, which allows
for proper type checking on the pointers that are passed.
Vaguely related to #509.
|
|
getenv() can return NULL if the environment variable is not set, but
the result of getenv() was always being passed to M_StringDuplicate()
without any check. This could cause crashes on some platforms.
Instead, rework the code into a first stage that gets the player's
name and a second that duplicates it into a mutable form.
This fixes #455.
|
|
With the addition of the Freedoom IWADs, the number of IWADs supported
by chocolate-doom has been raised to 10. However, the iwad_labels[]
array only holds place for up to 8 pointers. Incidently, I have all 10
IWADs installed and trying to warp into a game from
chocolate-doom-setup leads to an out-of-bounds access of this array
and so the application crashes with a segmentation fault.
Instead of increasing the array size to 10, which will bite us next
time, I decided to set its size dynamically as soon as the number of
IWADs of known.
|
|
strdup() can theoretically fail and return NULL. This could lead to
a crash or undesirable behavior. Add M_StringDuplicate() which does
the same thing but exits with an error if a string cannot be
allocated.
This fixes #456. Thanks to Quasar for the suggestion.
|
|
This change rewrites and simplifies the copyright headers at the top
of all source files:
* Remove "Emacs style mode select" line; this line was included in
the headers for the originally released source files and appears
to be to set the file type for old versions of Emacs. I'm not sure
entirely why it was required but I don't think it is any more.
* Remove "You should have received a copy of..." text from copyright
header. This refers to the old 59 Temple Place address where the
FSF headquarters used to be located and is no longer correct.
Rather than change to the new address, just remove the paragraph
as it is superfluous anyway. This fixes #311.
* Remove ---- separator lines so that the file headers are barer
and more simplified.
|
|
The table of IWAD data is stored in const memory, so make all iwad_t
pointers const to fix compiler warnings.
|
|
Once a game type (IWAD) has been selected, remember that selection
rather than resetting to the first item in the list when the window is
closed and reopened. This fixes #169 (thanks Alexandre Xavier).
|
|
Don't launch the game if a server address has not been entered; show a
message to the user instead. This fixes #121.
|
|
Use M_snprintf() or M_StringJoin() instead where appropriate.
This fixes #371.
|
|
Eliminate use of strcpy, strcat, strncpy, and use the new safe
alternatives.
|
|
Hexen: Deathkings of the Dark Citadel uses levels up to MAP60, not
MAP40 as used in hexen.wad. Bump up the limit for Hexen to MAP60 so
that it is possible to play multiplayer games with hexdd.wad loaded.
Thanks to Rhiyo for the bug report. This fixes #312.
|
|
Subversion-branch: /branches/v2-branch
Subversion-revision: 2754
|
|
Alexandre Xavier).
Subversion-branch: /branches/v2-branch
Subversion-revision: 2719
|
|
Alexandre Xavier).
Subversion-branch: /branches/v2-branch
Subversion-revision: 2619
|
|
Subversion-branch: /branches/v2-branch
Subversion-revision: 2577
|
|
Subversion-branch: /branches/v2-branch
Subversion-revision: 2569
|
|
class in Hexen mode. Add Strife skills list.
Subversion-branch: /branches/v2-branch
Subversion-revision: 2557
|
|
Subversion-branch: /branches/v2-branch
Subversion-revision: 2537
|
|
Subversion-branch: /branches/v2-branch
Subversion-revision: 2506
|
|
the setup tool (thanks Alexandre Xavier).
Subversion-branch: /branches/v2-branch
Subversion-revision: 2485
|
|
is -deathmatch and -altdeath enables Deathmatch II mode.
Subversion-branch: /branches/v2-branch
Subversion-revision: 2471
|
|
netgames, and remove code for -deathmatch parameter. Update setup tool
to remove game type dropdown when setting up a Strife game.
Subversion-branch: /branches/v2-branch
Subversion-revision: 2470
|
|
Subversion-branch: /branches/v2-branch
Subversion-revision: 2465
|
|
already in progress.
Subversion-branch: /branches/v2-branch
Subversion-revision: 2392
|
|
that we don't have the IWAD for.
Subversion-branch: /branches/v2-branch
Subversion-revision: 2389
|
|
Subversion-branch: /branches/v2-branch
Subversion-revision: 2387
|
|
Subversion-branch: /branches/v2-branch
Subversion-revision: 2384
|
|
Subversion-branch: /branches/v2-branch
Subversion-revision: 2383
|
|
interface.
Subversion-branch: /branches/v2-branch
Subversion-revision: 2382
|
|
Subversion-branch: /branches/raven-branch
Subversion-revision: 2258
|
|
Subversion-branch: /branches/raven-branch
Subversion-revision: 2214
|
|
Subversion-branch: /branches/raven-branch
Subversion-revision: 1987
|
|
Subversion-branch: /branches/raven-branch
Subversion-revision: 1737
|
|
Subversion-branch: /branches/raven-branch
Subversion-revision: 1616
|
|
Subversion-branch: /branches/raven-branch
Subversion-revision: 1523
|
|
it contains too many conflicts at present.
Subversion-branch: /branches/raven-branch
Subversion-revision: 1522
|
|
Subversion-branch: /branches/raven-branch
Subversion-revision: 1517
|
|
than the -findiwads command line hack.
Subversion-branch: /branches/raven-branch
Subversion-revision: 1516
|
|
chocolate-setup functional again.
Subversion-branch: /branches/raven-branch
Subversion-revision: 1388
|
|
code. Split out I_Endoom to separate i_endoom.c file.
Subversion-branch: /branches/raven-branch
Subversion-revision: 1384
|