remove more c/c++ confusion
remove missing increment operator

got stripped out during refactoring. be careful!
move bam reading code to its own module in preparation to replace it

now we can test it in isolation to make sure the new version behaves
similarly.