Github user HansBrende commented on the issue:

    https://github.com/apache/any23/pull/122
  
    While we're deprecating things anyway, there are a few notes of interest 
which we should mull over before any merges into master happen here:
    
    1. The method `WriterFactory.getMimeType()` appears to be redundant, as 
there also exists the `WriterFactory.getRdfFormat().getDefaultMimeType()`.
    
    2. Also note the presence of `FileFormat`, the superclass of `RDFFormat`, 
which we could possibly use to make the `FormatWriterFactory` interface more 
generic (possibly helpful for the `URIListWriterFactory`, and other writer 
factories which similarly do not print RDF triples as output).
    
    3. The method `WriterFactory.getRdfFormat()` is never actually used 
anywhere in the Any23 project.
    
    4. `JSONWriterFactory` and `URIListWriterFactory` both throw 
`RuntimeException` in the `getRdfFormat()` method (the former questionably so, 
since there exists the `RDFFormat.RDFJSON` file format).


---

Reply via email to