[GitHub] commons-rdf issue #27: COMMONSRDF-47 RDFSyntax as an interface

2017-02-16 Thread stain
Github user stain commented on the issue: https://github.com/apache/commons-rdf/pull/27 I've merged this. Thanks for the help, @ansell! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this fe

[GitHub] commons-rdf issue #27: COMMONSRDF-47 RDFSyntax as an interface

2017-02-10 Thread stain
Github user stain commented on the issue: https://github.com/apache/commons-rdf/pull/27 OK, so I added them as `Set` - it might be an ordered set (e.g. `LinkedHashSet`). I merged `RDFA_XHTML` and `RDFA_HTML` and made them return such a set to include both `text/html` and `a

[GitHub] commons-rdf issue #27: COMMONSRDF-47 RDFSyntax as an interface

2017-02-08 Thread ansell
Github user ansell commented on the issue: https://github.com/apache/commons-rdf/pull/27 Service discovery for other instances of RDFSyntax/Parsers can be done separately. The main objection was for users of the RDFSyntax interface to be able to offer multiple content types

[GitHub] commons-rdf issue #27: COMMONSRDF-47 RDFSyntax as an interface

2017-02-08 Thread stain
Github user stain commented on the issue: https://github.com/apache/commons-rdf/pull/27 Let's discuss the mutability etc. of the `RDFParser` factory separate on dev@commons - that's a general thing which anyway should not affect this pull request on `RDFSyntax`. This pull req