On Fri, 5 Dec 2025 10:32:14 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. > > Daisuke Yamazaki has updated the pull request incrementally with one > additional commit since the last revision: > > Fix wrong indentation in package-info.java src/jdk.httpserver/share/classes/com/sun/net/httpserver/package-info.java line 137: > 135: // statement above could throw IAE if any params invalid. > 136: // eg. if app has a UI and parameters supplied by a user. > 137: Suggestion: Suggestion is to delete the blank line ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/28660#discussion_r2598240195
