Mailing-List: contact dev-help@harmony.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@harmony.apache.org
Received-SPF: pass (nike.apache.org: domain of devyoudw@gmail.com designates
 74.125.83.49 as permitted sender)
DomainKey-Signature: a=rsa-sha1; c=nofws;
        d=gmail.com; s=gamma;
        h=mime-version:date:message-id:subject:from:to:content-type;
        b=PQ0vqEgb7Xlkfr4uC+sniJvFDThBUWAG39Gs9Wf5TWN93OYvpES4VaNNK4n+Ticuwk
         QhwxOm5RBoRbFo3enTily9T4nhL+j/53dRRxn0zRroJuDOhV0tZGgs8clDR9kyuw9JwT
         bfCQkCMD5ZOQuZYjPSFzFWk7ayCeUTqNqpBgg=
MIME-Version: 1.0
Date: Wed, 21 Jul 2010 13:24:02 +0800
Message-ID: <AANLkTim9SM2m4UNFLOQ-cVRuba1hstvaJqdaUpmUbw4h@mail.gmail.com>
Subject: [classlib][nio_cahr] Issue about CharsetEncdoer.flush() does not
	follow the spec and RI5 also doesn't follow the spec
From: Deven You <devyoudw@gmail.com>
To: dev <dev@harmony.apache.org>
Content-Type: multipart/alternative; boundary=0016e644c70c7e3625048bdf02ef

--0016e644c70c7e3625048bdf02ef
Content-Type: text/plain; charset=ISO-8859-1

See https://issues.apache.org/jira/browse/HARMONY-6594
<https://issues.apache.org/jira/browse/HARMONY-6594>
In Java5 Spec, the flush() method should always be invoked after reset() or the
three-argument encode<http://../../../java/nio/charset/CharsetEncoder.html#encode(java.nio.CharBuffer,+java.nio.ByteBuffer,+boolean)>
 method with a value of true for the endOfInput parameter, otherwise, an
IllegalStateException will be throwed. Harmony's implementation does not
implement this logic, when an encoder is created and followed by calling its
flush() method,  flush() should throw IllegalStateException. I have fix
previous case with
HARMONY-6594<https://issues.apache.org/jira/browse/HARMONY-6594>.
 However I checked that RI5 also  does not completely follow the spec.  For
the invocation sequence: reset -> encode with 3 arguments -> reset  ->
flush, RI5 throw IlegalStateException against the spec.
(seeorg.apache.harmony.nio_char.tests.java.nio.charset.ASCIICharsetEncoderTest.testInternalState_Flushed()
)
and sequence: encode(Charbuffer) -> flush(), RI5 doesn't throw
IlegalStateException against the spec. (after encode(Charbuffer), the
encoder should be in FLUSH state)
 (see seeorg.apache.harmony.nio_char.tests.java.nio.charset.ASCIICharsetEncoderTest.testInternalState_from_Encode)

Further Investigation shows from Java6 Spec, this behavior is changed, it
says  flush() will throw IllegalStateException if the previous step of the
current encoding operation was an invocation neither of the
flush<http://../../../java/nio/charset/CharsetEncoder.html#flush(java.nio.ByteBuffer)>
 method nor of the three-argument
encode<http://../../../java/nio/charset/CharsetEncoder.html#encode(java.nio.CharBuffer,+java.nio.ByteBuffer,+boolean)>
 method with a value of true for the endOfInput parameter. And actually, RI5
follows the java6 spec rather than java5!

So now I am confused if we should modify our harmony trunk CharsetEncoder to
comply with the java5 spec or in other hand modify it to comply with RI5 and
java6 spec for above 2 cases? Anyone could give me some suggestions for this
point?

--0016e644c70c7e3625048bdf02ef--