Skip to content

Commit 135114a

Browse files
committed
Tweak names and calls per review
1 parent 497eb8c commit 135114a

File tree

1 file changed

+9
-12
lines changed

1 file changed

+9
-12
lines changed

opengrok-indexer/src/main/java/org/opengrok/indexer/search/SettingsHelper.java

Lines changed: 9 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -67,13 +67,10 @@ public SettingsHelper(IndexReader reader) {
6767
* Gets any mapped symlinks (after having called {@link #getSettings(String)}).
6868
* @return either a defined map or {@code null}
6969
*/
70-
public Map<String, IndexedSymlink> getSymlinks(String projectName) {
71-
if (mappedIndexedSymlinks == null) {
72-
throw new IllegalStateException("getSettings() not yet called");
73-
}
74-
75-
String k = projectName != null ? projectName : "";
76-
Map<String, IndexedSymlink> indexSymlinks = mappedIndexedSymlinks.get(k);
70+
public Map<String, IndexedSymlink> getSymlinks(String projectName) throws IOException {
71+
getSettings(projectName);
72+
String projectKey = projectName != null ? projectName : "";
73+
Map<String, IndexedSymlink> indexSymlinks = mappedIndexedSymlinks.get(projectKey);
7774
if (indexSymlinks != null) {
7875
return Collections.unmodifiableMap(indexSymlinks);
7976
}
@@ -114,8 +111,8 @@ public IndexAnalysisSettings3 getSettings(String projectName) throws IOException
114111
map(setts);
115112
}
116113

117-
String k = projectName != null ? projectName : "";
118-
return mappedAnalysisSettings.get(k);
114+
String projectKey = projectName != null ? projectName : "";
115+
return mappedAnalysisSettings.get(projectKey);
119116
}
120117

121118
private void map(IndexAnalysisSettings3[] setts) {
@@ -125,9 +122,9 @@ private void map(IndexAnalysisSettings3[] setts) {
125122

126123
for (IndexAnalysisSettings3 settings : setts) {
127124
String projectName = settings.getProjectName();
128-
String k = projectName != null ? projectName : "";
129-
settingsMap.put(k, settings);
130-
symlinksMap.put(k, mapSymlinks(settings));
125+
String projectKey = projectName != null ? projectName : "";
126+
settingsMap.put(projectKey, settings);
127+
symlinksMap.put(projectKey, mapSymlinks(settings));
131128
}
132129
mappedAnalysisSettings = settingsMap;
133130
mappedIndexedSymlinks = symlinksMap;

0 commit comments

Comments
 (0)