Looks fine Brian!
On 11/21/2018 12:19 PM, Brian Burkhalter wrote:
Hello, again,
In response to comments on the CSR
https://bugs.openjdk.java.net/browse/JDK-8214072
the following change to the verbiage is proposed:
--- a/src/java.base/share/classes/java/io/InputStream.java
+++ b/src/java.base/share/classes/java/io/InputStream.java
@@ -562,7 +562,8 @@
* in an inconsistent state. It is strongly recommended that the
* stream be promptly closed if an I/O error occurs.
*
- * <p> Subclasses are encouraged to provide a more efficient implementation
+ * @implNote
+ * Subclasses are encouraged to provide a more efficient implementation
* of this method.
*
* @implSpec
@@ -572,7 +573,8 @@
* then {@link #read()} is invoked repeatedly until the stream is {@code
n}
* bytes beyond its position when this method was invoked or end of stream
* is reached. If the return value of {@code skip(n)} is negative or
- * greater than {@code n}, then an {@code IOException} is thrown.
+ * greater than {@code n}, then an {@code IOException} is thrown. Any
+ * exception thrown by {@code skip()} or {@code read()} will be propagated.
*
* @param n the number of bytes to be skipped.
* @throws EOFException if end of stream is encountered before the
Thanks,
Brian
On Nov 16, 2018, at 5:25 PM, Brian Burkhalter <brian.burkhal...@oracle.com>
wrote:
So updated in place.
http://cr.openjdk.java.net/~bpb/6516099/webrev.07/
<http://cr.openjdk.java.net/~bpb/6516099/webrev.07/>