On Wed, 31 May 2023 09:52:29 GMT, Tejesh R <[email protected]> wrote: >> This is a regression from fix >> [JDK-8281966](https://bugs.openjdk.org/browse/JDK-8281966): Absolute path of >> symlink is null in JFileChooser. The fix checks whether the file path is a >> symbolic link using `Files.isSymbolicLink()` method with path as input. In >> windows for specific folders like "This PC"/"Network"/"Libraries" the path >> value will be a hex values which causes InvalidPathException. In order to >> resolve the issue, since no other checks are available to validate the path >> of these folders, checking if the file is link firstly and then for symbolic >> link resolves the problem (since File.isLink() doesn't take path as input >> rather file is a parameter). Since every symbolic link is a link, this fix >> seems logical to me. >> The fix is tested in CI for regression and is green. The regression fix is >> also tested for confirmation and works fine. > > Tejesh R has updated the pull request incrementally with one additional > commit since the last revision: > > Updated based on review comments
Marked as reviewed by aivanov (Reviewer). src/java.desktop/windows/classes/sun/awt/shell/Win32ShellFolder2.java line 864: > 862: cachedIsLink = hasAttribute(ATTRIB_LINK); > 863: } > 864: Let's preserve this blank line. ------------- PR Review: https://git.openjdk.org/jdk/pull/13998#pullrequestreview-1452674513 PR Review Comment: https://git.openjdk.org/jdk/pull/13998#discussion_r1211395090
