On Thu, 13 Jan 2022 10:30:07 GMT, Pavel Rappo <pra...@openjdk.org> wrote:

> - Most of the typos are of a trivial kind: missing whitespace.
> - If any of the typos should be fixed in the upstream projects instead, 
> please say so; I will drop those typos from the patch.
> - As I understand it,   in ImageInputStream and DataInput is an irrelevant 
> formatting artefact and thus can be removed.
> - &#39; is an apostrophe, which does not require to be encoded.

src/java.sql/share/classes/java/sql/BatchUpdateException.java line 58:

> 56:  * A JDBC driver implementation should use
> 57:  * the constructor {@code BatchUpdateException(String reason, String 
> SQLState,
> 58:  * int vendorCode, long []updateCounts, Throwable cause) } instead of

While we are here, is it more normal to say "long[] updateCounts", not 
separating the [] from the type.
Similarly at line 81, 118, 151, 247, 277, 318, 354.

-------------

PR: https://git.openjdk.java.net/jdk/pull/7063

Reply via email to