Skip to content

Conversation

@Jint-lzxy
Copy link
Collaborator

After some experiments, it seems rainbow_delims's local strategy will inevitably have a large performance hit while handling any excessively tall ASTs. This PR restricts this behavior where appropriate.

After some experiments, it seems `rainbow_delims`'s local strategy will
inevitably have a large performance hit while handling any excessively
tall ASTs. This PR restricts this behavior where appropriate.
charliie-dev

This comment was marked as duplicate.

Copy link
Collaborator

@charliie-dev charliie-dev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ayamir ayamir merged commit 02727a0 into main Aug 7, 2023
@ayamir ayamir deleted the perf/improve/rainbow_delims branch August 7, 2023 12:56
singlemancombat pushed a commit to singlemancombat/nvim-config that referenced this pull request Aug 7, 2023
After some experiments, it seems `rainbow_delims`'s local strategy will
inevitably have a large performance hit while handling any excessively
tall ASTs. This PR restricts this behavior where appropriate.
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.

4 participants