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
