Merge pull request #144 from smithlabcode/sam_realloc

andrewdavidsmith · web-flow · commit 96dc147c3da5 · 2023-09-21T16:57:16.000-07:00
fixed the order of source and dest in sam_realloc as per the original…
diff --git a/src/common/bam_record_utils.cpp b/src/common/bam_record_utils.cpp
@@ -115,10 +115,10 @@ sam_realloc_bam_data(bam1_t *b, size_t desired) {
     new_data = static_cast<uint8_t *>(malloc(new_m_data));
     if (new_data != nullptr) {
       if (b->l_data > 0)
-        std::copy_n(new_data,
+        std::copy_n(b->data,
                     (static_cast<uint32_t>(b->l_data) < b->m_data) ? b->l_data
                                                                    : b->m_data,
-                    b->data);
+                    new_data);
       bam_set_mempolicy(b, bam_get_mempolicy(b) & (~BAM_USER_OWNS_DATA));
     }
   }

Original file line number	Diff line number	Diff line change
`@@ -115,10 +115,10 @@ sam_realloc_bam_data(bam1_t *b, size_t desired) {`
`115`	`115`	`new_data = static_cast<uint8_t *>(malloc(new_m_data));`
`116`	`116`	`if (new_data != nullptr) {`
`117`	`117`	`if (b->l_data > 0)`
`118`		`- std::copy_n(new_data,`
	`118`	`+ std::copy_n(b->data,`
`119`	`119`	`(static_cast<uint32_t>(b->l_data) < b->m_data) ? b->l_data`
`120`	`120`	`: b->m_data,`
`121`		`- b->data);`
	`121`	`+ new_data);`
`122`	`122`	`bam_set_mempolicy(b, bam_get_mempolicy(b) & (~BAM_USER_OWNS_DATA));`
`123`	`123`	`}`
`124`	`124`	`}`