]> de.git.xonotic.org Git - xonotic/darkplaces.git/blobdiff - snd_main.c
DP_SND_SOUND7_WIP1
[xonotic/darkplaces.git] / snd_main.c
index ec3d172c2614d5f23ff11bb7b118848a5bdf3ef2..e537d8388661b63d0c35345d6ff1568d9ffdff5b 100644 (file)
@@ -182,38 +182,46 @@ cvar_t snd_swapstereo = {CVAR_SAVE, "snd_swapstereo", "0", "swaps left/right spe
 extern cvar_t v_flipped;
 cvar_t snd_channellayout = {0, "snd_channellayout", "0", "channel layout. Can be 0 (auto - snd_restart needed), 1 (standard layout), or 2 (ALSA layout)"};
 cvar_t snd_mutewhenidle = {CVAR_SAVE, "snd_mutewhenidle", "1", "whether to disable sound output when game window is inactive"};
-cvar_t snd_entchannel0volume = {CVAR_SAVE, "snd_entchannel0volume", "1", "volume multiplier of the auto-allocate entity channel of regular entities"};
-cvar_t snd_entchannel1volume = {CVAR_SAVE, "snd_entchannel1volume", "1", "volume multiplier of the 1st entity channel of regular entities"};
-cvar_t snd_entchannel2volume = {CVAR_SAVE, "snd_entchannel2volume", "1", "volume multiplier of the 2nd entity channel of regular entities"};
-cvar_t snd_entchannel3volume = {CVAR_SAVE, "snd_entchannel3volume", "1", "volume multiplier of the 3rd entity channel of regular entities"};
-cvar_t snd_entchannel4volume = {CVAR_SAVE, "snd_entchannel4volume", "1", "volume multiplier of the 4th entity channel of regular entities"};
-cvar_t snd_entchannel5volume = {CVAR_SAVE, "snd_entchannel5volume", "1", "volume multiplier of the 5th entity channel of regular entities"};
-cvar_t snd_entchannel6volume = {CVAR_SAVE, "snd_entchannel6volume", "1", "volume multiplier of the 6th entity channel of regular entities"};
-cvar_t snd_entchannel7volume = {CVAR_SAVE, "snd_entchannel7volume", "1", "volume multiplier of the 7th entity channel of regular entities"};
-cvar_t snd_playerchannel0volume = {CVAR_SAVE, "snd_playerchannel0volume", "1", "volume multiplier of the auto-allocate entity channel of player entities"};
-cvar_t snd_playerchannel1volume = {CVAR_SAVE, "snd_playerchannel1volume", "1", "volume multiplier of the 1st entity channel of player entities"};
-cvar_t snd_playerchannel2volume = {CVAR_SAVE, "snd_playerchannel2volume", "1", "volume multiplier of the 2nd entity channel of player entities"};
-cvar_t snd_playerchannel3volume = {CVAR_SAVE, "snd_playerchannel3volume", "1", "volume multiplier of the 3rd entity channel of player entities"};
-cvar_t snd_playerchannel4volume = {CVAR_SAVE, "snd_playerchannel4volume", "1", "volume multiplier of the 4th entity channel of player entities"};
-cvar_t snd_playerchannel5volume = {CVAR_SAVE, "snd_playerchannel5volume", "1", "volume multiplier of the 5th entity channel of player entities"};
-cvar_t snd_playerchannel6volume = {CVAR_SAVE, "snd_playerchannel6volume", "1", "volume multiplier of the 6th entity channel of player entities"};
-cvar_t snd_playerchannel7volume = {CVAR_SAVE, "snd_playerchannel7volume", "1", "volume multiplier of the 7th entity channel of player entities"};
-cvar_t snd_worldchannel0volume = {CVAR_SAVE, "snd_worldchannel0volume", "1", "volume multiplier of the auto-allocate entity channel of the world entity"};
-cvar_t snd_worldchannel1volume = {CVAR_SAVE, "snd_worldchannel1volume", "1", "volume multiplier of the 1st entity channel of the world entity"};
-cvar_t snd_worldchannel2volume = {CVAR_SAVE, "snd_worldchannel2volume", "1", "volume multiplier of the 2nd entity channel of the world entity"};
-cvar_t snd_worldchannel3volume = {CVAR_SAVE, "snd_worldchannel3volume", "1", "volume multiplier of the 3rd entity channel of the world entity"};
-cvar_t snd_worldchannel4volume = {CVAR_SAVE, "snd_worldchannel4volume", "1", "volume multiplier of the 4th entity channel of the world entity"};
-cvar_t snd_worldchannel5volume = {CVAR_SAVE, "snd_worldchannel5volume", "1", "volume multiplier of the 5th entity channel of the world entity"};
-cvar_t snd_worldchannel6volume = {CVAR_SAVE, "snd_worldchannel6volume", "1", "volume multiplier of the 6th entity channel of the world entity"};
-cvar_t snd_worldchannel7volume = {CVAR_SAVE, "snd_worldchannel7volume", "1", "volume multiplier of the 7th entity channel of the world entity"};
-cvar_t snd_csqcchannel0volume = {CVAR_SAVE, "snd_csqcchannel0volume", "1", "volume multiplier of the auto-allocate entity channel of the world entity"};
-cvar_t snd_csqcchannel1volume = {CVAR_SAVE, "snd_csqcchannel1volume", "1", "volume multiplier of the 1st entity channel of the world entity"};
-cvar_t snd_csqcchannel2volume = {CVAR_SAVE, "snd_csqcchannel2volume", "1", "volume multiplier of the 2nd entity channel of the world entity"};
-cvar_t snd_csqcchannel3volume = {CVAR_SAVE, "snd_csqcchannel3volume", "1", "volume multiplier of the 3rd entity channel of the world entity"};
-cvar_t snd_csqcchannel4volume = {CVAR_SAVE, "snd_csqcchannel4volume", "1", "volume multiplier of the 4th entity channel of the world entity"};
-cvar_t snd_csqcchannel5volume = {CVAR_SAVE, "snd_csqcchannel5volume", "1", "volume multiplier of the 5th entity channel of the world entity"};
-cvar_t snd_csqcchannel6volume = {CVAR_SAVE, "snd_csqcchannel6volume", "1", "volume multiplier of the 6th entity channel of the world entity"};
-cvar_t snd_csqcchannel7volume = {CVAR_SAVE, "snd_csqcchannel7volume", "1", "volume multiplier of the 7th entity channel of the world entity"};
+cvar_t snd_entchannel0volume = {CVAR_SAVE, "snd_entchannel0volume", "1", "volume multiplier of the auto-allocate entity channel of regular entities (DEPRECATED)"};
+cvar_t snd_entchannel1volume = {CVAR_SAVE, "snd_entchannel1volume", "1", "volume multiplier of the 1st entity channel of regular entities (DEPRECATED)"};
+cvar_t snd_entchannel2volume = {CVAR_SAVE, "snd_entchannel2volume", "1", "volume multiplier of the 2nd entity channel of regular entities (DEPRECATED)"};
+cvar_t snd_entchannel3volume = {CVAR_SAVE, "snd_entchannel3volume", "1", "volume multiplier of the 3rd entity channel of regular entities (DEPRECATED)"};
+cvar_t snd_entchannel4volume = {CVAR_SAVE, "snd_entchannel4volume", "1", "volume multiplier of the 4th entity channel of regular entities (DEPRECATED)"};
+cvar_t snd_entchannel5volume = {CVAR_SAVE, "snd_entchannel5volume", "1", "volume multiplier of the 5th entity channel of regular entities (DEPRECATED)"};
+cvar_t snd_entchannel6volume = {CVAR_SAVE, "snd_entchannel6volume", "1", "volume multiplier of the 6th entity channel of regular entities (DEPRECATED)"};
+cvar_t snd_entchannel7volume = {CVAR_SAVE, "snd_entchannel7volume", "1", "volume multiplier of the 7th entity channel of regular entities (DEPRECATED)"};
+cvar_t snd_playerchannel0volume = {CVAR_SAVE, "snd_playerchannel0volume", "1", "volume multiplier of the auto-allocate entity channel of player entities (DEPRECATED)"};
+cvar_t snd_playerchannel1volume = {CVAR_SAVE, "snd_playerchannel1volume", "1", "volume multiplier of the 1st entity channel of player entities (DEPRECATED)"};
+cvar_t snd_playerchannel2volume = {CVAR_SAVE, "snd_playerchannel2volume", "1", "volume multiplier of the 2nd entity channel of player entities (DEPRECATED)"};
+cvar_t snd_playerchannel3volume = {CVAR_SAVE, "snd_playerchannel3volume", "1", "volume multiplier of the 3rd entity channel of player entities (DEPRECATED)"};
+cvar_t snd_playerchannel4volume = {CVAR_SAVE, "snd_playerchannel4volume", "1", "volume multiplier of the 4th entity channel of player entities (DEPRECATED)"};
+cvar_t snd_playerchannel5volume = {CVAR_SAVE, "snd_playerchannel5volume", "1", "volume multiplier of the 5th entity channel of player entities (DEPRECATED)"};
+cvar_t snd_playerchannel6volume = {CVAR_SAVE, "snd_playerchannel6volume", "1", "volume multiplier of the 6th entity channel of player entities (DEPRECATED)"};
+cvar_t snd_playerchannel7volume = {CVAR_SAVE, "snd_playerchannel7volume", "1", "volume multiplier of the 7th entity channel of player entities (DEPRECATED)"};
+cvar_t snd_worldchannel0volume = {CVAR_SAVE, "snd_worldchannel0volume", "1", "volume multiplier of the auto-allocate entity channel of the world entity (DEPRECATED)"};
+cvar_t snd_worldchannel1volume = {CVAR_SAVE, "snd_worldchannel1volume", "1", "volume multiplier of the 1st entity channel of the world entity (DEPRECATED)"};
+cvar_t snd_worldchannel2volume = {CVAR_SAVE, "snd_worldchannel2volume", "1", "volume multiplier of the 2nd entity channel of the world entity (DEPRECATED)"};
+cvar_t snd_worldchannel3volume = {CVAR_SAVE, "snd_worldchannel3volume", "1", "volume multiplier of the 3rd entity channel of the world entity (DEPRECATED)"};
+cvar_t snd_worldchannel4volume = {CVAR_SAVE, "snd_worldchannel4volume", "1", "volume multiplier of the 4th entity channel of the world entity (DEPRECATED)"};
+cvar_t snd_worldchannel5volume = {CVAR_SAVE, "snd_worldchannel5volume", "1", "volume multiplier of the 5th entity channel of the world entity (DEPRECATED)"};
+cvar_t snd_worldchannel6volume = {CVAR_SAVE, "snd_worldchannel6volume", "1", "volume multiplier of the 6th entity channel of the world entity (DEPRECATED)"};
+cvar_t snd_worldchannel7volume = {CVAR_SAVE, "snd_worldchannel7volume", "1", "volume multiplier of the 7th entity channel of the world entity (DEPRECATED)"};
+cvar_t snd_csqcchannel0volume = {CVAR_SAVE, "snd_csqcchannel0volume", "1", "volume multiplier of the auto-allocate entity channel CSQC entities (DEPRECATED)"};
+cvar_t snd_csqcchannel1volume = {CVAR_SAVE, "snd_csqcchannel1volume", "1", "volume multiplier of the 1st entity channel of CSQC entities (DEPRECATED)"};
+cvar_t snd_csqcchannel2volume = {CVAR_SAVE, "snd_csqcchannel2volume", "1", "volume multiplier of the 2nd entity channel of CSQC entities (DEPRECATED)"};
+cvar_t snd_csqcchannel3volume = {CVAR_SAVE, "snd_csqcchannel3volume", "1", "volume multiplier of the 3rd entity channel of CSQC entities (DEPRECATED)"};
+cvar_t snd_csqcchannel4volume = {CVAR_SAVE, "snd_csqcchannel4volume", "1", "volume multiplier of the 4th entity channel of CSQC entities (DEPRECATED)"};
+cvar_t snd_csqcchannel5volume = {CVAR_SAVE, "snd_csqcchannel5volume", "1", "volume multiplier of the 5th entity channel of CSQC entities (DEPRECATED)"};
+cvar_t snd_csqcchannel6volume = {CVAR_SAVE, "snd_csqcchannel6volume", "1", "volume multiplier of the 6th entity channel of CSQC entities (DEPRECATED)"};
+cvar_t snd_csqcchannel7volume = {CVAR_SAVE, "snd_csqcchannel7volume", "1", "volume multiplier of the 7th entity channel of CSQC entities (DEPRECATED)"};
+cvar_t snd_channel0volume = {CVAR_SAVE, "snd_channel0volume", "1", "volume multiplier of the auto-allocate entity channel"};
+cvar_t snd_channel1volume = {CVAR_SAVE, "snd_channel1volume", "1", "volume multiplier of the 1st entity channel"};
+cvar_t snd_channel2volume = {CVAR_SAVE, "snd_channel2volume", "1", "volume multiplier of the 2nd entity channel"};
+cvar_t snd_channel3volume = {CVAR_SAVE, "snd_channel3volume", "1", "volume multiplier of the 3rd entity channel"};
+cvar_t snd_channel4volume = {CVAR_SAVE, "snd_channel4volume", "1", "volume multiplier of the 4th entity channel"};
+cvar_t snd_channel5volume = {CVAR_SAVE, "snd_channel5volume", "1", "volume multiplier of the 5th entity channel"};
+cvar_t snd_channel6volume = {CVAR_SAVE, "snd_channel6volume", "1", "volume multiplier of the 6th entity channel"};
+cvar_t snd_channel7volume = {CVAR_SAVE, "snd_channel7volume", "1", "volume multiplier of the 7th entity channel"};
 
 // Local cvars
 static cvar_t nosound = {0, "nosound", "0", "disables sound"};
@@ -310,11 +318,10 @@ static void S_SoundList_f (void)
 
                        size = sfx->memsize;
                        format = sfx->fetcher->getfmt(sfx);
-                       Con_Printf ("%c%c%c%c(%2db, %6s) %8i : %s\n",
+                       Con_Printf ("%c%c%c(%2db, %6s) %8i : %s\n",
                                                (sfx->loopstart < sfx->total_length) ? 'L' : ' ',
                                                (sfx->flags & SFXFLAG_STREAMED) ? 'S' : ' ',
-                                               (sfx->locks > 0) ? 'K' : ' ',
-                                               (sfx->flags & SFXFLAG_PERMANENTLOCK) ? 'P' : ' ',
+                                               (sfx->flags & SFXFLAG_MENUSOUND) ? 'P' : ' ',
                                                format->width * 8,
                                                (format->channels == 1) ? "mono" : "stereo",
                                                size,
@@ -814,6 +821,14 @@ void S_Init(void)
        Cvar_RegisterVariable(&snd_csqcchannel5volume);
        Cvar_RegisterVariable(&snd_csqcchannel6volume);
        Cvar_RegisterVariable(&snd_csqcchannel7volume);
+       Cvar_RegisterVariable(&snd_channel0volume);
+       Cvar_RegisterVariable(&snd_channel1volume);
+       Cvar_RegisterVariable(&snd_channel2volume);
+       Cvar_RegisterVariable(&snd_channel3volume);
+       Cvar_RegisterVariable(&snd_channel4volume);
+       Cvar_RegisterVariable(&snd_channel5volume);
+       Cvar_RegisterVariable(&snd_channel6volume);
+       Cvar_RegisterVariable(&snd_channel7volume);
 
        Cvar_RegisterVariable(&snd_spatialization_min_radius);
        Cvar_RegisterVariable(&snd_spatialization_max_radius);
@@ -983,8 +998,8 @@ void S_FreeSfx (sfx_t *sfx, qboolean force)
 {
        unsigned int i;
 
-       // Never free a locked sfx unless forced
-       if (!force && (sfx->locks > 0 || (sfx->flags & SFXFLAG_PERMANENTLOCK)))
+       // Do not free a precached sound during purge
+       if (!force && (sfx->flags & (SFXFLAG_LEVELSOUND | SFXFLAG_MENUSOUND)))
                return;
 
        if (developer_loading.integer)
@@ -1012,8 +1027,13 @@ void S_FreeSfx (sfx_t *sfx, qboolean force)
 
        // Stop all channels using this sfx
        for (i = 0; i < total_channels; i++)
+       {
                if (channels[i].sfx == sfx)
-                       S_StopChannel (i, true);
+               {
+                       Con_Printf("S_FreeSfx: stopping channel %i for sfx \"%s\"\n", i, sfx->name);
+                       S_StopChannel (i, true, false);
+               }
+       }
 
        // Free it
        if (sfx->fetcher != NULL && sfx->fetcher->free != NULL)
@@ -1036,28 +1056,21 @@ void S_ClearUsed (void)
        // Start the ambient sounds and make them loop
        for (i = 0; i < sizeof (ambient_sfxs) / sizeof (ambient_sfxs[0]); i++)
        {
-               // Precache it if it's not done (request a lock to make sure it will never be freed)
+               // Precache it if it's not done (and pass false for levelsound because these are permanent)
                if (ambient_sfxs[i] == NULL)
-                       ambient_sfxs[i] = S_PrecacheSound (ambient_names[i], false, true);
+                       ambient_sfxs[i] = S_PrecacheSound (ambient_names[i], false, false);
                if (ambient_sfxs[i] != NULL)
                {
-                       // Add a lock to the SFX while playing. It will be
-                       // removed by S_StopAllSounds at the end of the level
-                       S_LockSfx (ambient_sfxs[i]);
-
                        channels[i].sfx = ambient_sfxs[i];
+                       channels[i].sfx->flags |= SFXFLAG_MENUSOUND;
                        channels[i].flags |= CHANNELFLAG_FORCELOOP;
                        channels[i].master_vol = 0;
                }
        }
 
-       // Remove 1 lock from all sfx with the SFXFLAG_SERVERSOUND flag, and remove the flag
+       // Clear SFXFLAG_LEVELSOUND flag so that sounds not precached this level will be purged
        for (sfx = known_sfx; sfx != NULL; sfx = sfx->next)
-               if (sfx->flags & SFXFLAG_SERVERSOUND)
-               {
-                       S_UnlockSfx (sfx);
-                       sfx->flags &= ~SFXFLAG_SERVERSOUND;
-               }
+               sfx->flags &= ~SFXFLAG_LEVELSOUND;
 }
 
 /*
@@ -1070,11 +1083,12 @@ void S_PurgeUnused(void)
        sfx_t *sfx;
        sfx_t *sfxnext;
 
-       // Free all unlocked sfx
+       // Free all not-precached per-level sfx
        for (sfx = known_sfx;sfx;sfx = sfxnext)
        {
                sfxnext = sfx->next;
-               S_FreeSfx (sfx, false);
+               if (!(sfx->flags & (SFXFLAG_LEVELSOUND | SFXFLAG_MENUSOUND)))
+                       S_FreeSfx (sfx, false);
        }
 }
 
@@ -1084,7 +1098,7 @@ void S_PurgeUnused(void)
 S_PrecacheSound
 ==================
 */
-sfx_t *S_PrecacheSound (const char *name, qboolean complain, qboolean serversound)
+sfx_t *S_PrecacheSound (const char *name, qboolean complain, qboolean levelsound)
 {
        sfx_t *sfx;
 
@@ -1103,11 +1117,11 @@ sfx_t *S_PrecacheSound (const char *name, qboolean complain, qboolean serversoun
        // previously missing file
        sfx->flags &= ~ SFXFLAG_FILEMISSING;
 
-       if (serversound && !(sfx->flags & SFXFLAG_SERVERSOUND))
-       {
-               S_LockSfx (sfx);
-               sfx->flags |= SFXFLAG_SERVERSOUND;
-       }
+       // set a flag to indicate this has been precached for this level or permanently
+       if (levelsound)
+               sfx->flags |= SFXFLAG_LEVELSOUND;
+       else
+               sfx->flags |= SFXFLAG_MENUSOUND;
 
        if (!nosound.integer && snd_precache.integer)
                S_LoadSound(sfx, complain);
@@ -1146,31 +1160,6 @@ qboolean S_IsSoundPrecached (const sfx_t *sfx)
        return (sfx != NULL && sfx->fetcher != NULL) || (sfx == &changevolume_sfx);
 }
 
-/*
-==================
-S_LockSfx
-
-Add a lock to a SFX
-==================
-*/
-void S_LockSfx (sfx_t *sfx)
-{
-       sfx->locks++;
-}
-
-/*
-==================
-S_UnlockSfx
-
-Remove a lock from a SFX
-==================
-*/
-void S_UnlockSfx (sfx_t *sfx)
-{
-       sfx->locks--;
-}
-
-
 /*
 ==================
 S_BlockSound
@@ -1213,15 +1202,16 @@ channel_t *SND_PickChannel(int entnum, int entchannel)
        first_life_left = 0x7fffffff;
 
        // entity channels try to replace the existing sound on the channel
-       if (entchannel != 0)
+       // channels <= 0 are autochannels
+       if (IS_CHAN_SINGLE(entchannel))
        {
                for (ch_idx=NUM_AMBIENTS ; ch_idx < NUM_AMBIENTS + MAX_DYNAMIC_CHANNELS ; ch_idx++)
                {
                        ch = &channels[ch_idx];
-                       if (ch->entnum == entnum && (ch->entchannel == entchannel || entchannel == -1) )
+                       if (ch->entnum == entnum && ch->entchannel == entchannel)
                        {
                                // always override sound from same entity
-                               S_StopChannel (ch_idx, true);
+                               S_StopChannel (ch_idx, true, false);
                                return &channels[ch_idx];
                        }
                }
@@ -1258,7 +1248,7 @@ channel_t *SND_PickChannel(int entnum, int entchannel)
        if (first_to_die == -1)
                return NULL;
        
-       S_StopChannel (first_to_die, true);
+       S_StopChannel (first_to_die, true, false);
 
 emptychan_found:
        return &channels[first_to_die];
@@ -1310,6 +1300,7 @@ void SND_Spatialize_WithSfx(channel_t *ch, qboolean isstatic, sfx_t *sfx)
                mastervol *= snd_staticvolume.value;
        else if(!(ch->flags & CHANNELFLAG_FULLVOLUME)) // same as SND_PaintChannel uses
        {
+               // old legacy separated cvars
                if(ch->entnum >= MAX_EDICTS)
                {
                        switch(ch->entchannel)
@@ -1370,6 +1361,19 @@ void SND_Spatialize_WithSfx(channel_t *ch, qboolean isstatic, sfx_t *sfx)
                                default:                                          break;
                        }
                }
+
+               switch(ch->entchannel)
+               {
+                       case 0:  mastervol *= snd_channel0volume.value; break;
+                       case 1:  mastervol *= snd_channel1volume.value; break;
+                       case 2:  mastervol *= snd_channel2volume.value; break;
+                       case 3:  mastervol *= snd_channel3volume.value; break;
+                       case 4:  mastervol *= snd_channel4volume.value; break;
+                       case 5:  mastervol *= snd_channel5volume.value; break;
+                       case 6:  mastervol *= snd_channel6volume.value; break;
+                       case 7:  mastervol *= snd_channel7volume.value; break;
+                       default: mastervol *= Cvar_VariableValueOr(va("snd_channel%dvolume", CHAN_ENGINE2CVAR(ch->entchannel)), 1.0); break;
+               }
        }
 
        // If this channel does not manage its own volume (like CD tracks)
@@ -1584,7 +1588,7 @@ void S_PlaySfxOnChannel (sfx_t *sfx, channel_t *target_chan, unsigned int flags,
        {
                int channelindex = (int)(target_chan - channels);
                Con_Printf("S_PlaySfxOnChannel(%s): channel %i already in use??  Clearing.\n", sfx->name, channelindex);
-               S_StopChannel (channelindex, true);
+               S_StopChannel (channelindex, true, false);
        }
        // We MUST set sfx LAST because otherwise we could crash a threaded mixer
        // (otherwise we'd have to call SndSys_LockRenderBuffer here)
@@ -1609,9 +1613,6 @@ void S_PlaySfxOnChannel (sfx_t *sfx, channel_t *target_chan, unsigned int flags,
        S_SetChannelVolume(target_chan - channels, fvol);
        SND_Spatialize_WithSfx (target_chan, isstatic, sfx);
 
-       // Lock the SFX during play
-       S_LockSfx (sfx);
-
        // finally, set the sfx pointer, so the channel becomes valid for playback
        // and will be noticed by the mixer
        target_chan->sfx = sfx;
@@ -1628,12 +1629,12 @@ int S_StartSound_StartPosition_Flags (int entnum, int entchannel, sfx_t *sfx, ve
 
        if(sfx == &changevolume_sfx)
        {
-               if(entchannel == 0)
+               if (!IS_CHAN_SINGLE(entchannel))
                        return -1;
                for (ch_idx=NUM_AMBIENTS ; ch_idx < NUM_AMBIENTS + MAX_DYNAMIC_CHANNELS ; ch_idx++)
                {
                        ch = &channels[ch_idx];
-                       if (ch->entnum == entnum && (ch->entchannel == entchannel || entchannel == -1) )
+                       if (ch->entnum == entnum && ch->entchannel == entchannel)
                        {
                                S_SetChannelVolume(ch_idx, fvol);
                                ch->dist_mult = attenuation / snd_soundradius.value;
@@ -1686,9 +1687,10 @@ int S_StartSound (int entnum, int entchannel, sfx_t *sfx, vec3_t origin, float f
        return S_StartSound_StartPosition(entnum, entchannel, sfx, origin, fvol, attenuation, 0);
 }
 
-void S_StopChannel (unsigned int channel_ind, qboolean lockmutex)
+void S_StopChannel (unsigned int channel_ind, qboolean lockmutex, qboolean freesfx)
 {
        channel_t *ch;
+       sfx_t *sfx;
 
        if (channel_ind >= total_channels)
                return;
@@ -1701,10 +1703,9 @@ void S_StopChannel (unsigned int channel_ind, qboolean lockmutex)
                SndSys_LockRenderBuffer();
        
        ch = &channels[channel_ind];
+       sfx = ch->sfx;
        if (ch->sfx != NULL)
        {
-               sfx_t *sfx = ch->sfx;
-
                if (sfx->fetcher != NULL)
                {
                        snd_fetcher_endsb_t fetcher_endsb = sfx->fetcher->endsb;
@@ -1712,14 +1713,13 @@ void S_StopChannel (unsigned int channel_ind, qboolean lockmutex)
                                fetcher_endsb (ch->fetcher_data);
                }
 
-               // Remove the lock it holds
-               S_UnlockSfx (sfx);
-
                ch->fetcher_data = NULL;
                ch->sfx = NULL;
        }
        if (lockmutex && !simsound)
                SndSys_UnlockRenderBuffer();
+       if (freesfx)
+               S_FreeSfx(sfx, true);
 }
 
 
@@ -1749,7 +1749,7 @@ void S_StopSound(int entnum, int entchannel)
        for (i = 0; i < MAX_DYNAMIC_CHANNELS; i++)
                if (channels[i].entnum == entnum && channels[i].entchannel == entchannel)
                {
-                       S_StopChannel (i, true);
+                       S_StopChannel (i, true, false);
                        return;
                }
 }
@@ -1772,7 +1772,8 @@ void S_StopAllSounds (void)
                size_t memsize;
 
                for (i = 0; i < total_channels; i++)
-                       S_StopChannel (i, false);
+                       if (channels[i].sfx)
+                               S_StopChannel (i, false, false);
 
                total_channels = MAX_DYNAMIC_CHANNELS + NUM_AMBIENTS;   // no statics
                memset(channels, 0, MAX_CHANNELS * sizeof(channel_t));
@@ -2266,9 +2267,12 @@ qboolean S_LocalSound (const char *sound)
                return false;
        }
 
-       // Local sounds must not be freed
-       sfx->flags |= SFXFLAG_PERMANENTLOCK;
+       // menu sounds must not be freed on level change
+       sfx->flags |= SFXFLAG_MENUSOUND;
 
+       // fun fact: in Quake 1, this used -1 "replace any entity channel",
+       // which we no longer support anyway
+       // changed by Black in r4297 "Changed S_LocalSound to play multiple sounds at a time."
        ch_ind = S_StartSound (cl.viewentity, 0, sfx, vec3_origin, 1, 0);
        if (ch_ind < 0)
                return false;