On Mon, 3 Oct 2022 11:10:27 GMT, Alexey Ivanov <aiva...@openjdk.org> wrote:

>> Tejesh R has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   Updated based on review comments
>
> You say that the current directory in `JFileChooser` gets set to `null` 
> because of forbidden navigation to a folder which is not traversable. You 
> avoid the NPE by a null-check.
> 
> However, you should prevent the current directory to be set to `null`. If 
> navigation is not allowed, the current directory should remain unchanged 
> _instead of being reset to `null`_.

@aivanov-jdk I have updated as per the review comments. Please let me know if 
its fine. I have put up the mach5 test link in JBS for reference.

-------------

PR: https://git.openjdk.org/jdk/pull/10485

Reply via email to