Skip to content

Document CompactString rationale in EnhancedCell module#222

Merged
ryanoneill merged 1 commit intomainfrom
docs/compact-str-rationale
Mar 9, 2026
Merged

Document CompactString rationale in EnhancedCell module#222
ryanoneill merged 1 commit intomainfrom
docs/compact-str-rationale

Conversation

@ryanoneill
Copy link
Owner

Summary

  • Adds module-level documentation explaining why CompactString is used for cell symbols
  • At 80×24, a terminal has 1,920 cells per frame; CompactString stores up to 24 bytes inline, eliminating thousands of small heap allocations per render cycle

Test plan

  • Documentation-only change, no behavioral impact
  • cargo doc --all-features builds cleanly

🤖 Generated with Claude Code

Explains why CompactString is used for cell symbols: avoids heap
allocation for the 1,920+ cells per frame in a typical 80x24 terminal.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.09%. Comparing base (fa6edbb) to head (8d606dc).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #222   +/-   ##
=======================================
  Coverage   92.09%   92.09%           
=======================================
  Files         146      146           
  Lines       12798    12798           
=======================================
  Hits        11786    11786           
  Misses       1012     1012           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@ryanoneill ryanoneill merged commit 09c9b69 into main Mar 9, 2026
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants