I had actually thought of both, the first I tested on the status line to see
what would be returned from that list and it gave me weird places for the
array {}. The second I had also thought of (I actually thought that I had
put it in a PS from that email) but really didn't like the syntax, coz it is
still very error prone (ie the way you have it there $balloon_message will
be expanded too early so the number/position of the parameters will be
incorrect.

Lio.

----- Original Message ----- 
From: "Youness Alaoui" <[EMAIL PROTECTED]>
To: <[email protected]>
Sent: Tuesday, January 24, 2006 7:06 AM
Subject: Re: [Amsn-devel] Fwd: [Amsn-commits] msn chatwindow.tcl,1.208,1.209
gui.tcl,1.1196,1.1197


well, that's one solution, another one would be to transfer :

$pgBuddy.text tag bind $user_unique_name <Enter> +[list balloon_enter %W
%X %Y $balloon_message [::skin::getDisplayPicture $user_login]]

into

$pgBuddy.text tag bind $user_unique_name <Enter> +[list balloon_enter %W
%X %Y $balloon_message \[::skin::getDisplayPicture $user_login\]]

OR into something like this (not sure of the syntax) :
$pgBuddy.text tag bind $user_unique_name <Enter> +[subst -nocommands
{balloon_enter %W
%X %Y $balloon_message [::skin::getDisplayPicture $user_login]}]


KaKaRoTo


On Mon, 23 Jan 2006 09:11:35 -0500, Arieh Schneier
<[EMAIL PROTECTED]> wrote:

> Ok, dw, alvalro gave me a hand. Just create a new proc which in turn will
> call both the balloon_enter and the getDisplayPicture (check commit soon
> :P).
>
> Lio.
>
> ----- Original Message -----
> From: "Arieh Schneier" <[EMAIL PROTECTED]>
> To: <[email protected]>
> Sent: Tuesday, January 24, 2006 12:35 AM
> Subject: Re: [Amsn-devel] Fwd: [Amsn-commits] msn
> chatwindow.tcl,1.208,1.209
> gui.tcl,1.1196,1.1197
>
>
>> This should be really simple but I just can't think of it.
>>
>> We have a line that reads like this:
>>
>>   $pgBuddy.text tag bind $user_unique_name <Enter> +[list balloon_enter
>> %W
>> %X %Y $balloon_message [::skin::getDisplayPicture $user_login]]
>>
>> Now the problem is that actually runs ::skin::getDisplayPicture when
>> creating the binding, however it should only run when the event actually
>> occurs. The tricky part is that $user_login should be evaluated at
>> binding
>> time to pass the actual user, not the variable which will not exist any
>> more.
>>
>> Can someone tell me what the line should be.
>>
>> Lio.
>>
>>
>> ----- Original Message -----
>> From: "Arieh Schneier" <[EMAIL PROTECTED]>
>> To: <[email protected]>
>> Sent: Monday, January 23, 2006 1:30 AM
>> Subject: Re: [Amsn-devel] Fwd: [Amsn-commits] msn
> chatwindow.tcl,1.208,1.209
>> gui.tcl,1.1196,1.1197
>>
>>
>> > I noticed the same problem last night for the bindings for the
>> balloons
> on
>> > the contactlist. Now it actually loads every display pic into memory
>> on
>> > startup - this uses more memory than necessary, and takes extra
>> unnecessary
>> > cpu when loading up the first time.
>> >
>> > If someone else doesn't fix this I will when I get the time.
>> >
>> > Lio.
>> >
>> > PS part of the reason of this email was as a reminder to myself that
>> it
>> > needs to be done.
>> >
>> > ----- Original Message -----
>> > From: "Youness Alaoui" <[EMAIL PROTECTED]>
>> > To: <[email protected]>
>> > Sent: Sunday, January 22, 2006 6:23 AM
>> > Subject: [Amsn-devel] Fwd: [Amsn-commits] msn
>> chatwindow.tcl,1.208,1.209
>> > gui.tcl,1.1196,1.1197
>> >
>> >
>> > Hello all,
>> > I just saw this commit and reminded me that I wanted to wrote to you
> about
>> > this...
>> > as you can see here, we had this bug for a long time.. we also have
>> > another critical bug...
>> > create a profile in amsn with a password (and remember-password
>> enabled)
>> > ... you can also just select a profile with password saved... exit
>> amsn,
>> > and open it again.. you now have that new profile selected by
>> default..
>> > click login as, select another profile, and login as that profile...
>> if
>> > you then click on the menu aMSN->open inbox, you will be signed in to
> the
>> > inbox of the other account, not yours...
>> > this is beause we use " " instead of { }
>> > wehich means the menu item had the following command associated to it
>> :
>> > open_inbox [EMAIL PROTECTED] mypassword
>> > instead of having :
>> > open_inbox [::config::getKey login] $password
>> >
>> > this is because we used :
>> > $menu add command -label "open inbox" -command "open_inbox
>> > [::config::getKey login] $password"
>> > so the [] command and $var will be substitued in the command value...
>> > please pay attention to this kind of code! I will try to fix that as
> well
>> > as the URL thing I talked about last time...
>> >
>> > KKRT
>> >
>> >
>> >
>> > ------- Forwarded message -------
>> > From: "Vivia Nikolaidou" <[EMAIL PROTECTED]>
>> > To: [EMAIL PROTECTED]
>> > Subject: [Amsn-commits] msn chatwindow.tcl,1.208,1.209
>> > gui.tcl,1.1196,1.1197
>> > Date: Thu, 19 Jan 2006 11:56:59 -0500
>> >
>> > Update of /cvsroot/amsn/msn
>> > In directory sc8-pr-cvs1.sourceforge.net:/tmp/cvs-serv29073
>> >
>> > Modified Files:
>> > chatwindow.tcl gui.tcl
>> > Log Message:
>> > Menu - open received files will work correctly when changing profiles.
>> > Thanx Youness for pointing out
>> >
>> > Index: chatwindow.tcl
>> > ===================================================================
>> > RCS file: /cvsroot/amsn/msn/chatwindow.tcl,v
>> > retrieving revision 1.208
>> > retrieving revision 1.209
>> > diff -C2 -d -r1.208 -r1.209
>> > *** chatwindow.tcl 19 Jan 2006 13:36:00 -0000 1.208
>> > --- chatwindow.tcl 19 Jan 2006 16:56:57 -0000 1.209
>> > ***************
>> > *** 1205,1209 ****
>> >    -command "::amsn::FileTransferSend \[::ChatWindow::getCurrentTab
> $w\]"
>> >    $msnmenu add command -label "[trans openreceived]..." \
>> > ! -command "launch_filemanager \"[::config::getKey receiveddir]\""
>> >    $msnmenu add separator
>> >
>> > --- 1205,1209 ----
>> >    -command "::amsn::FileTransferSend \[::ChatWindow::getCurrentTab
> $w\]"
>> >    $msnmenu add command -label "[trans openreceived]..." \
>> > ! -command {launch_filemanager "[::config::getKey receiveddir]"}
>> >    $msnmenu add separator
>> >
>> >
>> > Index: gui.tcl
>> > ===================================================================
>> > RCS file: /cvsroot/amsn/msn/gui.tcl,v
>> > retrieving revision 1.1196
>> > retrieving revision 1.1197
>> > diff -C2 -d -r1.1196 -r1.1197
>> > *** gui.tcl 19 Jan 2006 13:47:00 -0000 1.1196
>> > --- gui.tcl 19 Jan 2006 16:56:57 -0000 1.1197
>> > ***************
>> > *** 3294,3298 ****
>> >    .main_menu.file add command -label "[trans sendfile]..." -state
>> disabled
>> >    .main_menu.file add command -label "[trans openreceived]" \
>> > ! -command "launch_filemanager \"[::config::getKey receiveddir]\""
>> >    .main_menu.file add separator
>> >    .main_menu.file add command -label "[trans close]" -command
>> > "close_cleanup;exit"
>> > --- 3294,3298 ----
>> >    .main_menu.file add command -label "[trans sendfile]..." -state
>> disabled
>> >    .main_menu.file add command -label "[trans openreceived]" \
>> > ! -command {launch_filemanager "[::config::getKey receiveddir]"}
>> >    .main_menu.file add separator
>> >    .main_menu.file add command -label "[trans close]" -command
>> > "close_cleanup;exit"
>> >
>> >
>> >
>> > -------------------------------------------------------
>> > This SF.net email is sponsored by: Splunk Inc. Do you grep through log
>> > files
>> > for problems?  Stop!  Download the new AJAX search engine that makes
>> > searching your log files as easy as surfing the  web.  DOWNLOAD
>> SPLUNK!
>> >
>> http://sel.as-us.falkag.net/sel?cmd=lnk&kid=103432&bid=230486&dat=121642
>> > _______________________________________________
>> > Amsn-commits mailing list
>> > [EMAIL PROTECTED]
>> > https://lists.sourceforge.net/lists/listinfo/amsn-commits
>> >
>> >
>> >
>> > --
>> > KaKaRoTo
>> >
>> >
>> > -------------------------------------------------------
>> > This SF.net email is sponsored by: Splunk Inc. Do you grep through log
>> files
>> > for problems?  Stop!  Download the new AJAX search engine that makes
>> > searching your log files as easy as surfing the  web.  DOWNLOAD
>> SPLUNK!
>> >
>> http://sel.as-us.falkag.net/sel?cmd=lnk&kid=103432&bid=230486&dat=121642
>> > _______________________________________________
>> > Amsn-devel mailing list
>> > [email protected]
>> > https://lists.sourceforge.net/lists/listinfo/amsn-devel
>> >
>> >
>> >
>> > -------------------------------------------------------
>> > This SF.net email is sponsored by: Splunk Inc. Do you grep through log
>> files
>> > for problems?  Stop!  Download the new AJAX search engine that makes
>> > searching your log files as easy as surfing the  web.  DOWNLOAD
>> SPLUNK!
>> >
>> http://sel.as-us.falkag.net/sel?cmd=lnk&kid=103432&bid=230486&dat=121642
>> > _______________________________________________
>> > Amsn-devel mailing list
>> > [email protected]
>> > https://lists.sourceforge.net/lists/listinfo/amsn-devel
>> >
>>
>>
>>
>> -------------------------------------------------------
>> This SF.net email is sponsored by: Splunk Inc. Do you grep through log
> files
>> for problems?  Stop!  Download the new AJAX search engine that makes
>> searching your log files as easy as surfing the  web.  DOWNLOAD SPLUNK!
>> http://sel.as-us.falkag.net/sel?cmd=lnk&kid=103432&bid=230486&dat=121642
>> _______________________________________________
>> Amsn-devel mailing list
>> [email protected]
>> https://lists.sourceforge.net/lists/listinfo/amsn-devel
>>
>
>
>
> -------------------------------------------------------
> This SF.net email is sponsored by: Splunk Inc. Do you grep through log
> files
> for problems?  Stop!  Download the new AJAX search engine that makes
> searching your log files as easy as surfing the  web.  DOWNLOAD SPLUNK!
> http://sel.as-us.falkag.net/sel?cmd=lnk&kid=103432&bid=230486&dat=121642
> _______________________________________________
> Amsn-devel mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/amsn-devel



-- 
KaKaRoTo


-------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc. Do you grep through log files
for problems?  Stop!  Download the new AJAX search engine that makes
searching your log files as easy as surfing the  web.  DOWNLOAD SPLUNK!
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=103432&bid=230486&dat=121642
_______________________________________________
Amsn-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/amsn-devel



-------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc. Do you grep through log files
for problems?  Stop!  Download the new AJAX search engine that makes
searching your log files as easy as surfing the  web.  DOWNLOAD SPLUNK!
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=103432&bid=230486&dat=121642
_______________________________________________
Amsn-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/amsn-devel

Reply via email to