Oops, I forgot one thing:

Youness Alaoui ha scritto:
> While looking at this small portion of code in the diff, I also noticed 
> something else... we do a check for 
> whether the smiley is equal to %, in which case we insert %% (because the 
> 'bind' replaced %x with variable 
> values), but did we think of a smiley not being "%" but being "%W" instead 
> for example ? can you test something 
> like that ? and if it doesn't work (you get the window path inserted instead 
> of the "%W" smiley, then maybe it 
> would be wise to remove the if {[string match] } and replace it directly with 
> a bind "insert [string map { % %% 
> } $symbol] ...
I checked and works properly, but the same fix should be done with $name
in the editCustomEmotion calls; otherwise trying to edit a custom smiley
whose name contains the % will generate an error.

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Amsn-devel mailing list
Amsn-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/amsn-devel

Reply via email to