Does IndexedFastaSequenceFile need to reset its channel in every read operation?

[magicDGS]

Concretely in the following codepath:

htsjdk/src/main/java/htsjdk/samtools/reference/IndexedFastaSequenceFile.java

Lines 285 to 297 in 854d736

	private static int readFromPosition(final SeekableByteChannel channel, final ByteBuffer buffer, long position) throws IOException {
	if (channel instanceof FileChannel) { // special case to take advantage of native code path
	return ((FileChannel) channel).read(buffer,position);
	} else {
	long oldPos = channel.position();
	try {
	channel.position(position);
	return channel.read(buffer);
	} finally {
	channel.position(oldPos);
	}
	}
	}

The channel is reset to the original position. It looks like a solution to maintain the state of the channel even in the case of failure, as the FileChannel.read does. After #1014, the reset is also done in the compressed reference. This could have performance effects, and it does not look completely necessary.