Skip to content

GROOVY-6360: GString performance is slow with String's trim method#1486

Merged
daniellansun merged 1 commit intoapache:masterfrom
paulk-asert:groovy6360
Feb 14, 2021
Merged

GROOVY-6360: GString performance is slow with String's trim method#1486
daniellansun merged 1 commit intoapache:masterfrom
paulk-asert:groovy6360

Conversation

@paulk-asert
Copy link
Copy Markdown
Contributor

No description provided.

@paulk-asert
Copy link
Copy Markdown
Contributor Author

This avoids using reflection for two common String methods and, for Groovy 4, will piggy back on the already implemented improved caching.

@daniellansun
Copy link
Copy Markdown
Contributor

How about adding all methods of String to GStringImpl, e.g. substring, indexOf, lastIndexOf, etc.

@daniellansun daniellansun merged commit 381aed3 into apache:master Feb 14, 2021
@daniellansun
Copy link
Copy Markdown
Contributor

Merged. Thanks!

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