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-secure-xml.git
The following commit(s) were added to refs/heads/main by this push:
new 92724fb Javadoc
92724fb is described below
commit 92724fb1065180dcb8852cf678eb04556d2bdf4b
Author: Gary Gregory <[email protected]>
AuthorDate: Wed Sep 2 10:36:47 2026 -0400
Javadoc
Fix text.
Fix inline comment.
Fix @throws {@code} reference.
---
.../java/org/apache/commons/xml/secure/FallbackIgnoreXMLResolver.java | 2 +-
.../java/org/apache/commons/xml/secure/SecureSAXParserFactory.java | 2 +-
.../java/org/apache/commons/xml/secure/SecureTransformerFactory.java | 2 +-
src/main/java/org/apache/commons/xml/secure/SecureXMLFilter.java | 2 +-
.../java/org/apache/commons/xml/secure/SecureXMLInputFactory.java | 4 ++--
5 files changed, 6 insertions(+), 6 deletions(-)
diff --git
a/src/main/java/org/apache/commons/xml/secure/FallbackIgnoreXMLResolver.java
b/src/main/java/org/apache/commons/xml/secure/FallbackIgnoreXMLResolver.java
index 93a5c4c..0c92274 100644
--- a/src/main/java/org/apache/commons/xml/secure/FallbackIgnoreXMLResolver.java
+++ b/src/main/java/org/apache/commons/xml/secure/FallbackIgnoreXMLResolver.java
@@ -75,7 +75,7 @@ public Object resolveEntity(final String publicID, final
String systemID, final
/**
* Sets the delegate to consult first, replacing any previous delegate.
* <p>
- * A null value removes the delegate and leaves a pure ignore-all floor.
+ * A {@code null} value removes the delegate and leaves a pure ignore-all
floor.
* </p>
*
* @param delegate The delegate to consult first, replacing any previous
delegate.
diff --git
a/src/main/java/org/apache/commons/xml/secure/SecureSAXParserFactory.java
b/src/main/java/org/apache/commons/xml/secure/SecureSAXParserFactory.java
index fd3320b..5759080 100644
--- a/src/main/java/org/apache/commons/xml/secure/SecureSAXParserFactory.java
+++ b/src/main/java/org/apache/commons/xml/secure/SecureSAXParserFactory.java
@@ -319,7 +319,7 @@ static XMLReader newXMLReader(final boolean
overrideDefaultParser) {
/**
* Capability-driven securing for any {@link SAXParserFactory} on the
classpath.
*
- * <p>Rather than branching on the implementation class, this method
probes what the parser supports and adapts. Because
+ * <p>Rather than branching on the implementation class, this method
probes what the factory supports and adapts. Because
* {@link SAXParserFactory} exposes only a feature API and no property
API, the per-parse configuration runs on each {@link XMLReader} the factory
produces,
* funneled through the nested wrapper into {@link #secure(XMLReader)}:</p>
* <ul>
diff --git
a/src/main/java/org/apache/commons/xml/secure/SecureTransformerFactory.java
b/src/main/java/org/apache/commons/xml/secure/SecureTransformerFactory.java
index eac32f6..7e6db46 100644
--- a/src/main/java/org/apache/commons/xml/secure/SecureTransformerFactory.java
+++ b/src/main/java/org/apache/commons/xml/secure/SecureTransformerFactory.java
@@ -287,7 +287,7 @@ public Transformer newTransformer() throws
TransformerConfigurationException {
*/
@Override
public Transformer newTransformer(final Source source) throws
TransformerConfigurationException {
- // newTemplatesHandler() should never return null for a
specification-compliant factory.
+ // newTransformer() should never return null for a
specification-compliant factory.
final Transformer transformer =
delegate.newTransformer(SecureSAXParserFactory.secure(source,
overrideDefaultParser()));
return transformer == null ? null : new
SecureTransformer(transformer, getURIResolver(), emptySource,
overrideDefaultParser());
}
diff --git a/src/main/java/org/apache/commons/xml/secure/SecureXMLFilter.java
b/src/main/java/org/apache/commons/xml/secure/SecureXMLFilter.java
index 8741872..204e9f8 100644
--- a/src/main/java/org/apache/commons/xml/secure/SecureXMLFilter.java
+++ b/src/main/java/org/apache/commons/xml/secure/SecureXMLFilter.java
@@ -74,7 +74,7 @@ private static SAXParseException toSAXParseException(final
TransformerException
* Constructs a new instance.
*
* @param templates the templates to wrap; must not be {@code null}.
- * @throws NullPointerException if {@code delegate} is {@code null}.
+ * @throws NullPointerException Thrown if {@code templates} is {@code
null}.
*/
SecureXMLFilter(final SecureTemplates templates) {
this.templates = Objects.requireNonNull(templates, "templates");
diff --git
a/src/main/java/org/apache/commons/xml/secure/SecureXMLInputFactory.java
b/src/main/java/org/apache/commons/xml/secure/SecureXMLInputFactory.java
index e4fc358..9b6873e 100644
--- a/src/main/java/org/apache/commons/xml/secure/SecureXMLInputFactory.java
+++ b/src/main/java/org/apache/commons/xml/secure/SecureXMLInputFactory.java
@@ -61,8 +61,8 @@ public final class SecureXMLInputFactory {
* <p>
* Every resolver-valued entry point ({@link
#setXMLResolver(XMLResolver)}, {@code setProperty(XMLInputFactory.RESOLVER,
...)} and the Woodstox
* {@code com.ctc.wstx.*Resolver} keys) is routed uniformly: a caller who
supplies their own {@link FallbackIgnoreXMLResolver} takes control and it is
- * passed straight to the delegate; otherwise the caller's resolver is
wrapped in a new floor installed on that hook, so the caller's opt-in cannot be
- * removed by dropping the resolver. This matters because Woodstox does
not chain resolvers: when a resolver returns {@code null},
+ * passed straight to the delegate; otherwise the caller's resolver is
wrapped in a new floor installed on that hook, so the floor cannot be removed
by dropping
+ * the resolver. This matters because Woodstox does not chain resolvers:
when a resolver returns {@code null},
* {@code DefaultInputResolver} falls through to fetching the systemId URL
itself, so a caller-set resolver that returns {@code null} must still land
behind
* the floor. {@link #getXMLResolver()} and {@code getProperty} report the
caller's resolver unwrapped.
* </p>
