if (_LowerCase(FProtoStr) <> 'tcp') and (_Trim(FProtoStr) <> '6') then
begin
RaiseException('TCP is the only protocol supported thru socks
server'); { V5.26 }
Exit;
end;
a) looks quite weird, as it likely to throw exception in every case
(FProtoStr can't be equal to 'tcp' and '6' at the same time)
This is an "and" not an "or". The exception is raised when FProtoStr is
neither 'tcp' no '6'.
b) UDP through Socks5 is possible through UDP ASSOCIATE command instead
of CONNECT,
as well as listening is available through BIND command
Probably. Maybe you'll implement those features ?
2)
begin
FBufHandler.Free;
FBufHandler := nil;
end;
=> FreeAndNil ? Much shorter
Shorter in code, but slower. Note that this has been written at a timle
where FreeAndNil didn't existed.
3)
TriggerError { Should be modified to pass Msg ! }
it really should be :)
The problem is that is would break existing code. On of the gold rules in
ICS has always been and will always be to avoid as much as possible breaking
existing code. The error message, if required, should be passed thru a
property for example. This wouldn't break any existing code.
--
francois.pie...@overbyte.be
http://www.overbyte.be
--
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
