Anton,

>> I do not think that people use multiple log destinations at the
>> same time. Currently the string is allocated and concatenated
>> only when actually required.
>>
> But it's possible. My modification is more compact also.

Yes it is possible but rather unlikely IMO.

>> This is non-optimal as well.
>> You always concatenate and allocate the string even though it's
>> not used. That sure leads to smaller source code but only performs
>> faster _if you actually write to multiple log destinations.
>
> Where? I concatenate it _only_ if loAddStamp is set and it's the same
> as execute procedure with AddTimeStamp + Msg.

You do it always if loAddStamp is set regardless whether one of
the destination options is set and in case of loDestEvent the 
OnIcsLogEvent is assigned or not.

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