This is an automated email from the ASF dual-hosted git repository. andy pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/jena.git
commit c896d8976e32853a078585f7de335d33778823a1 Author: Andy Seaborne <[email protected]> AuthorDate: Sat Jan 11 10:19:15 2025 +0000 GH-2800: Remove unused checking cache --- .../main/java/org/apache/jena/riot/system/Checker.java | 9 --------- .../org/apache/jena/riot/system/ParserProfileStd.java | 17 ++++------------- 2 files changed, 4 insertions(+), 22 deletions(-) diff --git a/jena-arq/src/main/java/org/apache/jena/riot/system/Checker.java b/jena-arq/src/main/java/org/apache/jena/riot/system/Checker.java index caed482e23..125adda74a 100644 --- a/jena-arq/src/main/java/org/apache/jena/riot/system/Checker.java +++ b/jena-arq/src/main/java/org/apache/jena/riot/system/Checker.java @@ -130,15 +130,6 @@ public class Checker { } catch (org.apache.jena.iri.IRIException0 | org.apache.jena.irix.IRIException ex) {} } -// /** -// * Process violations on an IRI Calls the {@link ErrorHandler} on all errors and -// * warnings (as warnings). -// */ -// @Deprecated(forRemoval = true) -// public static void iriViolations(IRI iri) { -// iriViolations(iri, nullErrorHandler, false, true, -1L, -1L); -// } -// final static private Pattern langPattern = Pattern.compile("[a-zA-Z]{1,8}(-[a-zA-Z0-9]{1,8})*"); public static boolean checkLiteral(Node node) { diff --git a/jena-arq/src/main/java/org/apache/jena/riot/system/ParserProfileStd.java b/jena-arq/src/main/java/org/apache/jena/riot/system/ParserProfileStd.java index 71a19b677e..87187663b7 100644 --- a/jena-arq/src/main/java/org/apache/jena/riot/system/ParserProfileStd.java +++ b/jena-arq/src/main/java/org/apache/jena/riot/system/ParserProfileStd.java @@ -20,13 +20,13 @@ package org.apache.jena.riot.system; import java.util.Objects; -import org.apache.jena.atlas.lib.Cache; -import org.apache.jena.atlas.lib.CacheFactory; import org.apache.jena.datatypes.RDFDatatype; import org.apache.jena.datatypes.xsd.XSDDatatype; import org.apache.jena.graph.*; -import org.apache.jena.iri.IRI; -import org.apache.jena.irix.*; +import org.apache.jena.irix.IRIException; +import org.apache.jena.irix.IRIx; +import org.apache.jena.irix.IRIxResolver; +import org.apache.jena.irix.RelativeIRIException; import org.apache.jena.query.ARQ; import org.apache.jena.riot.RiotException; import org.apache.jena.riot.tokens.Token; @@ -48,7 +48,6 @@ public class ParserProfileStd implements ParserProfile { private final boolean strictMode; private final boolean checking; private static int DftCacheSize = 500; - private final Cache<String, IRI> iriCache; private boolean allowNodeExtentions; @@ -61,7 +60,6 @@ public class ParserProfileStd implements ParserProfile { this.prefixMap = prefixMap; this.context = context; this.checking = checking; - this.iriCache = checking ? CacheFactory.createCache(DftCacheSize) : null; this.strictMode = strictMode; this.allowNodeExtentions = true; // (context.isTrue(RIOT.ALLOW_NODE_EXT)) ; } @@ -132,13 +130,6 @@ public class ParserProfileStd implements ParserProfile { Checker.iriViolationMessage(uriStr, isError, message, line, col, errorHandler); }); - // Jena up to 5.2.0 behaviour: Always jena-iri messages -// IRI iri; -// if ( irix instanceof IRIProviderJenaIRI.IRIxJena ) -// iri = (IRI)irix.getImpl(); -// else -// iri = iriCache.get(uriStr, x -> SetupJenaIRI.iriCheckerFactory().create(x)); -// Checker.iriViolations(iri, errorHandler, false, true, line, col); } /**
