merge: trim tailing tab in merged result. #38

shenwei356 · Sep 2, 2023 · fd8808f · fd8808f
1 parent a57cc96
commit fd8808f
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -2,9 +2,11 @@
 
 ### v0.9.4 - 2023-07-16
 
+- `kmcp merge`:
+ - trim tailing tab in merged result, download results are not affected. [#38](https:/shenwei356/kmcp/issues/38)
 - `kmcp profile`:
  - fix log of "xx% (yy/zz) reads matched", the numerator should be the number of matched reads instead of reads belonging to references.
- - early quick when no target detected.
+ - early quit when no target detected.
 
 ### v0.9.3 - 2023-07-16
 

diff --git a/kmcp/cmd/merge.go b/kmcp/cmd/merge.go
@@ -212,10 +212,11 @@ Example:
  for _, match = range buf0 {
  (*match.Data)[hitsFieldM1] = hits
 
- for _, item = range *match.Data {
+ for _, item = range (*match.Data)[:len(*match.Data)-1] {
  outfh.WriteString(item)
  outfh.WriteByte('\t')
  }
+ outfh.WriteString((*match.Data)[len(*match.Data)-1])
  outfh.WriteByte('\n')
 
  // poolStrings.Put(match.Data)
@@ -248,10 +249,11 @@ Example:
  for _, match = range buf0 {
  (*match.Data)[hitsFieldM1] = hits
 
- for _, item = range *match.Data {
+ for _, item = range (*match.Data)[:len(*match.Data)-1] {
  outfh.WriteString(item)
  outfh.WriteByte('\t')
  }
+ outfh.WriteString((*match.Data)[len(*match.Data)-1])
  outfh.WriteByte('\n')
 
  // poolStrings.Put(match.Data)