pezmaster31 / bamtools

C++ API & command-line toolkit for working with BAM data
MIT License
418 stars 153 forks source link

Replace std::endl by "\n" to avoid force flush #202

Closed bioinfornatics closed 4 years ago

bioinfornatics commented 4 years ago

Dear,

we encounter on our cluster a huge loss of performance due to a flush on each line by the use of std::endl

This merge request help to reduce the impact of writing an output stream

SoapZA commented 4 years ago

@bioinfornatics your change is good, just switch to using the char literal '\n' instead of the string literal "\n" and I'll merge it.

bioinfornatics commented 4 years ago

Thanks @SoapZA