ノート

[BUG] 8366058: Outdated comment in WinCAPISeedGenerator

Windows 95 についての古いコメントを削除しました。

概要#

 Windows 向けの​ WinCAPISeedGenerator.c では,​MS CryptoAPI から​乱数シードを​取得する​ nativeGenerateSeed が​定義されています1。​ この​メソッドの​直前には,​以下のように​ Windows 95 の​初期バージ​ョンに​関する​コメントが​残っていました2

/*
 * Get a random seed from the MS CryptoAPI. Return true if successful, false
 * otherwise.
 *
 * Some early versions of Windows 95 do not support the required functions.
 * Use runtime linking to avoid problems.
 *
 */

 しかし,​JDK では​既に​ Windows 95 を​サポートしていません。​ その​ため,​現在の​実装を​理解するうえで​この​コメントは​有用ではなく,むしろ​「Windows 95 への​配慮が​まだ​必要なのか」と​いう​誤解を​招く​恐れが​ありました。

 今回の​修正では,​この​古くなった​ Windows 95 に​ついての​コメントだけを​削除しました3。​ コードの​挙動は​変更していないため,​テストを​追加する​種類の​変更ではなく,​JBS には​ noreg-doc を​付与しました。

2025/11/02 JBS で​発見#

 本バグを​ JBS で​発見し,​対象ファイルを​確認しました。​ 問題の​コメントは​ nativeGenerateSeed の​説明コメントの​一部と​して​残っていましたが,​現在サポートされる​ Windows とは​関係が​ありません。

 修正内容は​コメントの​削除のみです。​ 以下の​ 4 行を​削除し,​メソッド説明だけを​残しました4

- *
- * Some early versions of Windows 95 do not support the required functions.
- * Use runtime linking to avoid problems.
- *

 PR 作成直後は​ core-libs ラベルが​自動で​付与されましたが,​対象は​セキュリティ系の​ネイティブ実装である​ため,​Alan Bateman5 氏に​より​ security ラベルへ​変更されました。

2026/01/05 レビュー#

 Sean Mullan6 氏に​レビューを​いただき,​Approve されました。​ また,​あわせて​ JBS に​ noreg-doc ラベルを​付与するようコメントを​いただき,​追加しました。

 問題ない​ことを​確認して,​統合リクエストを​行いました。

2026/01/05 統合#

 Mullan6 氏に​ /sponsor を​していただき eee58547 と​して​ JDK へ​統合されました。


Footnotes#

  1. 2