On Tue, 17 Jun 2025 21:43:06 GMT, Valerie Peng <[email protected]> wrote:
>> Weijun Wang has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> more random testing
>
> src/java.security.jgss/share/classes/sun/security/krb5/Config.java line 678:
>
>> 676: Path fullp = Paths.get(fileName).toAbsolutePath();
>> 677: Path path = Paths.get(fileName);
>> 678: if (!Files.exists(path)) {
>
> Is `path` needed? Can't you just use `fullp` for the `!Files.exists()` check?
You're right. I'll remove it.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/25421#discussion_r2155596225
