Github user HansBrende commented on the issue:

    https://github.com/apache/any23/pull/122
  
    @jgrzebyta thanks for your +1. Is the improved state of the javadoc to your 
liking?
    
    > I will add my unit in a separate ticket.
    
    I'm confused: are you referring to your existing PR #121 ? This PR is meant 
to be an alternative to that one. In my last commit, I have also added your 
[`ExtractorsFlowTest`](https://github.com/apache/any23/blob/f95f23865c0a7088e4ab1cbe507b8457fc90dda5/cli/src/test/java/org/apache/any23/cli/ExtractorsFlowTest.java)
 proof-of-concept to this PR to clarify that this PR provides at least as much 
functionality as #121 does. The only difference being: this PR uses the new 
`DelegatingWriterFactory` to accomplish the same behavior previously provided 
by `ModelExtractor` in #121. So if you approve this PR, my assumption would be 
that you prefer it over #121, and that #121 would be discarded.
    
    Any additional comments or concerns? Do I still have your +1?



---

Reply via email to