[
https://issues.apache.org/jira/browse/TIKA-1710?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14704631#comment-14704631
]
Nick Burch commented on TIKA-1710:
----------------------------------
Thanks for this, applied in smaller chunks in r1696741 through 1696751.
Two questions:
* Your patch removed guava, but I couldn't see an explanation of why? I didn't
commit that part, could you explain why you think it can / should be removed?
* Our TaggedInputStream get syntax looks cleaner than what you've had to do
instead. Do you think it's worth adding a helper for that / asking Commons IO
to implement that pattern for us?
> Replace usages of classes in org.apache.tika.io with current alternatives
> -------------------------------------------------------------------------
>
> Key: TIKA-1710
> URL: https://issues.apache.org/jira/browse/TIKA-1710
> Project: Tika
> Issue Type: Improvement
> Components: batch, cli, core, example, gui, parser, server,
> translation
> Reporter: Yaniv Kunda
> Priority: Minor
> Fix For: 1.11
>
> Attachments: TIKA-1710.patch
>
>
> Many of the classes in org.apache.tika.io were inlined from commons-io in
> TIKA-249, but these days most components use commons-io anyway, so in order
> to clean the dependencies on org.apache.tika.io in preparation of adding
> commons-io to tika-core, the following can be done:
> - Replace usages of classes in org.apache.tika.io within non-core components
> with the corresponding classes in commons-io
> - Replace usages of org.apache.tika.io.IOUtils.UTF_8 with
> java.nio.charset.StandardCharsets.UTF_8 (in all components, including
> tika-core)
> - Replace other uses of String encoding names of standard charsets with their
> corresponding Charsets instances from StandardCharsets (this is logically
> related to IOUtils as these constants should have been there as UTF_8 was
> before Java 7)
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
