Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Promote 'delim' to property #963

Open
colinodell opened this issue Feb 27, 2023 · 0 comments
Open

Promote 'delim' to property #963

colinodell opened this issue Feb 27, 2023 · 0 comments
Assignees
@colinodell
Labels
do not close Issue which won't close due to inactivity performance Something could be made faster or more efficient
Milestone
v3.0

Comments

@colinodell
Copy link
Member

Instead of using Text::$data to store whether it might be a delimiter, create a new $isDelimiter property. This should speed up delimiter processing by ~10%.

This will be a breaking change. Also, update the docs accordingly.
@colinodell colinodell added performance Something could be made faster or more efficient do not close Issue which won't close due to inactivity labels Feb 27, 2023
@colinodell colinodell added this to the v3.0 milestone Feb 27, 2023
@colinodell colinodell self-assigned this Feb 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@colinodell colinodell

Labels
do not close Issue which won't close due to inactivity performance Something could be made faster or more efficient
Projects
None yet
Milestone
v3.0
Development

No branches or pull requests
1 participant
@colinodell