Hi William!

Thank you for pointing this out. I have fixed it. Can you check and close
the issue?

How can I improve the documentation (or the code itself) so that the new
code is easier to understand? Any advice?

Aliaksandr

On Thu, Jan 19, 2012 at 2:57 PM, [email protected] <
[email protected]> wrote:

> Aliaksandr,
>
> Could you also check the issue
> https://issues.apache.org/jira/browse/OPENNLP-418 ? I tried to fix it
> by myself but I could not find an appropriate solution. I still have
> to better understand the new code.
>
> Thank you,
> William
>
>
> On Wed, Jan 18, 2012 at 7:34 AM, Jörn Kottmann <[email protected]> wrote:
> > On 1/18/12 1:05 AM, James Kosin wrote:
> >>
> >> I put the TODO there; because I couldn't determine if it was a better
> >> place.  The only big downside to using the Stream is we have no control
> >> over the encoding.  So, I was thinking more that this method of loading
> >> the item would be deprecated anyway.  In favor of the other method.
> >
> >
> > Decoding with the correct encoding is the responsibility of the
> > Reader. There is no need to pass an encoding along in this method, right?
> >
> > Jörn
>

Reply via email to