garydgregory commented on code in PR #486:
URL: https://github.com/apache/commons-io/pull/486#discussion_r1339989692
##########
src/main/java/org/apache/commons/io/FileUtils.java:
##########
@@ -2618,15 +2615,14 @@ public static String readFileToString(final File file,
final Charset charsetName
/**
* Reads the contents of a file into a String. The file is always closed.
*
- * @param file the file to read, must not be {@code null}
+ * @param file the file to read, must not be {@code null}
* @param charsetName the name of the requested charset, {@code null}
means platform default
* @return the file contents, never {@code null}
* @throws NullPointerException if file is {@code null}.
- * @throws FileNotFoundException if the file does not exist, is a
directory rather than a regular file, or for some
- * other reason cannot be opened for reading.
- * @throws IOException if an I/O error occurs.
+ * @throws IOException if an I/O error occurs, including when the file
does not exist, is a directory rather than a
+ * regular file, or for some other reason why the file cannot be
opened for reading.
* @throws java.nio.charset.UnsupportedCharsetException thrown instead of
{@link java.io
- * .UnsupportedEncodingException} in version 2.2 if the named charset is
unavailable.
+ *
.UnsupportedEncodingException} in version 2.2 if the named charset is
unavailable.
Review Comment:
Hi @chkpnt
Thank you for your PR.
Please don't change the formatting, it makes it harder to review the actual
changes.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
