[
https://issues.apache.org/jira/browse/JENA-1403?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16215042#comment-16215042
]
ASF GitHub Bot commented on JENA-1403:
--------------------------------------
GitHub user afs opened a pull request:
https://github.com/apache/jena/pull/292
JENA-1403: Tidy up regex pattern handling.
This includes PR#291, and consolidates pattern exceptions into RegexJava.
Full handling with Xerces regexp for "replace" functionality is a lot of
work as Xerces has a separate, parallel set of classes; it is only the "x" flag
that is missing from Java. JDK regex is likely to be faster. JDK has been the
normal setting; Xerces is there for exact spec compliance in REGEX only.
We don't want more Xerces specific dependencies.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/afs/jena jena-1403_replace
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/jena/pull/292.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #292
----
----
> strReplace should throw a ExprEvalException on regex syntax error
> -----------------------------------------------------------------
>
> Key: JENA-1403
> URL: https://issues.apache.org/jira/browse/JENA-1403
> Project: Apache Jena
> Issue Type: Bug
> Reporter: Andy Seaborne
> Assignee: Andy Seaborne
>
> From a report on [user@jena
> (2017-10-23)|https://lists.apache.org/thread.html/210a70570b776ba0ce1974ff3fd7f8118e3c5bc8f98c6368d3231d64@%3Cusers.jena.apache.org%3E]
> This is the dynamic case of {{REPLACE}}, not the static compilation in
> {{E_strReplace}} although that should throw ExprException.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
