On Thu, 4 Dec 2025 16:19:07 GMT, Daisuke Yamazaki <[email protected]> wrote:
> Hello mainteners,
>
> I cleaned up inconsistent spacing across the httpserver codebase.
>
> 1. Removed extra spaces before parentheses in function and constructor
> declarations to make them consistent.
> 2. Applied the same rule to function and constructor calls.
> 3. Normalised other spacing based on the majority style already used in the
> codebase.
> 4. Left spots untouched where keeping the spacing improves readability.
>
> **This PR only adds or removes spaces. No functional changes.**
>
> If you think it would have been better to update only the documentation part
> and leave the logic untouched, please let me know.
> I split the commits, so I can easily revert the one that touches the logic.
src/jdk.httpserver/share/classes/com/sun/net/httpserver/package-info.java line
123:
> 121: public void configure (HttpsParameters params) {
> 122:
> 123: // get the remote address if needed
Always hard to tell, but I _think_ the indentation of this whole block (the
body of the `configure` method) is still off - all the lines 123-136 should be
indented 4 more space?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/28660#discussion_r2591122288
