id	summary	reporter	owner	description	type	status	priority	milestone	component	resolution	keywords	cc	phab_field
3578	[PATCH] Seed random sounds	Thomas		"Visual Actors now use the same seed for every sound.

Closes #1786.

Before using the random function it uses srand() with the actor id to seed the random function. I also made changes to rand.cpp. Apparently the first value returned by rand() is very dependent on the seed (see [http://stackoverflow.com/questions/8599238/random-number-generation-in-c-first-number-not-very-random]). Most actors were returning the same random value as they have similar seeds. Using modulo instead of dividing uses the least significant bits of the random value, yielding better results.

This is my first patch, so please tell me if I'm doing something wrong."	defect	new	Should Have	Alpha 19	Core engine		review patch