Re: [crossfire] sound2 protocol and issues: sound_chance

2010-10-02 Thread Nicolas Weeger
> The sound_chance feature prevents it from reaching the client. > > > The idea was to just send free-form strings, to enable clients to > > "override" sounds. > > Really, on trunk, sounds are entirely disabled at present due to these > changes. True. I guess I expected to work on that after

Re: [crossfire] sound2 protocol and issues: sound_chance

2010-10-02 Thread Nicolas Weeger
> Using a string for action name is probably fine, OTOH, there is > presumably a finite (and actually quite small) set of actions. Since > right now, all the play_sound calls are basically hardcoded on where they > get called, putting in an integer type would be fairly easy. What about plugins

Re: [crossfire] sound2 protocol and issues: sound_chance

2010-09-29 Thread Mark Wedel
On 09/29/10 09:26 AM, Nicolas Weeger wrote: >>Looking at the code, it also seems suspect for all the hard coded sound >> values (most all sounds), in that they are looking at an object that does >> not have any sound information defined - it is looking at skill values, >> monsters, etc, as for

Re: [crossfire] sound2 protocol and issues: sound_chance

2010-09-29 Thread Kevin Bulgrien
> > The problem is that the sounds already defined were disabled by the change, > > so it is not a matter of simply not implementing things incrementally. > > All that used to work no longer does.. > > I don't remember disabling previous sound support, but maybe I indeed did, > considering not m

Re: [crossfire] sound2 protocol and issues: sound_chance

2010-09-29 Thread Nicolas Weeger
> The problem is that the sounds already defined were disabled by the change, > so it is not a matter of simply not implementing things incrementally. > All that used to work no longer does.. I don't remember disabling previous sound support, but maybe I indeed did, considering not many people u

Re: [crossfire] sound2 protocol and issues: sound_chance

2010-09-29 Thread Nicolas Weeger
> Looking at the code, it also seems suspect for all the hard coded sound > values (most all sounds), in that they are looking at an object that does > not have any sound information defined - it is looking at skill values, > monsters, etc, as for the chance of the sound being generated. > > I

Re: [crossfire] sound2 protocol and issues: sound_chance

2010-09-26 Thread Mark Wedel
On 09/26/10 02:38 PM, Kevin Bulgrien wrote: >>> I retract the "I think sound_chance should be dispensed with". I finally >>> found the mail list note that mentioned some changes to sound so it could >>> support random ambient sounds to improve overall immersion in the game >>> environment... Stil

Re: [crossfire] sound2 protocol and issues: sound_chance

2010-09-26 Thread Kevin Bulgrien
> > I retract the "I think sound_chance should be dispensed with". I finally > > found the mail list note that mentioned some changes to sound so it could > > support random ambient sounds to improve overall immersion in the game > > environment... Still, the idea that 0 behaves the same as 100 s

Re: [crossfire] sound2 protocol and issues: sound_chance

2010-09-26 Thread Nicolas Weeger
Hello. > I retract the "I think sound_chance should be dispensed with". I finally > found the mail list note that mentioned some changes to sound so it could > support random ambient sounds to improve overall immersion in the game > environment... Still, the idea that 0 behaves the same as 100

Re: [crossfire] sound2 protocol and issues: sound_chance

2010-09-25 Thread Kevin R. Bulgrien
> What purpose can sound_chance even be put to in such quantity that it makes > such sweeping arch changes reasonable? If an item/action produces sound, > it is very difficult to conceive of a logical reason for. I.e. Explode > this bomb and 9 out of 10 times the massive, concussive blast produc

[crossfire] sound2 protocol and issues: sound_chance

2010-09-25 Thread Kevin R. Bulgrien
At the time sound2 was implemented, a sound_chance property was set up. No arches were modified. This effectively disables all sound support. By SVN records, sound has been disabled for about 2.75 years because of sound2 implementation of sound_chance. What purpose can sound_chance even be put