[twsocket] TIcsLogger proposal

2009-05-09 Thread Anton Sviridov
Arno, if you care about "slowing" string concatenations (although concatenation will be executed *anyway*: Time+Sep+Msg), then we could do something like this: FDateFormat, FTimeFormat, FDateTimeFormat: string; ... procedure SetDateTimeFormat(df, tf: string); begin FDateFormat := df; FTimeFor

Re: [twsocket] TIcsLogger proposal

2009-05-08 Thread Francois PIETTE
Francois, you're right, properties will go better. I'll be waiting for updates of Logger in SVN =) >>> >>> Won't it be much cleaner to just add a single property >>> "TimeStampFormatString : String"? That covered everything and can be >>> set to "hh:nn:ss:zzz " by default in order to main

Re: [twsocket] TIcsLogger proposal

2009-05-08 Thread Arno Garrels
Piotr Dałek wrote: > Hello! > That's true, but concatenating strings is slow and should be avoided whenever possible. In Anton's change there are three properties to form a simple DataTime string including the separator, this should be simplified IMO. >>> >>> "Format" could be

Re: [twsocket] TIcsLogger proposal

2009-05-08 Thread Piotr Dałek
Hello! >>> That's true, but concatenating strings is slow and should be avoided >>> whenever possible. In Anton's change there are three properties to >>> form a simple DataTime string including the separator, this should be >>> simplified IMO. >> >> "Format" could be used instead of concatenatio

Re: [twsocket] TIcsLogger proposal

2009-05-08 Thread Arno Garrels
Piotr Dałek wrote: > Hello! > >>> LogColumnSeparator is also useful. >>> It is clear that having a separate date format and time format is >>> not required. A single format string is enough because one can add >>> the LogColumnSeparator that format string (anyway, that's what >>> Anton's code do b

Re: [twsocket] TIcsLogger proposal

2009-05-08 Thread Piotr Dałek
Hello! >> LogColumnSeparator is also useful. >> It is clear that having a separate date format and time format is not >> required. A single format string is enough because one can add the >> LogColumnSeparator that format string (anyway, that's what Anton's >> code do by concatenation dateformat,

Re: [twsocket] TIcsLogger proposal

2009-05-08 Thread Arno Garrels
Francois PIETTE wrote: >>> Francois, you're right, properties will go better. I'll be waiting >>> for updates of Logger in SVN =) >> >> Won't it be much cleaner to just add a single property >> "TimeStampFormatString : String"? That covered everything and can be >> set to "hh:nn:ss:zzz " by defaul

Re: [twsocket] TIcsLogger proposal

2009-05-08 Thread Francois PIETTE
>> Francois, you're right, properties will go better. I'll be waiting >> for updates of Logger in SVN =) > > Won't it be much cleaner to just add a single property > "TimeStampFormatString : String"? That covered everything and can be > set to "hh:nn:ss:zzz " by default in order to maintain backwar

Re: [twsocket] TIcsLogger proposal

2009-05-08 Thread Arno Garrels
Anton Sviridov wrote: > Francois, you're right, properties will go better. I'll be waiting > for updates of Logger in SVN =) Won't it be much cleaner to just add a single property "TimeStampFormatString : String"? That covered everything and can be set to "hh:nn:ss:zzz " by default in order to m

Re: [twsocket] TIcsLogger proposal

2009-05-07 Thread Francois Piette
ware) http://www.overbyte.be - Original Message - From: "Anton Sviridov" To: Sent: Thursday, May 07, 2009 11:07 AM Subject: [twsocket] TIcsLogger proposal > Francois, you're right, properties will go better. I'll be waiting for updates of Logger in SVN =) &

[twsocket] TIcsLogger proposal

2009-05-07 Thread Anton Sviridov
Francois, you're right, properties will go better. I'll be waiting for updates of Logger in SVN =) -- Regards, Anton -- To unsubscribe or change your settings for TWSocket mailing list please goto http://lists.elists.org/cgi-bin/mailman/listinfo/twsocket Visit our website at http://www.overbyte.

Re: [twsocket] TIcsLogger proposal

2009-05-06 Thread Francois PIETTE
(ICS) http://www.overbyte.be - Original Message - From: "Anton Sviridov" To: Sent: Wednesday, May 06, 2009 12:14 PM Subject: [twsocket] TIcsLogger proposal >I have a little suggestion to improve TIcsLogger. The reason is that I wish >to have both date and time in l

[twsocket] TIcsLogger proposal

2009-05-06 Thread Anton Sviridov
I have a little suggestion to improve TIcsLogger. The reason is that I wish to have both date and time in logfile and don't care about millisecs. Also, I don't want to derive classes, override events and all that stuff, just want to make output format customizable. 1) add public variable var Ti