RFR: 8358530: Properties#list should warn against non-String values [v2]
cagliostro92
duke at openjdk.org
Wed Jul 16 13:46:29 UTC 2025
On Tue, 15 Jul 2025 16:29:11 GMT, Justin Lu <jlu at openjdk.org> wrote:
>> cagliostro92 has updated the pull request incrementally with one additional commit since the last revision:
>>
>> 8358530: adds warnings against non-String values in Properties class documentation
>
> src/java.base/share/classes/java/util/Properties.java line 78:
>
>> 76: * the call to the {@code propertyNames} or {@code list} method
>> 77: * will fail if it is called on a "compromised" {@code Properties}
>> 78: * object that contains a non-{@code String} key or value.
>
> Since `propertyNames` won't throw `CCE` with a non-String value, instead of adding the "or value" wording here, we should move the `list` method to the sentence above and remove it from the current sentence.
>
> E.g., on line 73
>
>> "If the {@code store}, {@code save}, or {@code list) method ...
My bad, sorry. Fixed with 8d7e3d2fde65bd1e4ad85eca5ea93a430e2e8238
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/25334#discussion_r2210484938
More information about the core-libs-dev
mailing list