garydgregory commented on a change in pull request #127:
URL: https://github.com/apache/commons-csv/pull/127#discussion_r549826167



##########
File path: src/main/java/org/apache/commons/csv/CSVFormat.java
##########
@@ -1628,7 +1628,8 @@ private CharSequence trim(final CharSequence 
charSequence) {
     /**
      * Verifies the consistency of the parameters and throws an 
IllegalArgumentException if necessary.
      *
-     * @throws IllegalArgumentException
+     * @throws IllegalArgumentException If something goes wrong

Review comment:
       Hi @arturobernalg 
   This change does not help anyone, and it's confusing when the line above 
tells you when the exception would be thrown. I've updated git master for this 
Javadoc. Please rebase on master. TY.
   




----------------------------------------------------------------
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