[Amsn-devel] Re: [Amsn-commits] msn gui.tcl,1.1215,1.1216 picture.tcl,1.18,1.19 skins.tcl,1.112,1.113

[Jérôme Gagnon-Voyer]

Before we got the "long message", now we get the "longest commit message ever" :D I'm very happy of all the changes you did, optmization of the actions we do frequently (like reloading contact list) is very important.  aMSN should run a little faster now (or takes less CPU, or both!), and I know that a lot of users will love that :P Le 1 février 2006 à 11:50, Alaoui Youness a écrit : Logo animation on connect now uses ::skin::loadPixmap to avoid deleting/recreating the image everytime.. Logo animation on connect is now an inserted image instead of a label with an image.. clicableImages (in CL and chatwindows, like the mailbox button for example) are now an image inserted into the text instead of being an image in a label  and the label being inserted... cleaned up some code for getClicableDisapplyPicture.. my DP and my smaller DP (resized for the CL) are now recreated only if the DP changed, not everytime the CL is updated... Our own PSM is now bold/italic in the CL Some code cleaning... check for globalnick before inserting PSM... We now check if the DP of users is animated ONLY if we have DPs in CL and if the small_dp doesn't already exist! We now have a cache for checking animated files, since a file will rarely change, and we don't want to open/close a gif everytime, in case we already know if it's animated Changed some "catch {image width}" into "lsearch [image names]" to avoid the catch failing which is cpu intensive (compared to lsearch...) Now, getDisplayPicture returns the no_pic image, instead of copying the no_pic image into the user's image, this way, we won't create useless images and a user without a DP has no image for him.. cleaner code I think.. NEEDS TESTING! The colorbar under the mail line of CL is now saved and is not regenerated unless the CL width changed.. deleting the preview images used by the skin selector once it's destroyed. OUPS : REverted back my changes from the catch {image width} to lsearch... it seems, Arieh is  right and it is far faster to use the catch instead of the search.. even if the catch is itself  cpu consuming, the lsearch is even more cpu consuming... changed it back and merged my change with  his fix on the search for image names for the DPS in CL