Github user HansBrende commented on the issue:

    https://github.com/apache/any23/pull/122
  
    > @HansBrende.  Sorry, I did not follow your latest commits. My proof of
    concept was done for different approach so I thought it would be useless in
    your one. I had thought I would need to write the new one. But if you
    reused so I am happy of that. You have still my +1.
    
    @jgrzebyta glad to hear I still have your +1! Yes, although I used your 
original unit tests, I did have to modify the way they were implemented. Here 
are the implementation changes I made:
    1. `ExtractorsFlowTest` diff: https://www.diffchecker.com/pPGAQxE6
    2. `PeopleExtractorFactory` diff: https://www.diffchecker.com/Mn7XTZOB
    3. `PeopleExtractor` diff: https://www.diffchecker.com/x4du9RqE



---

Reply via email to