<div dir="ltr">There are a number of places where I&#39;ve found issues with the versifications.<div><br></div><div style>The methods in AbstractPassage should take account of the v11n passed in:</div><div style>e.g. addAll, add, remove, retain, etc. If the passed in parameter is of a different versification, we should deal with this properly. Currently we end up with very strange things where keys suddenly mean something completely different.</div>
<div style><br></div><div style>We should remove VerseRange(v11n, Verse), or at the very least throw an exception if the v11n is different to Verse.v11n. There are a number of examples where this is the case.</div><div style>
<br></div><div style>I think we can simply a lot of the constructors asking for a v11n. Since, if we&#39;re passing something in, then we can determine the versification from the passed in object.  The main issue (above) is that we&#39;re can&#39;t guarantee that a verse maps to a range. But we could do our best to create whatever is passed in.</div>
<div style><br></div><div style>Chris</div><div style><br></div></div>