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