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