Fixing a bug in methcounts checking if output is working

andrewdavidsmith · andrewdavidsmith · commit 8b1295d51e28 · 2023-08-09T15:16:30.000-07:00
diff --git a/src/analysis/methcounts.cpp b/src/analysis/methcounts.cpp
@@ -267,7 +267,7 @@ write_output(const bamxx::bam_header &hdr, bamxx::bgzf_file &out,
           << tag_values[tag_with_mut(the_tag, mut)] << '\t'
           << (n_reads > 0 ? unconverted/n_reads : 0.0) << '\t'
           << n_reads << '\n';
-      if (out.write(buf.c_str(), buf.tellp()))
+      if (!out.write(buf.c_str(), buf.tellp()))
         throw dnmt_error("error writing output");
     }
   }

Original file line number	Diff line number	Diff line change
`@@ -267,7 +267,7 @@ write_output(const bamxx::bam_header &hdr, bamxx::bgzf_file &out,`
`267`	`267`	`<< tag_values[tag_with_mut(the_tag, mut)] << '\t'`
`268`	`268`	`<< (n_reads > 0 ? unconverted/n_reads : 0.0) << '\t'`
`269`	`269`	`<< n_reads << '\n';`
`270`		`- if (out.write(buf.c_str(), buf.tellp()))`
	`270`	`+ if (!out.write(buf.c_str(), buf.tellp()))`
`271`	`271`	`throw dnmt_error("error writing output");`
`272`	`272`	`}`
`273`	`273`	`}`