From 87d77eb2a0c9cfcea72d1c10149d02f2e88b82f6 Mon Sep 17 00:00:00 2001
From: Jonathan Gibbons <jjg@openjdk.org>
Date: Fri, 2 Oct 2020 16:12:26 +0000
Subject: [PATCH] 8253879: Simplify redundant code in IndexBuilder

Reviewed-by: ksrini
---
 .../doclets/toolkit/util/IndexBuilder.java         | 14 +++++++-------
 1 file changed, 7 insertions(+), 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
index 2f6e852f2a503..27fd500a8083e 100644
--- 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))));
         }
     }