findResourceWriters often creates HashSets and ArrayLists while not n…

…ecessary
quarkusio · Jun 29, 2023 · 5f8c7f1 · 5f8c7f1
1 parent b61ceea
commit 5f8c7f1
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/...ive/common/runtime/src/main/java/org/jboss/resteasy/reactive/common/core/Serialisers.java b/...ive/common/runtime/src/main/java/org/jboss/resteasy/reactive/common/core/Serialisers.java
@@ -164,7 +164,7 @@ protected List<ResourceWriter> findResourceWriters(QuarkusMultivaluedMap<Class<?
         List<ResourceWriter> ret = new ArrayList<>();
         Deque<Class<?>> toProcess = new LinkedList<>();
         do {
-            if (currentClass == Object.class) {
+            if (currentClass == Object.class && !toProcess.isEmpty()) {
                 //spec extension, look for interfaces as well
                 //we match interfaces before Object
                 Set<Class<?>> seen = new HashSet<>(toProcess);
@@ -182,13 +182,16 @@ protected List<ResourceWriter> findResourceWriters(QuarkusMultivaluedMap<Class<?
             }
             List<ResourceWriter> goodTypeWriters = writers.get(currentClass);
             writerLookup(runtimeType, produces, desired, ret, goodTypeWriters);
-            toProcess.addAll(Arrays.asList(currentClass.getInterfaces()));
+            var prevClass = currentClass;
             // if we're an interface, pretend our superclass is Object to get us through the same logic as a class
             if (currentClass.isInterface()) {
                 currentClass = Object.class;
             } else {
                 currentClass = currentClass.getSuperclass();
             }
+            if (currentClass != null) {
+                toProcess.addAll(List.of(prevClass.getInterfaces()));
+            }
         } while (currentClass != null);
 
         return ret;