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 t.p.ellison@gmail.com
 designates 209.85.220.228 as permitted sender)
DomainKey-Signature: a=rsa-sha1; c=nofws;
        d=gmail.com; s=gamma;
        h=message-id:date:from:user-agent:mime-version:to:subject:references
         :in-reply-to:x-enigmail-version:content-type
         :content-transfer-encoding;
        b=axxNhFDIzGiAWzXV7jbVrN7kG1xbC3JYHFRJb/73yT0SShiHni5C/FDM8KNCjBJsF4
         79ggUa5SUZZEqp9+eIrBS/814KTEzXIBUtvmDameVyNCiw9c+wzr0ps+6x6lF5ffzeVI
         Q8sggPEtRlbjM2Aiyqb3p6RNC5u2GlHcl1DM4=
Message-ID: <4B5DCDE0.1000009@gmail.com>
Date: Mon, 25 Jan 2010 16:59:12 +0000
From: Tim Ellison <t.p.ellison@gmail.com>
User-Agent: Thunderbird 2.0.0.23 (Windows/20090812)
MIME-Version: 1.0
To: dev@harmony.apache.org
Subject: Re: [jira] Commented: (HARMONY-6392) [classlib][swing] Test failures
 	in javax.swing.text.html.HTMLDocument_Reader_ActionsTest
References: <655176759.1259237139553.JavaMail.jira@brutus>
	 <2119824580.9361264417415179.JavaMail.jira@brutus.apache.org>
 <4d9fb1a01001250410h3ec9e5f8r2bdaa5a867e24fbd@mail.gmail.com>
 <4B5D996F.70809@googlemail.com> <4B5DC241.3010404@googlemail.com>
In-Reply-To: <4B5DC241.3010404@googlemail.com>
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 7bit

On 25/Jan/2010 16:09, Oliver Deakin wrote:
> Looks like this is caused by DTD.java's use of available() again. This
> time the test is failing because RAFStream.available() is being called
> by DTD.read() and its return value is expected to be a true indication
> of the number of bytes available. However, the RAFStream.available()
> call will only return 0 or 1 in our implementation which causes the
> DTD.read() to only read a single byte rather than the whole DTD. I
> hacked RAFStream.available() to return (mLength-mOffset) and this makes
> the test pass, but twe really need to fix the DTD.read() method to not
> use available() at all I think.

It looks like both need fixing then.  I thought we had done all these
already?

RAFStream should return the actual number of bytes available, and
DTD.read() should not use available, but read until it gets EOF (-1).

Regards,
Tim

> On 25/01/2010 13:15, Oliver Deakin wrote:
>> Hi Ray,
>>
>> Thanks for the information - from your results it seems that there
>> must be a difference in common classes between java 5 and 6 that are
>> causing the failures. I see the same result - with the java5 M12a
>> swing.jar the test passes 100% and with the java6 swing.jar it fails
>> 100%.
>>
>> I'll dig a little deeper at the code differences between the modules.
>>
>> Regards,
>> Oliver
>>
>> On 25/01/2010 12:10, Ray Chen wrote:
>>> Hi Oliver,
>>> I have done some investigation for this issue before, and disscussed
>>> it on the 6.0 Milestone1 thread.
>>>
>>> I compared two swing.jar of trunk and java6, the only difference
>>> between them is that java6 added some new classes:
>>>
>>> javax\swing\event\RowSorterEvent$Type.class
>>> javax\swing\event\RowSorterEvent.class
>>> javax\swing\event\RowSorterListener.class
>>> javax\swing\filechooser\FileNameExtensionFilter.class
>>> javax\swing\RowSorter$SortKey.class
>>> javax\swing\RowSorter.class
>>> javax\swing\SortOrder.class
>>>
>>> I think these classes are not related to this test. However, even
>>> after I removed these new added classes or replaced trunk's swing.jar
>>> with java6's swing.jar and run the test on the modified trunk , the
>>> test still failed.
>>> And trunk's swing.jar works well in both trunk and java6 to run this
>>> test.
>>> So I am a little confused,  same classes, same vm, why different result?
>>>
>>> On Mon, Jan 25, 2010 at 7:03 PM, Oliver Deakin
>>> (JIRA)<jira@apache.org>  wrote:
>>>>     [
>>>> https://issues.apache.org/jira/browse/HARMONY-6392?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12804502#action_12804502
>>>> ]
>>>>
>>>> Oliver Deakin commented on HARMONY-6392:
>>>> ----------------------------------------
>>>>
>>>> This issue has been fixed in java5 M12, but still persists in the
>>>> java6 branch.
>>>>
>>>>> [classlib][swing] Test failures in
>>>>> javax.swing.text.html.HTMLDocument_Reader_ActionsTest
>>>>> ----------------------------------------------------------------------------------------
>>>>>
>>>>>
>>>>>                  Key: HARMONY-6392
>>>>>                  URL:
>>>>> https://issues.apache.org/jira/browse/HARMONY-6392
>>>>>              Project: Harmony
>>>>>           Issue Type: Bug
>>>>>           Components: Classlib
>>>>>     Affects Versions: 6.0M1, 5.0M12
>>>>>          Environment: Windows x86
>>>>>             Reporter: Oliver Deakin
>>>>>              Fix For: 5.0M12
>>>>>
>>>>>
>>>>> I see 1 failure and 1 error.
>>>>> javax.swing.text.html.HTMLDocument_Reader_ActionsTest.testHarmony_4582
>>>>> fails with output:
>>>>> N/A
>>>>> java.lang.NullPointerException
>>>>> at
>>>>> javax.swing.text.html.HTMLDocument_Reader_ActionsTest.testHarmony_4582(HTMLDocument_Reader_ActionsTest.java:584)
>>>>>
>>>>> at java.lang.reflect.VMReflection.invokeMethod(VMReflection.java)
>>>>> at
>>>>> javax.swing.BasicSwingTestCase.runBareSuper(BasicSwingTestCase.java:116)
>>>>>
>>>>> at
>>>>> javax.swing.BasicSwingTestCase.runBareImpl(BasicSwingTestCase.java:121)
>>>>>
>>>>> at javax.swing.BasicSwingTestCase$1.run(BasicSwingTestCase.java:135)
>>>>> at java.lang.Thread.run(Thread.java:669)
>>>>> javax.swing.text.html.HTMLDocument_Reader_ActionsTest.testHarmony_4615
>>>>> fails with output:
>>>>> null expected:<[line4 line4]>  but was:<[ line4 line]>
>>>>> junit.framework.ComparisonFailure: null expected:<[line4 line4]> 
>>>>> but was:<[line4 line]>
>>>>> at
>>>>> javax.swing.text.html.HTMLDocument_Reader_ActionsTest.testHarmony_4615(HTMLDocument_Reader_ActionsTest.java:601)
>>>>>
>>>>> at java.lang.reflect.VMReflection.invokeMethod(VMReflection.java)
>>>>> at
>>>>> javax.swing.BasicSwingTestCase.runBareSuper(BasicSwingTestCase.java:116)
>>>>>
>>>>> at
>>>>> javax.swing.BasicSwingTestCase.runBareImpl(BasicSwingTestCase.java:121)
>>>>>
>>>>> at javax.swing.BasicSwingTestCase$1.run(BasicSwingTestCase.java:135)
>>>>> at java.lang.Thread.run(Thread.java:669)
>>>>> These tests pass for me against M11.
>>>> -- 
>>>> This message is automatically generated by JIRA.
>>>> -
>>>> You can reply to this email to add a comment to the issue online.
>>>>
>>>>
>>>
>>>
>>
>