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