Github user osma commented on the pull request:

    https://github.com/apache/jena/pull/97#issuecomment-154092054
  
    Thanks @ajs6f for your detailed notes! I fixed the things you pointed out 
but I think especially the Guava Sets.newHashSet pattern could be applied in 
many other places in the jena-text code as well. My test case code was just 
adapted from similar other jena-text tests and they all use the same pattern. 
Fixing those as well isn't really in the scope for this PR but could be done 
separately.


---
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 feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to