On Tue, 23 Mar 2021 13:39:06 GMT, Coleen Phillimore <cole...@openjdk.org> wrote:

>> I kind of thought these would make a long complicated expression and so the 
>> single use variables is helpful.  I don't think performance is important 
>> here.
>
> {code}
>       -      match = compare_entry_to(recur1, cp2, recur2);
>       -      if (match) {
>      +      if (compare_entry_to(recur1, cp2, recur2)) {
>               return true;
>             }
> I could do this

You can't return `true` after just `compare_entry_to()` because you still
have to check `compare_operand_to()`.

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

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

Reply via email to