<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&#39;t think any will.  However, I don&#39;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>