You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by gg...@apache.org on 2026/08/28 01:17:13 UTC

(commons-xml) branch main updated: Remove constructor that's only used from a test

This is an automated email from the ASF dual-hosted git repository.

garydgregory pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/commons-xml.git


The following commit(s) were added to refs/heads/main by this push:
     new 220c139  Remove constructor that's only used from a test
220c139 is described below

commit 220c1390fa105380f2093e2b98b28d7b48d9b82e
Author: Gary Gregory <ga...@gmail.com>
AuthorDate: Thu Aug 27 21:17:07 2026 -0400

    Remove constructor that's only used from a test
    
    Replace the call site with the equivalent call.
---
 .../java/org/apache/commons/xml/FallbackIgnoreURIResolver.java   | 9 ---------
 src/test/java/org/apache/commons/xml/DenyUnresolvedTest.java     | 2 +-
 2 files changed, 1 insertion(+), 10 deletions(-)

diff --git a/src/main/java/org/apache/commons/xml/FallbackIgnoreURIResolver.java b/src/main/java/org/apache/commons/xml/FallbackIgnoreURIResolver.java
index 7e34f46..a6fc7d5 100644
--- a/src/main/java/org/apache/commons/xml/FallbackIgnoreURIResolver.java
+++ b/src/main/java/org/apache/commons/xml/FallbackIgnoreURIResolver.java
@@ -85,15 +85,6 @@ private static Document newEmptyDocument() {
      */
     private final Supplier<Source> emptySource;
 
-    /**
-     * Constructs a new resolver.
-     *
-     * @param delegate optional caller-supplied resolver to consult first; may be {@code null}.
-     */
-    FallbackIgnoreURIResolver(final URIResolver delegate) {
-        this(delegate, null);
-    }
-
     /**
      * Constructs a new resolver.
      *
diff --git a/src/test/java/org/apache/commons/xml/DenyUnresolvedTest.java b/src/test/java/org/apache/commons/xml/DenyUnresolvedTest.java
index 1e91add..54b25b7 100644
--- a/src/test/java/org/apache/commons/xml/DenyUnresolvedTest.java
+++ b/src/test/java/org/apache/commons/xml/DenyUnresolvedTest.java
@@ -56,7 +56,7 @@ void floorsThrowOnUnresolved() {
                 "XMLResolver floor should throw on an unresolved entity");
         assertThrows(LSException.class, () -> new FallbackIgnoreLSResourceResolver(null).resolveResource(null, null, null, SYSTEM_ID, null),
                 "LSResourceResolver floor should throw on an unresolved resource");
-        assertThrows(TransformerException.class, () -> new FallbackIgnoreURIResolver(null).resolve(SYSTEM_ID, null),
+        assertThrows(TransformerException.class, () -> new FallbackIgnoreURIResolver(null, null).resolve(SYSTEM_ID, null),
                 "URIResolver floor should throw on an unresolved URI");
     }
 }