true, good point! :) but in practice, the failed is set to 1 when fontformat is 
set to "", so it's ok ;) but 
I'll change it anyways, just to make it a clean code.
thanks for the correction! :)

KKRT

On Fri, Feb 02, 2007 at 07:26:25AM +0100, square87 wrote:
> Hello.
> KKRT
> You have added
> 
> proc PutLog { chatid user msg {fontformat ""} {failed 0}} {
>        if {$fontformat == ""} {
>                 set color "NOR"
>        } elseif { $failed == 1 } {
>                 set color "RED"
>                # When the message failed to deliver, we should show the
> deliverfail message instead of the user's nickname.
>              set user [trans deliverfail]
>         } else {
>                   set color "C[lindex $fontformat 2]"
>         }
> if {[::OIM_GUI::IsOIM $chatid]} {
> 
> But it should be:
> 
>        if {$fontformat == ""} {
>            set color "NOR"
>        } else {
>            set color "C[lindex $fontformat 2]"
>        }
>        if { $failed == 1 } {
>                      set color "RED"  ;# When the message failed to
> deliver, we should show the deliverfail message instead of the user's
> nickname.
>                set user "\|\"LRED[trans deliverfail]"
>        }
>        if {[::OIM_GUI::IsOIM $user]} {
> 
> PS. I am Square87 and not Square86 :)
> G'day ( I have an exam! :S )

> -------------------------------------------------------------------------
> Using Tomcat but need to do more? Need to support web services, security?
> Get stuff done quickly with pre-integrated technology to make your job easier.
> Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
> http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
> _______________________________________________
> Amsn-devel mailing list
> Amsn-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/amsn-devel


-------------------------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier.
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
_______________________________________________
Amsn-devel mailing list
Amsn-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/amsn-devel

Reply via email to