Hi why is this difference necessary? Or maybe rather, why the need for the I-prefix on the "internal" interfaces?
/niklas /niklas Rana Bhattacharyya wrote: > Hi, > > This is the logic behind the interface name. > > 1. ftplet is basic interface package which will be > exposed to add custom ftplet. > > 2. The package interfaces has all the other interfaces > used in the server. Here all the observers will not > have I in front. Everything else will have I in front. > > So it bolis down to > > 1. No observer interface will have I prefix. > 2. No ftplet will have I prefix. > 3. Everything else will have I prefix. > > Thanks, > Rana Bhattacharyya > > > --- Niklas Gustavsson <[EMAIL PROTECTED]> wrote: > > >>Hi >> >>I've taken a quick look through the new code for >>Ftpserver commited by >>Rana. For the most of it I'm impressed and think it >>looks very good. I >>do have some minor comments though: >> >>1. Interfaces are inconsitently named, some starting >>with "I", some not. >>They are also somewhat randomly placed in different >>packages with some >>in a special "interfaces" package and some not. >>2. Logging, I'm not so sure that it's a good idea to >>invent our own >>logging solution. There are plenty out there that >>works great and are >>highly configurable. I think the cost of introducing >>a dependency is >>lower then that of writing our own logger. >>3. There is a great need for unit tests. I'll try to >>work on this. >>4. In general, the code could be brokern down into >>more replacable parts >>(think IoC). I'll try to work on this as well. >> >>Anyways, great work! >> >>/niklas >> >> >>------------------ >>Niklas Gustavsson >>[EMAIL PROTECTED] >>http://www.protocol7.com >> >> >> > > --------------------------------------------------------------------- > >>To unsubscribe, e-mail: >>[EMAIL PROTECTED] >>For additional commands, e-mail: >>[EMAIL PROTECTED] >> >> > > > > > > __________________________________ > Yahoo! Mail - PC Magazine Editors' Choice 2005 > http://mail.yahoo.com -- ------------------ Niklas Gustavsson [EMAIL PROTECTED] http://www.protocol7.com --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
