aviProgrammer commented on a change in pull request #677:
URL: https://github.com/apache/commons-lang/pull/677#discussion_r546249952



##########
File path: src/main/java/org/apache/commons/lang3/RandomStringUtils.java
##########
@@ -309,8 +309,10 @@ public static String random(final int count, final int 
start, final int end, fin
      * @param count  the length of random string to create
      * @param start  the position in set of chars to start at
      * @param end  the position in set of chars to end before
-     * @param letters  only allow letters?
-     * @param numbers  only allow numbers?
+     * @param letters  if {@code true}, generated string may include
+     *  alphabetic characters
+     * @param numbers  if {@code true}, generated string may include
+     *  numeric characters

Review comment:
       Hi @kinow , Thanks for your observation. 
   The same fix of Javadoc has been applied at line 348 and 349. 
   
   Let me know in case you find any other improvements that can be done in the 
same Javadoc.




----------------------------------------------------------------
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:
[email protected]


Reply via email to