rve...@apache.org wrote: > Also fixed a bug with SysRiot.resetJenaReaders() related to setting the > RDF/JSON readers and writer class names to null (now uses empty strings > instead)
[...] > Modified: > incubator/jena/Jena2/ARQ/trunk/src/main/java/org/openjena/riot/SysRIOT.java > URL: > http://svn.apache.org/viewvc/incubator/jena/Jena2/ARQ/trunk/src/main/java/org/openjena/riot/SysRIOT.java?rev=1311528&r1=1311527&r2=1311528&view=diff > ============================================================================== > --- > incubator/jena/Jena2/ARQ/trunk/src/main/java/org/openjena/riot/SysRIOT.java > (original) > +++ > incubator/jena/Jena2/ARQ/trunk/src/main/java/org/openjena/riot/SysRIOT.java > Tue Apr 10 00:15:25 2012 > @@ -103,8 +103,8 @@ public class SysRIOT > RDFReaderFImpl.setBaseReaderClassName("Turtle", jenaTurtleReader) ; > RDFReaderFImpl.setBaseReaderClassName("TTL", jenaTurtleReader) ; > > - RDFReaderFImpl.setBaseReaderClassName("RDF/JSON", null) ; > - RDFWriterFImpl.setBaseWriterClassName("RDF/JSON", null) ; > + RDFReaderFImpl.setBaseReaderClassName("RDF/JSON", "") ; > + RDFWriterFImpl.setBaseWriterClassName("RDF/JSON", "") ; > } > > } Yep, I remember that. The thing is that there is no explicit way to unregister a reader or a writer. What's the advantage of using "" instead of null? (I don't like both... ;-)) Was null causing any problem? Perhaps, we should add an explicit unsetBase*ClassName(...) call. In future, it would be nice to give users the ability to easily register/unregister their RDF readers and writers making easy for third parties to add their own serialization formats and/or parsers from native formats. Paolo
