yamad のホーム・ページ

ノート

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

jdk.httpserver の複数ファイルに残っていた空白まわりの不揃いを整理しました。

概要#

　jdk.httpserver には，​古くから​残っている​空白まわりの​不揃いが​いくつか​ありました。​ た​とえば，​メソッド呼び出しや​コンストラクタ呼び出しで，​メソッド名と​ ( の​間に​空白が​入っている​箇所です1。

return create (null, 0);

　現在の​多くの​ Java コードでは​この​空白は​入れないため，​今回の​修正では​以下のように​整えました2。

return create(null, 0);

　また，​JavaDoc の​ @link で​引数リストの​カンマ後に​空白が​ない​箇所や，Map<String,List<String>> のように​型引数の​カンマ後に​空白が​ない​箇所も​整理しました。​ 変更対象は​ com.sun.net.httpserver と​ sun.net.httpserver 配下を​中心と​する​ 36 ファイルです3。​ 変更箇所が​膨大で，​単純ながら骨の​折れる​作業でした。

　この​ PR は​空白と​整形の​修正のみで，​機能変更は​ありません。

2025/12/04 JBS で​発見#

　本件を​ JBS で​発見し，jdk.httpserver 配下の​ファイルを​確認しました。​ Authenticator，BasicAuthenticator，Filter，HttpServer，ServerImpl など，​複数の​ファイルで​同じ​種類の​空白の​不揃いが​残っていました。

　修正方​針は，​以下の​ 3 点です。

  1. メソッド名・​コンストラクタ名と​ ( の​間に​ある​余分な​空白を​削除する
  2. JavaDoc の​リンクや​型引数の​カンマ後に​空白を​補う
  3. 読みやすさの​ために​意図的に​残されている​整形は​無理に​触らない

　Pull Request 作成時には，​「空白の​追加・削除のみで，​機能変更は​ない」ことを​明記しました4。

2025/12/09 レビュー#

　Mikael Vidstedt 氏5，​Michael McMahon 氏6，​Daniel Jeliński 氏7，​Daniel Fuchs 氏8に​レビューを​いただきました。

　Fuchs 氏8からは，​変更内容を​確認したうえで​ CI に​回し，​テストが​ green だったとの​コメントを​いただきました。​ また，​Jeliński 氏7と​ Fuchs 氏8から​ Approve されました。

NOTE

レビュアーの​アベンジャーズみたいで​楽しかったです。

　Jaikiran Pai 氏9から，​変更した​ファイルの​著作権表示の​年も​更新する​必要が​あるとの​指摘を​いただきました。​ 私は，​非本質的な​変更が​発生した​時に​年号を​変えるのだ，と​思っていましたが，​それに​ついて​訊くと，​変更の​種類に​かかわらず更新年を​反映するのが​期待されるとの​説明を​受けました。

　​その​ため，​対象ファイルの​ copyright 年を​ 2025 に​更新しました。​ 一部の​ファイルに​ついては​漏れを​追加で​指摘いただき，​それも​反映しました。

　今回の​ PR で​触っていない​ファイルの​古い​ copyright 年は，​この​ PR では​更新せず，​将来​その​ファイルを​変更する​際に​更新すれば​よい，と​いう​整理に​なりました。

2025/12/12 統合リクエスト#

　Jaikiran 氏9から​ This looks good to me. と​して​ Approve されました。​ その​後，/integrate を​行い，b35243510 の​変更が​スポンサー待ちと​なりました。

2025/12/12 統合#

　Jaikiran 氏9に​ CI を​再実行していただき，​問題が​ない​ことを​確認したのち，/sponsor を​していただきました。​ 最終的に​ 180d8c111 と​して​ JDK へ​統合されました。

Footnotes#

  1. jdk/src/jdk.httpserver/share/classes/com/sun/net/httpserver/HttpServer.java at 2735140147b159d3a3238804f221db4f835ef744…JDK main-line development https://openjdk.org/projects/jdk - jdk/src/jdk.httpserver/share/classes/com/sun/net/httpserver/HttpServer.java at 2735140147b159d3a3238804f221db4f835ef74…GitHub
     ↩
  2. 8372746: Some httpserver files could benefit from some formatting cleanup by PeyaPeyaPeyang · Pull Request #28660 · ope…Hello mainteners, I cleaned up inconsistent spacing across the httpserver codebase. Removed extra spaces before parentheses in function and constructor declarations to make them c…GitHub
     ↩
  3. 8372746: Some httpserver files could benefit from some formatting cleanup by PeyaPeyaPeyang · Pull Request #28660 · ope…Hello mainteners, I cleaned up inconsistent spacing across the httpserver codebase. Removed extra spaces before parentheses in function and constructor declarations to make them c…GitHub
     ↩
  4. 8372746: Some httpserver files could benefit from some formatting cleanup by PeyaPeyaPeyang · Pull Request #28660 · ope…JDK main-line development https://openjdk.org/projects/jdk - 8372746: Some httpserver files could benefit from some formatting cleanup by PeyaPeyaPeyang · Pull Request #28660 · op…GitHub
     ↩
  5. https://openjdk.org/census#mikaelopenjdk.org
     ↩
  6. https://openjdk.org/census#michaelmopenjdk.org
     ↩
  7. https://openjdk.org/census#djelinskiopenjdk.org
     ↩ ↩2
  8. https://openjdk.org/census#dfuchsopenjdk.org
     ↩ ↩2 ↩3
  9. https://openjdk.org/census#jpaiopenjdk.org
     ↩ ↩2 ↩3
  10. Add missed copyright year updates to HTTP server files · openjdk/jdk@b352435JDK main-line development https://openjdk.org/projects/jdk - Add missed copyright year updates to HTTP server files · openjdk/jdk@b352435GitHub
     ↩
  11. 8372746: Some httpserver files could benefit from some formatting cle… · openjdk/jdk@180d8c1JDK main-line development https://openjdk.org/projects/jdk - 8372746: Some httpserver files could benefit from some formatting cle… · openjdk/jdk@180d8c1GitHub
     ↩