Added the recent suggestions to VelocityServlet. Note : 1) I didn't move setContentType() - since this is 'contextual' (it appears that it can matter when and where you call it), we can leave this as is, and the clueful can deal - just override setContentType() to do nothing, and deal with it in the handleRequest(). 2) I think I was able to preserve the behavior of the existing VelServlet wrt returning null from handleRequest( Context) - if you don't implement either handleRequest() then the default impl of handleContext( Context) throws an exception telling you do override *something* - if you implement handleRequest( Context ) and return null, then the current release behavior is preserved in that an exception is thrown with a message indicating that you didn't return a template. - if you implement handleContext( Req, resp, ctx ), then you can return null w/o any processing happening afterwards. I think I got it right. I tested a little even :) geir -- Geir Magnusson Jr. [EMAIL PROTECTED] Developing for the web? See http://jakarta.apache.org/velocity/
