markt-asf commented on a change in pull request #207: Fix typos
URL: https://github.com/apache/tomcat/pull/207#discussion_r324573610
 
 

 ##########
 File path: java/javax/servlet/http/Part.java
 ##########
 @@ -48,9 +48,6 @@
     public String getContentType();
 
     /**
-     * Obtain the name of the field in the multipart form corresponding to this
-     * part.
-     *
      * @return The name of the field in the multipart form corresponding to 
this
 
 Review comment:
   -1 to this and all similar changes. The first sentence in the Javadoc is 
used as the method description in the table of methods. By removing this text, 
the description is lost from that table. While I am never a fan of duplication, 
in this case it is better to have the duplication than not.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

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

Reply via email to