Skip to content

Memory leak risk: Unbounded log storage in ConsoleWindow #154

Description

@coderabbitai

Problem

The ConsoleWindow class stores all log messages in an unbounded std::vector<LogMessage> m_logs vector.

For long-running sessions or applications generating high volumes of logs, this could lead to excessive memory usage since logs accumulate indefinitely.

Possible Solutions

  • Implement a circular buffer with a maximum size
  • Add periodic purging of old log entries
  • Add a configurable maximum log count

References

Impact

Low priority - Not a critical issue for development but should be addressed before release builds to prevent potential memory problems in long-running sessions.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

No labels
No labels

Projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions