From 8ad43dd70c21cfdd6a965a528363511b52d25c5f Mon Sep 17 00:00:00 2001 From: On behalf of John Marshall Date: Wed, 22 Sep 2010 14:47:42 +0000 Subject: [PATCH] 0 is a successful return value from bam_read1(). (In practice, it never returns 0 anyway; but all the other callers treat 0 as successful.) --- bam_index.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/bam_index.c b/bam_index.c index e218a70..9d819d3 100644 --- a/bam_index.c +++ b/bam_index.c @@ -674,7 +674,7 @@ int bam_iter_read(bamFile fp, bam_iter_t iter, bam1_t *b) } ++iter->i; } - if ((ret = bam_read1(fp, b)) > 0) { + if ((ret = bam_read1(fp, b)) >= 0) { iter->curr_off = bam_tell(fp); if (b->core.tid != iter->tid || b->core.pos >= iter->end) break; // no need to proceed else if (is_overlap(iter->beg, iter->end, b)) return ret; -- 2.39.5