| Issue 2: | InternalScanner uses inefficient String.replace() method | |
| 1 person starred this issue and may be notified of changes. | Back to list |
Profiling Confluence's start-up showed a "hot spot" was
InternalScanner.loadImplementationsInJar(), particularly the line that does
this:
> pkg = pkg.replace("/", ".");
You can replace this with:
> pkg = pkg.replace('/', '.');
This is functionally equivalent, and faster because it iterates through the
String rather than doing a regular expression replacement.
Nov 25, 2008
(No comment was entered for this change.)
Labels:
Performance
|
Labels: -Type-Defect Type-Enhancement