Yeah, sorry, I just saw it reported on the forums and fixed it, the weird thing 
is that I tested it a few times before commiting... 
anyways, yes, it's limit.. because I realized that we can send up to 1665 chars 
in a message
instead of the 400 limit we used
but it needs to include the msg header which contains the friendlyname.. it's 
pretty much fixed apart from the p4c that we calculcate to adjust our 
limit..
I also added a bit of 'loose end'... we will usually use 1400 chars instead of 
1665, just in case something gets added in the header or whatever, it 
will not overflow..
the good solution would be to move that chopping in chunks into the SendChatMsg 
proc in protocol.tcl. I might do that tonight..

KaKaRoTo

On Fri, Jan 11, 2008 at 06:53:04PM +0100, Álvaro J. Iradier wrote:
> It looks so, in rev 9350...
> 
> In fact that code will crash! Can you fix it, or should I?
> 
> Greets.
> 
> On Jan 11, 2008 6:34 PM, Karel Demeyer <[EMAIL PROTECTED]> wrote:
> > Should it be :
> >                 incr limit -[string length $friendlyname]
> > ?
> >
> > Younness, you comitted this ... ? :)
> >
> > Karel .
> >
> > 2008/1/11, Karel Demeyer < [EMAIL PROTECTED]>:
> > >                 incr -[string length $friendlyname]
> > >
> > > what should that be ? line 2787 ...
> > >
> > >
> > > Karel.
> > >
> >
> >
> > -------------------------------------------------------------------------
> > Check out the new SourceForge.net Marketplace.
> > It's the best place to buy or sell services for
> > just about anything Open Source.
> > http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
> > _______________________________________________
> > Amsn-devel mailing list
> > Amsn-devel@lists.sourceforge.net
> > https://lists.sourceforge.net/lists/listinfo/amsn-devel
> >
> >
> 
> 
> 
> -- 
> (:===========================================:)
>   Alvaro J. Iradier Muro - [EMAIL PROTECTED]
> 
> -------------------------------------------------------------------------
> Check out the new SourceForge.net Marketplace.
> It's the best place to buy or sell services for
> just about anything Open Source.
> http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
> _______________________________________________
> Amsn-devel mailing list
> Amsn-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/amsn-devel

-------------------------------------------------------------------------
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
_______________________________________________
Amsn-devel mailing list
Amsn-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/amsn-devel

Reply via email to