Unverified Commit 04a02c1b authored by Marco Borgeaud's avatar Marco Borgeaud Committed by GitHub
Browse files

[ASTMatchers] forCallable should not erase binding on success (#89657)

Do not erase Builder when the first check fails because it could succeed
on the second stack frame.

The problem was that `InnerMatcher.matches` erases the bindings when it
returns false. The appropriate solution is to pass a copy of the
bindings, similar to what `matchesFirstInRange` does.
parent 419e7b82
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment