spectre.console icon indicating copy to clipboard operation
spectre.console copied to clipboard

Reduce memory usage for rune width cache.

Open Pannoniae opened this issue 9 months ago • 2 comments

  • [X] I have read the Contribution Guidelines
  • [ ] I have commented on the issue above and discussed the intended changes
  • [ ] A maintainer has signed off on the changes and the issue was assigned to me
  • [X] All newly added code is adequately covered by tests
  • [X] All existing tests are still running without errors
  • [X] The documentation was modified to reflect the changes OR no documentation changes are required.

Changes

The rune width cache array consumes half a megabyte because it uses a Nullable to store values. This PR slashes the memory requirements to an eighth of what it was before with no loss of functionality.


Please upvote :+1: this pull request if you are interested in it.

Pannoniae avatar Feb 07 '25 12:02 Pannoniae