bugfix

marekkokot · marekkokot · commit 003e5ca42490 · 2023-12-08T16:10:34.000+01:00
diff --git a/kmc_core/fastq_reader.cpp b/kmc_core/fastq_reader.cpp
@@ -1163,8 +1163,9 @@ uint64 CFastqReaderDataSrc::read(uchar* buff, uint64 size, bool& last_in_file, b
 			{
 				pmm_binary_file_reader->free(in_data);
 				in_data = nullptr;
-				pop_pack(in_data, in_data_size, file_part, compression_type, last_in_file);
-				if (file_part == FilePart::End)
+				//may be false even if file_part != FilePart::End in stats mode
+				auto pop_res = pop_pack(in_data, in_data_size, file_part, compression_type, last_in_file);
+				if (!pop_res || file_part == FilePart::End)
 				{
 					in_progress = false;
 					last_in_file = true;

Original file line number	Diff line number	Diff line change
`@@ -1163,8 +1163,9 @@ uint64 CFastqReaderDataSrc::read(uchar* buff, uint64 size, bool& last_in_file, b`
`1163`	`1163`	`{`
`1164`	`1164`	`pmm_binary_file_reader->free(in_data);`
`1165`	`1165`	`in_data = nullptr;`
`1166`		`- pop_pack(in_data, in_data_size, file_part, compression_type, last_in_file);`
`1167`		`- if (file_part == FilePart::End)`
	`1166`	`+ //may be false even if file_part != FilePart::End in stats mode`
	`1167`	`+ auto pop_res = pop_pack(in_data, in_data_size, file_part, compression_type, last_in_file);`
	`1168`	`+ if (!pop_res \|\| file_part == FilePart::End)`
`1168`	`1169`	`{`
`1169`	`1170`	`in_progress = false;`
`1170`	`1171`	`last_in_file = true;`