summaryrefslogtreecommitdiff
path: root/src/hexen/p_spec.c
AgeCommit message (Collapse)Author
2014-11-27hexen: audit calls to P_StartACS().Simon Howard
Scripts can have a maximum of three arguments when started (see ACS_Execute / #80 linedef special type). This means that the array of arguments passed must be at least three elements in length. When loading scripts, check the argument count never exceeds 3, and print a warning message if it does. Don't pass pointers to structure members implicitly treating them as elements of an array (as this relies on compiler-dependent behavior as to how structure members are laid out). Instead, copy values into a temporary array to make the intended behavior explicit. This fixes #477. Thanks to Quasar for reporting it.
2014-05-05Clean up file headers.Simon Howard
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.
2014-04-01Replace all snprintf() calls with M_snprintf().Simon Howard
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.
2014-03-30hexen: Eliminate use of sprintf().Simon Howard
Use snprintf() in place of sprintf(). This is part of fixing #371.
2014-03-24Fix various Clang compiler warnings.Simon Howard
2010-05-03Fix compiler warnings.Simon Howard
Subversion-branch: /branches/raven-branch Subversion-revision: 1934
2008-11-02Fix up raven-branch copyright notices.Simon Howard
Subversion-branch: /branches/raven-branch Subversion-revision: 1375
2008-10-02Split out high-level sound code from hexen/i_ibm.c and refactor to useSimon Howard
common interface. Subversion-branch: /branches/raven-branch Subversion-revision: 1324
2008-10-01Use common versions of ticcmd_t and event_t for Hexen code. Remove oldSimon Howard
i_video definitions and use common i_video.c interface. Subversion-branch: /branches/raven-branch Subversion-revision: 1312
2008-10-01Merge Hexen fixed_t and angle definitions to common code, plus byteSimon Howard
swapping macros, bounding box checking, read/write file, screenshot and command line argument code. Update Heretic code to use ANG1_X rather than the new (correct) ANG1 definition. Subversion-branch: /branches/raven-branch Subversion-revision: 1311
2008-09-17Convert MAXINT/MININT -> INT_MAX/INT_MIN in Heretic/Hexen code.Simon Howard
Subversion-branch: /branches/raven-branch Subversion-revision: 1234
2008-09-05Reformat (beautify) Raven sources and add GPL headers.Simon Howard
Subversion-branch: /branches/raven-branch Subversion-revision: 1197
2008-09-04Add GPLed Heretic/Hexen source.Simon Howard
Subversion-branch: /branches/raven-branch Subversion-revision: 1195