Minor nits: can we avoid the first toString call here, and instead call subSequence directly on the CharSeqence that getSequence returns? Also, instead of "startPos" and "endPos", mind calling it something that makes it clear this is an offset into the sequence read, not reference loci (and avoids abbreviations)? Maybe startReadOffset and endReadOffset? Thanks @arahuja