I replaced sun.io converters by an adaptor to sun.nio.cs coders
Ulf Zibis
Ulf.Zibis at gmx.de
Mon Sep 15 12:26:00 UTC 2008
Hi Sherman,
after finishing my work on tests, compairing my reengeneerings against
the legacy sun.nio.cs encoders, and rethinking malformed sequence length
, I come back to sun.io package...
Am 05.09.2008 00:59, Xueming Shen schrieb:
>
> Ulf, you are really productive:-) thanks for working so hard on the
> "adapter" idea!
Oh, thank's for your flowers. it was nearly 2 weeks of work.
Also thanks about reviewing my code. The bugs, you have found, are serious.
>
> Took a very quick scan on the ByteToChar adapter, here are some
> comments for your considering
>
> (1)in convert(), the decoder.decode(src, dst, true) is used instead of
> the decode(src, dst, false), which probably
> is against the specified "a buffer by buffer conversion" use scenario,
> consider the possibility that we have some
> "incomplete" bytes in the "input" stream, which might be "completed"
> by sequential "input" in a second invoking
> of convert().
I see only 1 solution: ByteToCharConverter#flush() should first invoke
decoder.decode(src, dst, true) before decoder.flush(), because there is
no compatible endOfInput-logic in sun.io package.
In this context, I must admit, that I don't understand the necessity of
this endOfInput-logic. It forces an additional invocation of
encodeLoop() even if there is nothing to do in most cases. Why can't
decoder.flush() do this job as in sun.io package???
>
> (2)flush(), the spec says you need to "reset() before throw the
> MalformedInputException,so the charOff need
> to be zero.
OK.
>
> (3)reset() does not set the badInputlength to 0.
1.) The legacy ByteToCharXXX don't do this, why should I do?
2.) If flush() has to reset before throwing MalformedInputException,
then badInputlength maybe would be invalid.
Regards,
-Ulf
More information about the core-libs-dev
mailing list