RFR: 8260401: StackOverflowError on open WindowsPreferences
Brian Burkhalter
bpb at openjdk.java.net
Tue Feb 2 02:43:57 UTC 2021
On Tue, 2 Feb 2021 02:31:09 GMT, Jaikiran Pai <jpai at openjdk.org> wrote:
>> Hello Brian,
>>
>> Thank you for the review.
>>
>>> It would be better if there were a test, but apparently this might depend on the user who runs the test not having registry access rights.
>>
>> That's correct. Looking at the code, it looks to me that this will require very specific setup of the Windows system to be able to trigger the error.
>>
>>> The code change looks all right.
>>
>> Should I go ahead and integrate this?
>
>> > The code change looks all right.
>>
>> Should I go ahead and integrate this?
>
> Actually, I didn't notice that this PR wasn't marked as reviewed. I'll wait for the review(s) then.
I'd let it sit for a bit in case others want to comment.
-------------
PR: https://git.openjdk.java.net/jdk/pull/2326
More information about the core-libs-dev
mailing list