<div dir="ltr">When I switched to using RangedPassage instead of RocketPassage by default it improved memory usage but there are 8 specific uses of RocketPassage in VersificationToKJVMapper.<div>E.g.</div><div>303: matchingVerses = new RocketPassage(this.nonKjv);</div>
<div>409: newKey = new RocketPassage(versification);</div><div>503: return new QualifiedKey(new RocketPassage(target));</div><div>Etc.</div><div><div><br></div><div>Is there any reason why any of the 8 specific references to RocketPassage in VersificationToKJVMapper should not use the default passage class returned by PassageKeyFactory.createEmptyKeyList()? I could imagine that if any of the Passages would be likely to contain a large number of verses then a RocketPassage would be better, but I don't think any will. However, I don't have enough familiarity with the code to be sure.</div>
</div><div><br></div><div>I am actually getting an OutOfMemoryError occasionally when testing on a virtual device caused by the occurrence on line 303 and possibly elsewhere.</div><div><br></div><div>Martin</div></div>