| Age | Commit message (Collapse) | Author |
|---|
|
There was no example for how to write function calls. Do this to
make clear that there should be no space between the function name
and the open paren. Give examples of assignments and if() conditions
as well, and tweak the for() style - there's no reason to omit spaces
around operators for the looping conditions.
|
|
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 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.
|
|
Add a section to the HACKING file listing unsafe C library functions
that should not be used.
|
|
Subversion-branch: /trunk/chocolate-doom
Subversion-revision: 2315
|
|
Subversion-branch: /trunk/chocolate-doom
Subversion-revision: 921
|
|
characters link and an introduction giving a brief explanation for the
choices.
Subversion-branch: /trunk/chocolate-doom
Subversion-revision: 703
|
|
Subversion-branch: /trunk/chocolate-doom
Subversion-revision: 632
|
|
Subversion-branch: /trunk/chocolate-doom
Subversion-revision: 631
|
