From: derek <derekwbarnett@gmail.com>
Date: Fri, 14 Oct 2011 05:14:30 +0000 (-0400)
Subject: Regression fixed: wasn't properly merging from multiple BAMs
X-Git-Url: https://git.donarmstrong.com/?p=bamtools.git;a=commitdiff_plain;h=6a75f0d886c44d98a6fba6513d2f3e9f158659ed

Regression fixed: wasn't properly merging from multiple BAMs
---

diff --git a/src/api/internal/BamMultiReader_p.cpp b/src/api/internal/BamMultiReader_p.cpp
index ecf2839..55ae615 100644
--- a/src/api/internal/BamMultiReader_p.cpp
+++ b/src/api/internal/BamMultiReader_p.cpp
@@ -2,7 +2,7 @@
 // BamMultiReader_p.cpp (c) 2010 Derek Barnett, Erik Garrison
 // Marth Lab, Department of Biology, Boston College
 // ---------------------------------------------------------------------------
-// Last modified: 10 October 2011 (DB)
+// Last modified: 14 October 2011 (DB)
 // ---------------------------------------------------------------------------
 // Functionality for simultaneously reading multiple BAM files
 // *************************************************************************
@@ -568,6 +568,10 @@ bool BamMultiReaderPrivate::PopNextCachedAlignment(BamAlignment& al, const bool
 // returns BAM file pointers to beginning of alignment data & resets alignment cache
 bool BamMultiReaderPrivate::Rewind(void) {
 
+    // skip if no readers open
+    if ( m_readers.empty() )
+        return true;
+
     // attempt to rewind files
     if ( !RewindReaders() ) {
         const string currentError = m_errorString;