Cleanup and harmonize output streams
Open
@nkoukpaizan is already working on this.
Since Sep 9, 2026.
- Dominant language
- C++
- Stars
- 27
- Forks
- 11
- Avg merge
- 3d 8h
- Merged PRs (30d)
- 23
Description
We currently have several instances for std::cout and std::cerr throughout the code. We now have a Logger that we can use instead.
- We can replace most
std::cerrinstances withLog::error()(orLog::warning()depending on the severity) - We can replace most
std::coutwithLog::misc(). - We currently have some commented out
std::coutfor debugging. Assess whether they are still needed and either remove or replace withLog::misc().
We may want to consider supporting both std::cout and std::cerr by default in the logger, but I personally don't think that's necessary.
Note: in earlier profiling, Log::misc() << "some output" << std::endl; would show up as a hotspot. I recommend "\n" moving forward unless flushing is really necessary.
CC @PhilipFackler @pelesh
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.