On 12/10/2012 08:54, Konstantin Kolinko wrote:
> 2012/10/12  <ma...@apache.org>:
>> Author: markt
>> Date: Thu Oct 11 20:29:01 2012
>> New Revision: 1397291
>>
>> URL: http://svn.apache.org/viewvc?rev=1397291&view=rev
>> Log:
>> Fill in some of the HTTP upgrade Javadoc
>>
>> Modified:
>>     tomcat/trunk/java/javax/servlet/http/HttpServletRequest.java
>>     tomcat/trunk/java/javax/servlet/http/HttpServletRequestWrapper.java
>>     tomcat/trunk/java/javax/servlet/http/ProtocolHandler.java
>>     tomcat/trunk/java/javax/servlet/http/WebConnection.java
>>
> 
>>
>>      /**
>> -     * TODO SERVLET 3.1
>> +     * Start the HTTP upgrade process and pass the connection to the 
>> provided
>> +     * protocol handler once the current request/response pair has completed
>> +     * processing.
>>       */
>> -    public abstract void upgrade(javax.servlet.http.ProtocolHandler handler)
>> -            throws java.io.IOException;
>> +    public void upgrade(ProtocolHandler handler) throws java.io.IOException;
> 
> It would be better to mark such new API as "@since Servlet 3.1"

Agreed. I've done this and cleaned up / filled-in the Servlet 3.0 and
Servlet 3.1 Javadoc for those classes. There is a fair amount of
clean-up required in the Javadoc. I'm aiming to fix these as I touch the
classes / have a spare 5 minutes to fill.

Mark


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to