[GitHub] jena issue #449: JENA-1578

2018-08-01 Thread afs
Github user afs commented on the issue: https://github.com/apache/jena/pull/449 There is no need to use the un-parenthesed form - it is a syntactic short form and the full form can always be used. This may make the code generation easier and have less at risk of misalignment. ```

[GitHub] jena issue #449: JENA-1578

2018-07-30 Thread GregAlbiston
Github user GregAlbiston commented on the issue: https://github.com/apache/jena/pull/449 I've made updates to try and address the comments made so far. - Now using _FmtUtils.stringForNode_ for conversion of RDFNode to replacement string. - Local method

[GitHub] jena issue #449: JENA-1578

2018-07-28 Thread afs
Github user afs commented on the issue: https://github.com/apache/jena/pull/449 @ajs6f Will take a look. A particular thing to be careful of with ParameterizedSparqlString in general is injection attacks. cc @rvesse ---