On Fri, 22 Apr 2022 19:10:38 GMT, Jonathan Gibbons <j...@openjdk.org> wrote:
> Please review a moderately simple cleanup change, to eliminate using an > exception to terminate scanning a `Tree` or `DocTree` in > `(Doc)TreePath.getPath`. > > The change is to set a field with the intended result, and once that field is > set, do "best-effort" to eliminate any additional scanning. > > It is easy enough to stop scanning items in a list, but it is not practical > to totally stop scanning the subsequent sibling nodes, but we can > substantially reduce the cost of scanning those nodes, and can definitely > avoid scanning any children of those nodes. This pull request has now been integrated. Changeset: 80a7f7be Author: Jonathan Gibbons <j...@openjdk.org> URL: https://git.openjdk.java.net/jdk/commit/80a7f7becfc362fc3918a2442aea912533373c26 Stats: 69 lines in 2 files changed: 26 ins; 9 del; 34 mod 8267690: Revisit (Doc)Tree search implemented by throwing an exception Reviewed-by: vromero, prappo ------------- PR: https://git.openjdk.java.net/jdk/pull/8369
