Fixed multiset union output upon input of two or more Starch archives

bedops · Jun 18, 2017 · 6ee04d3 · 6ee04d3
1 parent 42ed6f5
commit 6ee04d3
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/docs/content/revision-history.rst b/docs/content/revision-history.rst
@@ -51,6 +51,10 @@ This build of BEDOPS includes significant performance improvements. Tests were p
 * :ref:`starch <starch>`
 
   * Fixed potential segmentation fault with `--header` usage.
+
+* :ref: Starch C++ API
+
+  * Fixed output from `bedops -u` (multiset union) on two or more Starch archives, such that the remainder string was not being cleared correctly.
 
 =================
 Previous versions

diff --git a/interfaces/general-headers/data/starch/starchApi.hpp b/interfaces/general-headers/data/starch/starchApi.hpp
@@ -1787,6 +1787,8 @@ namespace starch
             else
                 std::sprintf(out, "%s\t%" PRId64 "\t%" PRId64, _currChr, _currStart, _currStop);
             line = out;
+            _currRemainder[0] = '\0';
+            _currRemainderLen = 0;
 
             if (archType == kGzip)
                 postBreakdownZValuesIdentical = (zOutBufIdx == zHave);
@@ -1896,14 +1898,14 @@ namespace starch
             needToInflateZChunk = false;
         }
 
-        /* read through zOutBuf for newlines */                    
+        /* read through zOutBuf for newlines */
         for (; zOutBufIdx < zHave; zBufIdx++) {
             if (zOutBuf[zOutBufIdx++] == '\n') {
                 zLineBuf[zBufIdx] = '\0';
                 zBufIdx = 0;
 #ifdef DEBUG
                 std::fprintf(stderr, "\n--> full line [ %s ]\n", zLineBuf);
-#endif                
+#endif
                 return EXIT_SUCCESS;
             }
             else {