Is there a reason why we do this once per SAMRecord, rather than (first) collapsing all the ADAMRecords down into a SAMFileHeader and then passing that value in (calculated once) as an argument to this method?