RFR: 8372746: Some httpserver files could benefit from some formatting cleanup [v4]

Daniel Fuchs dfuchs at openjdk.org
Tue Dec 9 18:01:15 UTC 2025


On Tue, 9 Dec 2025 11:30:51 GMT, Daisuke Yamazaki <dyama at openjdk.org> 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:
> 
>   Add a missing parenthesis
>   
>   Co-authored-by: Daniel Jelinski <djelinski1 at gmail.com>

I have looked at the changes and they look good. Thanks for the cleanup!
I have sent this change to the CI for a last round of checking and tests came back green.

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

Marked as reviewed by dfuchs (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/28660#pullrequestreview-3558842903


More information about the net-dev mailing list