man, 08 12 2008 kl. 12:25 -0500, skrev Roman Stanchak:
> Patch is attached.

I've applied it.

> I also noticed that the logic for checking the global
> sound_play_utility variable in sound.m wasn't working for me.
> 
> The existing code uses the 'exist' function to see if the users has
> already set sound_play_utility:
> 
> function sound()
> ## ...
> if (exist("sound_play_utility", "var"))
>     ## The user is always right, so just declare the set variable as global
>     global sound_play_utility;
> else ## check path for known audio players...
> 
> exist always returns false for me inside the function, regardless of
> whether I declare sound_play_utility global in my octaverc or
> interpreter. After reading the docs on global variables, I think it
> should be:
> 
> function sound()
> ## ...
> global sound_play_utility;
> if ~isempty(sound_play_utility),
>   ## User has defined a play command
> else ## check path for known audio players ...

I think you are correct. Care to suggest a patch :-)  ?

Søren


------------------------------------------------------------------------------
SF.Net email is Sponsored by MIX09, March 18-20, 2009 in Las Vegas, Nevada.
The future of the web can't happen without you.  Join us at MIX09 to help
pave the way to the Next Web now. Learn more and register at
http://ad.doubleclick.net/clk;208669438;13503038;i?http://2009.visitmix.com/
_______________________________________________
Octave-dev mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/octave-dev

Reply via email to