EvilGuru changed status from new to accepted

EvilGuru changed owner from `` to EvilGuru

EvilGuru commented

This is the offending code (from intelmap.c):

#!c
switch(psResearch->iconID)
{
	case IMAGE_RES_DROIDTECH:
		audio_PlayStream("sequenceaudio/res_droid.ogg", maxVolume, NULL, NULL);
		break;
	case IMAGE_RES_WEAPONTECH:
		audio_PlayStream("sequenceaudio/res_weapons.ogg", maxVolume, NULL, NULL);
			break;
	case IMAGE_RES_COMPUTERTECH:
		audio_PlayStream("sequenceaudio/res_com.ogg", maxVolume, NULL, NULL);
				break;
	case IMAGE_RES_POWERTECH:
		audio_PlayStream("sequenceaudio/res_pow.ogg", maxVolume, NULL, NULL);
			break;
	case IMAGE_RES_SYSTEMTECH:
		audio_PlayStream("sequenceaudio/res_systech.ogg", maxVolume, NULL, NULL);
			break;
	case IMAGE_RES_STRUCTURETECH:
		audio_PlayStream("sequenceaudio/res_strutech.ogg", maxVolume, NULL, NULL);
			break;
	case IMAGE_RES_CYBORGTECH:
		audio_PlayStream("sequenceaudio/res_droid.ogg", maxVolume, NULL, NULL);
			break;
	case IMAGE_RES_DEFENCE:
		audio_PlayStream("sequenceaudio/res_strutech.ogg", maxVolume, NULL, NULL);
			break;
}

If the community can reach a consensus on this I'll happily remove the code in question (or leave it alone, depending on the aforementioned consensus). For a bottle of Gin I'll make it a configurable option (with the default being whatever the consensus is).

Kreuvf commented

The thing is that for every research-type mentioned in that piece of code in the original WZ a video has been played (that has audio that added some information on what type of research you researched) just to the right of the model.

So a better option imho would be to extract the sound from the videos (they are GPL now, so we are allowed to do this :D) and replace the .ogg-files with the corresponding (correct) sound.

Giel changed operating_system which not transferred by tractive

Giel changed milestone from `` to 2.1

Buginator changed blocking which not transferred by tractive

Buginator changed blockedby which not transferred by tractive

Buginator commented

The intel FMVs don't have any sound.
It just uses the wav (.ogg) file for this. I guess since they supported more than one language.

It repeats since we don't have a way to stop said sample. Well, easily that is.

I was thinking we should just use a dedicated channel (source) for this, and all FMV related stuff, so we can stop it.

Zarel changed priority from minor to trivial

Zarel commented

Well, now that we do have FMVs, thoughts?

Buginator removed milestone (was 2.1)

Buginator commented

Milestone 2.1 deleted

Buginator changed status from accepted to closed

Buginator set resolution to fixed

Buginator commented

(In [11147]) Prevent overlapping sounds when looking at intel information.

fixes #60

Buginator commented

(In [11148]) Prevent overlapping sounds when looking at intel information.

fixes #60

2.3:[11147]

Buginator commented

(In [11149]) Prevent overlapping sounds when looking at intel information.

fixes #60

2.3: [11147]
trunk: [11148]

Git SVN Gateway <gateway@...> commented

(In [0dca1b06a1b7607f06d1b58a5869a3f3420e7f5b]) Prevent overlapping sounds when looking at intel information.

fixes #60

2.3:[11147]

git-svn-id: https://warzone2100.svn.sourceforge.net/svnroot/warzone2100/trunk@11148 4a71c877-e1ca-e34f-864e-861f7616d084

Zarel set milestone to unspecified

Zarel commented

I think the easier solution would be just to not play that sound at all. I mean, I don't think it's much of a benefit, and it gets kind of annoying after a while, with or without the overlap...

Git SVN Gateway <gateway@...> commented

In [0dca1b06a1b7607f06d1b58a5869a3f3420e7f5b]:

#CommitTicketReference repository="" revision="0dca1b06a1b7607f06d1b58a5869a3f3420e7f5b"
Prevent overlapping sounds when looking at intel information.

fixes #60

2.3:[11147]

git-svn-id: https://warzone2100.svn.sourceforge.net/svnroot/warzone2100/trunk@11148 4a71c877-e1ca-e34f-864e-861f7616d084