8253879: Simplify redundant code in IndexBuilder

Reviewed-by: ksrini
openjdk · Oct 2, 2020 · 87d77eb · 87d77eb · bridgekeeper · Oct 2, 2020
1 parent dffe9db
commit 87d77eb
Showing 1 changed file with 7 additions and 7 deletions.
diff --git a/src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/toolkit/util/IndexBuilder.java b/src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/toolkit/util/IndexBuilder.java
@@ -165,7 +165,7 @@ private void indexElements(Iterable<? extends Element> elements, TypeElement typ
                 String name = utils.getSimpleName(element);
                 Character ch = keyCharacter(name);
                 SortedSet<IndexItem> set = indexMap.computeIfAbsent(ch, c -> new TreeSet<>(comparator));
-                set.add(new IndexItem(element, typeElement, configuration.utils));
+                set.add(new IndexItem(element, typeElement, utils));
             }
         }
     }
@@ -181,7 +181,7 @@ private void indexTypeElements(Iterable<TypeElement> elements) {
                 String name = utils.getSimpleName(typeElement);
                 Character ch = keyCharacter(name);
                 SortedSet<IndexItem> set = indexMap.computeIfAbsent(ch, c -> new TreeSet<>(comparator));
-                set.add(new IndexItem(typeElement, configuration.utils));
+                set.add(new IndexItem(typeElement, utils));
             }
         }
     }
@@ -197,7 +197,7 @@ private void indexModules() {
         for (ModuleElement m : configuration.modules) {
             Character ch = keyCharacter(m.getQualifiedName().toString());
             SortedSet<IndexItem> set = indexMap.computeIfAbsent(ch, c -> new TreeSet<>(comparator));
-            set.add(new IndexItem(m, configuration.utils));
+            set.add(new IndexItem(m, utils));
         }
     }
 
@@ -210,7 +210,7 @@ private void indexPackage(PackageElement packageElement) {
         if (shouldIndex(packageElement)) {
             Character ch = keyCharacter(utils.getPackageName(packageElement));
             SortedSet<IndexItem> set = indexMap.computeIfAbsent(ch, c -> new TreeSet<>(comparator));
-            set.add(new IndexItem(packageElement, configuration.utils));
+            set.add(new IndexItem(packageElement, utils));
         }
     }
 
@@ -225,14 +225,14 @@ private boolean shouldIndex(Element element) {
         if (utils.isPackage(element)) {
             // Do not add to index map if -nodeprecated option is set and the
             // package is marked as deprecated.
-            return !(noDeprecated && configuration.utils.isDeprecated(element));
+            return !(noDeprecated && utils.isDeprecated(element));
         } else {
             // Do not add to index map if -nodeprecated option is set and if the
             // element is marked as deprecated or the containing package is marked as
             // deprecated.
             return !(noDeprecated &&
-                    (configuration.utils.isDeprecated(element) ||
-                    configuration.utils.isDeprecated(utils.containingPackage(element))));
+                    (utils.isDeprecated(element) ||
+                    utils.isDeprecated(utils.containingPackage(element))));
         }
     }