Re: [twsocket] Improving TFtpServer

2005-12-06 Thread Teträm Corp
Angus Robertson - Magenta Systems Ltd a écrit : those are not FTP commands, but some Application specific commands. In fact, I'm making a service based on a FTP server. For example, commands are RESTART, RELOAD, and other more specific to my application. Rather than adding private

[twsocket] Improving TFtpServer

2005-12-05 Thread Teträm Corp
Currently, to add some custom commands to the TFtpServer, there're only two possibilities: - use the OnClientCommand event - override the TFtpServer and override the TriggerClientCommand Those two possibilities had the same problem: we have to put exactly (approximately) the same code as the

Re: [twsocket] Improving TFtpServer

2005-12-05 Thread Angus Robertson - Magenta Systems Ltd
- why two declarations? (may be for SSL but it seems not) It is because of SSL, there are more commands. - why 5? while it seems there're only two new commands for SSL (AUTH and PROT) which are not added to the FCmdTable with AddCommand (at least in TFtpServer) Can not recall exactly

Re: [twsocket] Improving TFtpServer

2005-12-05 Thread Teträm Corp
It is because of SSL, there are more commands. how are they added to the FCmdTable? using AddCommand or with direct access to FCmdTable? (I do not have the SSL source code) - why 5? while it seems there're only two new commands for SSL (AUTH and PROT) which are not added to the

Re: [twsocket] Improving TFtpServer

2005-12-05 Thread Francois Piette
- why 5? while it seems there're only two new commands for SSL (AUTH and PROT) which are not added to the FCmdTable with AddCommand (at least in TFtpServer) Can not recall exactly why I added that comment, except without adding five the array was too much. It could be the component

Re: [twsocket] Improving TFtpServer

2005-12-05 Thread Teträm Corp
Except that using TList complicates the code unnecessarily since the memory needs to be freed at some point, and it will lead to more changes in the source code, that need to be properly tested, all so we can continue to support compilers obsoleted six years ago. The changes needed seems