Age | Commit message (Collapse) | Author |
|
Fixed by commenting out the reported variables and their settings
instead of removing them. Since I am not sure if they were added by
mistake or by purpose and then overseen, I think it's better to keep
them in the code but not compile them in for now.
|
|
|
|
Signed/unsigned comparison mismatches
|
|
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.
|
|
If LOOP_START and LOOP_END are both set to zero, ignore them. This
is consistent with other source ports.
|
|
If a substitute music track is played in a non-looping configuration
(eg. the title screen music), ignore loop tags in the file to be
consistent with other source ports. This fixes a bug that was discussed
on #245.
|
|
For the Hexen substitute mapping configuration file, it's desirable
to be able to include two mappings for each music lump: a straight
high quality recording of that lump, and the recording that was
included on the Hexen CD audio tracks. So allow multiple mappings
so that we can fall back to try other filenames if the first choice
file doesn't appear to exist.
|
|
|
|
For substitute music files we want to be able to specify relative
paths in a platform-independent way using Unix path separators.
Replace Unix-style / separators in the path that was read with the
path separator for the native system.
|
|
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.
|
|
Allow comments to be attached to the end of configuration lines, as
well as being specified on a line on their own.
|
|
ZDoom has defined a format for Vorbis metadata comments named
LOOP_START and LOOP_END that allow the start and end points to be
defined in .ogg and .flac files for looping music. Add support for
these (they are used in Brandon Blume's SC-55 recordings).
|
|
We are free()ing the const char* a few lines later.
|
|
The config dumping command line option assumed that music lumps were
named like D_MYLUMP, but this is not the case for Heretic and Hexen,
where there is no D_ prefix and music lumps can have any name.
Change the logic to instead look at the contents of lumps and identify
music lumps from the MUS / MIDI header.
|
|
Using the .mp3 extension in generated music config files carries an
implicit recommendation to use that format. There are a number of
reasons to prefer other formats. The biggest is that MP3 is patent
encumbered. Furthermore, while FLAC and Ogg Vorbis both support the
Vorbis comment format that allows metadata loop point tags to be
specified.
|
|
Use appropriate directory separator for OS when generating config file
paths. When generating config files, include a header line that
indicates what is being assigned.
|
|
This adds support for high quality music packs that replace Doom's
built-in MIDI music with digital recordings. In particular this allows
recordings of the Roland SC-55 to be used in Chocolate Doom. This is
the first essential step for bug #245.
|
|
Mistake introduced in previous commit.
|
|
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.
|
|
Mix_SetMusicCMD() allows a program to be specified to configure an
external program to be invoked for music playback. Add a config
variable (snd_musiccmd) to allow this to be set from a configuration
file. Thanks to Holering for his comments on Doomworld about how to do
this.
|
|
Subversion-branch: /branches/v2-branch
Subversion-revision: 2566
|
|
Subversion-branch: /branches/v2-branch
Subversion-revision: 2565
|
|
file.
Subversion-branch: /branches/v2-branch
Subversion-revision: 2564
|
|
Subversion-branch: /branches/raven-branch
Subversion-revision: 1924
|
|
Subversion-branch: /branches/raven-branch
Subversion-revision: 1737
|
|
Subversion-branch: /branches/opl-branch
Subversion-revision: 1734
|
|
crash. Check and disable MIDI by default if using an older version of
SDL on OS X.
Subversion-branch: /trunk/chocolate-doom
Subversion-revision: 1730
|
|
Subversion-branch: /branches/opl-branch
Subversion-revision: 1702
|
|
Subversion-branch: /trunk/chocolate-doom
Subversion-revision: 1701
|
|
Subversion-branch: /trunk/chocolate-doom
Subversion-revision: 1699
|
|
Subversion-branch: /branches/opl-branch
Subversion-revision: 1447
|
|
Subversion-branch: /branches/raven-branch
Subversion-revision: 1398
|
|
Subversion-branch: /branches/raven-branch
Subversion-revision: 1213
|
|
old low-level sound API as a thin wrapper around the module system.
Decouple the low-level code from the high-level code.
Subversion-branch: /branches/raven-branch
Subversion-revision: 1212
|
|
common code and remove dependencies on deh_main.h.
Subversion-branch: /branches/raven-branch
Subversion-revision: 1210
|
|
level.
Subversion-branch: /branches/raven-branch
Subversion-revision: 1204
|
|
Subversion-branch: /trunk/chocolate-doom
Subversion-revision: 957
|
|
sizeof(*array)
Subversion-branch: /trunk/chocolate-doom
Subversion-revision: 922
|
|
is only one module. Remove s_dummy.c.
Subversion-branch: /trunk/chocolate-doom
Subversion-revision: 914
|
|
driver" modules, one for PC speaker and one for digital output.
Subversion-branch: /trunk/chocolate-doom
Subversion-revision: 913
|