On Fri, 8 Apr 2022 16:20:10 GMT, Phil Race <p...@openjdk.org> wrote:

> why did you get rid of the just in case ?

I honestly couldn't find any way for this code to get executed. I can keep it 
there, but then I would have to explain why it is needed.

> Actually why is this method being modified at all ?
The changes are really semantically null. All I did was shorten a 30 lines 
method to 15 and gave a name to a piece of it (but that's in the next comment).

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

PR: https://git.openjdk.java.net/jdk/pull/7182

Reply via email to