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

Factorize call to string.length() in StringExpression #5

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Factorize call to string.length() in StringExpression #5

wants to merge 1 commit into from

Conversation

benzonico
Copy link
Contributor

Method was called 3 times for each call to execute.
By factorizing it to when object is constructed the number of invocation is really reduced
and we can see a small gain in performance on small project.
@dbolkensteyn
Copy link
Contributor

LGTM @benzonico
@Godin Godin self-assigned this Oct 21, 2016
@benzonico
Factorize call to string.length()
d00838e 
Method was called 3 times for each call to execute.
By factorizing it to when object is constructed the number of invocation is really reduced
and we can see a small gain in performance on small project.
@Godin Godin force-pushed the master branch 3 times, most recently from e1979f9 to 9a641a5 Compare October 28, 2020 15:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants
@benzonico @dbolkensteyn @Godin