Possible bug in internal/index Chunks()

[biogo]

Possible bug in internal/index Chunks()

[kortschak]

Can you post a reproducer of the code that is failing for you and a minimal input data set? Please ensure that you are checking the error returned by idx.Chunks. internal.Index.Chunks should not return a nil []bgzf.Chunk without an error, but I suspect that it does in this case.

[kortschak]

Sorry, I misread what you are worried about. It is not possible to return a nil []bgzf.Chunk without saying why. If you use a nil []bgzf.Chunk as a parameter to bgzf.Reader.SetChunk it removes the chunk limitation on the reader which Iterator behaviour is based on. The invalid interval error is not hard (maybe it should have just used io.EOF, but it's done now), and it just there to inform the caller that the returned []bgzf.Chunk should not be used. idx.Chunks returns an error, this is documentation that it may fail and that the error should be checked before the returned []bgzf.Chunks is used.

[kortschak]

Are you satisfied with the answer above? If not, please respond, otherwise I'll close this as invalid in the next few days.