avoid volatile setter in API, we do not need it because of DefaultChooseByNameProvider sets IdFilter at FindSymbolParameters creation

This commit is contained in:
Maxim.Mossienko
2013-10-21 13:33:55 +02:00
parent 315db0823d
commit d3eca3e137
2 changed files with 1 additions and 8 deletions
@@ -25,7 +25,7 @@ public class FindSymbolParameters {
private final String myCompletePattern;
private final String myLocalPatternName;
private final GlobalSearchScope mySearchScope;
private volatile IdFilter myIdFilter;
private final IdFilter myIdFilter;
public FindSymbolParameters(@NotNull String pattern, @NotNull String name, @NotNull GlobalSearchScope scope, @Nullable IdFilter idFilter) {
myCompletePattern = pattern;
@@ -50,10 +50,6 @@ public class FindSymbolParameters {
return myIdFilter;
}
public void setIdFilter(IdFilter idFilter) {
myIdFilter = idFilter;
}
public static FindSymbolParameters wrap(@NotNull String pattern, @NotNull Project project, boolean searchInLibraries) {
return new FindSymbolParameters(
pattern,
@@ -202,9 +202,6 @@ public abstract class ContributorsBasedGotoByModel implements ChooseByNameModelE
long contributorStarted = System.currentTimeMillis();
if (contributor instanceof ChooseByNameContributorEx) {
if (parameters.getIdFilter() == null) {
parameters.setIdFilter(getIdFilter(searchInLibraries));
}
((ChooseByNameContributorEx)contributor).processElementsWithName(name, new Processor<NavigationItem>() {
@Override
public boolean process(NavigationItem item) {