fixed potential bug with previous commit

The previous commit made assumptions about the ordering of subtags
within @RG header lines.  This commit only assumes that the read group
ID is specified by "ID", thus following spec.

diff --git a/BamMultiReader.cpp b/BamMultiReader.cpp
index 58d82a2d016b49de532d1ec85be39650a982d915..92911c90da8793a0a86bed63b9ba49ddce137009 100644 (file)
--- a/BamMultiReader.cpp
+++ b/BamMultiReader.cpp
@@ -237,12 +237,12 @@ const string BamMultiReader::GetHeaderText(void) const {
                 stringstream headerLineSs(headerLine);
                 string part, readGroupPart, readGroup;
                 while(std::getline(headerLineSs, part, '\t')) {
-                    if (part == "@RG") {
-                       std::getline(headerLineSs, readGroupPart, '\t');
-                       stringstream readGroupPartSs(readGroupPart);
-                       std::getline(readGroupPartSs, readGroup, ':');
-                       std::getline(readGroupPartSs, readGroup, ':');
-                       break;
+                    stringstream partSs(part);
+                    string subtag;
+                    std::getline(partSs, subtag, ':');
+                    if (subtag == "ID") {
+                        std::getline(partSs, readGroup, ':');
+                        break;
                     }
                 }
                 if (readGroups.find(readGroup) == readGroups.end()) { // prevents duplicate @RG entries