in terms of the performance impact, just changing this to a fixed number of 10, achieves most of what the pull achieves, right?

the pull also adds the seen_locations logic, the reraise logic, and a better design for unit testing, but just want to be clear we're on the same page as for what's helping the performance.   

e.g. just changing this to 10 has the same effect on dropping install time for coverage (the example that @hltbra listed)