Should ItemVisitor.visitItem() be boolean for early-exit?

[micycle1]

I think the visitItem() method in org.locationtech.jts.index.ItemVisitor the class should not be void. Instead it should be a boolean that allows early-termination by the caller (false is "don't continue").

One use case is using a spatial index for potential point-in-area queries. Rather than collecting all candidates then running point-in-area tests, one can run the test on each candidate within visitItem(), and early exit upon finding the first match. Here we'd save work searching the indexed structure.