Warxim commented on code in PR #26:
URL: https://github.com/apache/commons-jxpath/pull/26#discussion_r999896659


##########
src/main/java/org/apache/commons/jxpath/ClassFunctions.java:
##########
@@ -91,6 +92,13 @@ public Function getFunction(
         final String namespace,
         final String name,
         Object[] parameters) {
+        JXPathFilter jxPathFilter = new JXPathFilter();

Review Comment:
   Not sure, but I am a little bit worried that this can be quite limiting to 
developers. We force them to use system properties in order to allow some 
classes in jxpath functions. Also, it is not possible to change the filter per 
JXContext



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to