On Fri, Mar 6, 2009 at 10:40 AM, Leigh Klotz <[email protected]> wrote:
> > I've gone back to using a copy of InjectedHttpServlet from the > previous snapshot, and am whole again with onyl a small amount of > code. > > I think the new servlet stuff is cool for people who are using > servlets as a programming methodology. Nope it's for everyone. Jesse made sure I did not exclude anyone =) > > > However, for people like me who aren't writing new servlets, merely > using a servlet as an adapter between an existing system and something > better (Restlet, in my case), it's too much engineering for too little > benefit: > > - It makes it difficult for me to get the Injector for use by Tim > Peirels' Restlet integration Hmm, I'll ask Tim if I can update his restlet thing for GS2. He will be loads happier to use ctor injection I am sure! (Tim are you still on this list?) > > - There's no clear place to put the warm-up code that currently goes > into myservlet.init, other than a series of eager singletons with > sculpted dependencies. > - Legacy app developers who look for my entry-point servlets in > web.xml will be puzzled when they can't find anything. > > Personally, my preference would be to see the Guice servlet > functionality split into two parts > - one for bare support for injection of servlets (InjectedHTTPServlet, > GuiceServletContextListener) We already have this--InjectedHttpServlet is evil, GS2 has this support natively coz servlets are in the club. > > - a second with the cool new dispatch, Request/Response/Session > injection, and scopes. it's totally symmetrical to web.xml so I don't get how you can have a servlet without registering it for dispatch (in either GS1 or 2). =( Dhanji --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "google-guice" group. To post to this group, send email to [email protected] To unsubscribe from this group, send email to [email protected] For more options, visit this group at http://groups.google.com/group/google-guice?hl=en -~----------~----~----~----~------~----~------~--~---
