On Thu, 13 Jan 2022 10:30:07 GMT, Pavel Rappo <[email protected]> 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.
> - ' 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