RFR: 8294361: Cleanup usages of StringBuffer in SQLOutputImpl

Roger Riggs rriggs at openjdk.org
Mon Sep 26 15:03:21 UTC 2022


On Sat, 17 Sep 2022 21:43:04 GMT, Andrey Turbanov <aturbanov at openjdk.org> wrote:

> There a few unnecessary usages of StringBuffer in SQLOutputImpl class.
> All of them create StringBuffer from single character and then concatenate with String. Instead of this, we can concatenate character directly.

src/java.sql.rowset/share/classes/javax/sql/rowset/serial/SQLOutputImpl.java line 328:

> 326:         try {
> 327:             int i;
> 328:             while ((i = bufReader.read()) != -1) {

Is the single character read really necessary?
BufferedReader.readLine() returns null on EOF.
It seems plausable that only the readLine and writeString are needed in the loop.

Similarly, for the read loops below for InputStreams and binary stream.
The BufferedReader and readline can be used for the binary streams by constructing with the ASCII Charset.

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

PR: https://git.openjdk.org/jdk/pull/10320


More information about the core-libs-dev mailing list