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 default in order to maintain backwards
>> compatibility.
> 
> 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, columnsep and time format).

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.

--
Arno Garrels


-- 
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.be

Reply via email to