On Mon, 23 Jun 2025 23:09:13 GMT, simon <d...@openjdk.org> wrote:

>> 8360122: Refine formatting of Connection.java interface
>> 
>> ---------
>> ### Progress
>> - [ ] Change must be properly reviewed (1 review required, with at least 1 
>> [Reviewer](https://openjdk.org/bylaws#reviewer))
>> - [x] Change must not contain extraneous whitespace
>> - [x] Commit message must refer to an issue
>> 
>> 
>> 
>> ### Reviewing
>> <details><summary>Using <code>git</code></summary>
>> 
>> Checkout this PR locally: \
>> `$ git fetch https://git.openjdk.org/jdk.git pull/25925/head:pull/25925` \
>> `$ git checkout pull/25925`
>> 
>> Update a local copy of the PR: \
>> `$ git checkout pull/25925` \
>> `$ git pull https://git.openjdk.org/jdk.git pull/25925/head`
>> 
>> </details>
>> <details><summary>Using Skara CLI tools</summary>
>> 
>> Checkout this PR locally: \
>> `$ git pr checkout 25925`
>> 
>> View PR using the GUI difftool: \
>> `$ git pr show -t 25925`
>> 
>> </details>
>> <details><summary>Using diff file</summary>
>> 
>> Download this PR as a diff file: \
>> <a 
>> href="https://git.openjdk.org/jdk/pull/25925.diff";>https://git.openjdk.org/jdk/pull/25925.diff</a>
>> 
>> </details>
>> <details><summary>Using Webrev</summary>
>> 
>> [Link to Webrev 
>> Comment](https://git.openjdk.org/jdk/pull/25925#issuecomment-2993856738)
>> </details>
>
> simon has updated the pull request incrementally with one additional commit 
> since the last revision:
> 
>   8360122: revert reformatting method signatures

Some initial comments.

There are some lines in the methods that were modified that need to wrap around 
80 characters, but it would be best to leave as is and can be done as part of 
the updates that will be coming shorty

src/java.sql/share/classes/java/sql/Connection.java line 303:

> 301:     
> //======================================================================
> 302:     // Advanced features:
> 303: 

Why was this removed as this is not format related

src/java.sql/share/classes/java/sql/Connection.java line 1175:

> 1173:      */
> 1174:     void setClientInfo(String name, String value)
> 1175:         throws SQLClientInfoException;

The throws clause could be moved up

src/java.sql/share/classes/java/sql/Connection.java line 1208:

> 1206:      */
> 1207:     void setClientInfo(Properties properties)
> 1208:         throws SQLClientInfoException;

The throws clause could be moved up

src/java.sql/share/classes/java/sql/Connection.java line 1232:

> 1230:      * @see java.sql.DatabaseMetaData#getClientInfoProperties
> 1231:      */
> 1232:     String getClientInfo(String name)

the throws clause could be moved up

src/java.sql/share/classes/java/sql/Connection.java line 1251:

> 1249:      */
> 1250:     Properties getClientInfo()
> 1251:         throws SQLException;

The throws clause could be moved up

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

Changes requested by lancea (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/25925#pullrequestreview-2979431612
PR Review Comment: https://git.openjdk.org/jdk/pull/25925#discussion_r2180233608
PR Review Comment: https://git.openjdk.org/jdk/pull/25925#discussion_r2180324024
PR Review Comment: https://git.openjdk.org/jdk/pull/25925#discussion_r2180322060
PR Review Comment: https://git.openjdk.org/jdk/pull/25925#discussion_r2180318581
PR Review Comment: https://git.openjdk.org/jdk/pull/25925#discussion_r2180317869

Reply via email to