Skip to content

Comments

Optimizations#128

Open
stokito wants to merge 10 commits intocheptsov:masterfrom
stokito:optimizations
Open

Optimizations#128
stokito wants to merge 10 commits intocheptsov:masterfrom
stokito:optimizations

Conversation

@stokito
Copy link

@stokito stokito commented Nov 12, 2018

Small micro optimizations mostly to avoid memory allocation:

  1. create StringBuilder with initial capacity
  2. use StringBuilder.append(char) instead of .append(String) or .append(Object)
    etc

@stickler-ci
Copy link

I couldn't find a .stickler.yml file in this repository. I can make one for you, or you can create one by following the documentation.

1. Precompile regexp GENERICS_PATTERN.
2 Use Collections.addAll(list, array) without creation of ArrayList by Arrays.asList()
@stickler-ci
Copy link

I couldn't find a .stickler.yml file in this repository. I can make one for you, or you can create one by following the documentation.

@stickler-ci
Copy link

I couldn't find a .stickler.yml file in this repository. I can make one for you, or you can create one by following the documentation.

@AntoniRokitnicki
Copy link

@stokito I have merged your PR to my fork

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