Mailing-List: contact dev-help@harmony.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@harmony.apache.org
Received-SPF: pass (athena.apache.org: domain of nbeyer@gmail.com designates
 64.233.178.244 as permitted sender)
DomainKey-Signature: a=rsa-sha1; c=nofws;
        d=gmail.com; s=gamma;
        h=message-id:date:from:sender:to:subject:in-reply-to:mime-version
         :content-type:content-transfer-encoding:content-disposition
         :references:x-google-sender-auth;
        b=eDovwoHHTWWicR8ECBzs4MG4EuqcW8OSQkL/qE7VEfeF2mZPW5LAyv52OMHSFC9Svs
         wkT9G6F54kPq5cJltBqlFeACa6PJ7u7eJv77imKe5AAHO/O2mY/qktsRfNNCBC1ZVnJ6
         Kxfd2vsnimCIPfucxKLqrOrUba9nRB7yueteY=
Message-ID: <3b3f27c60811091346h61eb8a13u9013e91c7766c510@mail.gmail.com>
Date: Sun, 9 Nov 2008 15:46:01 -0600
From: "Nathan Beyer" <ndbeyer@apache.org>
Sender: nbeyer@gmail.com
To: dev@harmony.apache.org
Subject: Re: [general] M8 code freeze now in effect
In-Reply-To: <5899fca70811091337g39cd913bx290f10d9a9ac3353@mail.gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
Content-Disposition: inline
References: <fc2fc95c0811030314t1f07ca96ybdb44cafb8ff9c6b@mail.gmail.com>
	 <f9761b910811070512m680e33acg7f63f3fa9fd252b1@mail.gmail.com>
	 <fc2fc95c0811070520n1f948830o2806e6d50bfb26b5@mail.gmail.com>
	 <491442B9.3060703@gmail.com>
	 <fc2fc95c0811070629i13b3f3fbg71ba755b107459a2@mail.gmail.com>
	 <f9761b910811070715t15b416b1rd2794f1e5aca8dd3@mail.gmail.com>
	 <4914824A.8070404@gmail.com>
	 <3b3f27c60811091141x58af739fu65012830f04cdd69@mail.gmail.com>
	 <3b3f27c60811091155t2cd2ad5bgab89b37ea04b3691@mail.gmail.com>
	 <5899fca70811091337g39cd913bx290f10d9a9ac3353@mail.gmail.com>

I can't say that it just started showing up for me, I just started
running a full-length test on Win today and this was the first I ran
into.

I suspect it has been failing since we enabled assertions for all
tests. The integrity page is showing the same failure on Win [1].

[1] http://people.apache.org/~varlax/harmony-integrity/windows_x86/classlib-gui-test/

On Sun, Nov 9, 2008 at 3:37 PM, Tim Ellison <t.p.ellison@gmail.com> wrote:
> On 09/11/2008, Nathan Beyer <ndbeyer@apache.org> wrote:
>> It looks like this class has been like this since the initial contribution
>> [1].
>
> Which implies it should not stop us declaring M8 - but any idea why it
> has started failing for you now?
>
> Regards,
> Tim
>
>>
>> [1]
>> http://svn.eu.apache.org/viewvc/harmony/enhanced/classlib/trunk/modules/awt/src/main/java/common/java/awt/GridBagLayout.java?view=log
>>
>> On Sun, Nov 9, 2008 at 1:41 PM, Nathan Beyer <ndbeyer@apache.org> wrote:
>>> I'm seeing an test error in 'awt'.
>>>
>>>  <testcase classname="java.awt.GridBagLayoutRTest"
>>> name="testAddLayoutComponent" time="0.0">
>>>    <error message="AddLayoutComponent: attempt to add null component"
>>> type="java.lang.AssertionError">java.lang.AssertionError:
>>> AddLayoutComponent: attempt to add null component
>>>        at java.awt.GridBagLayout.addLayoutComponent(GridBagLayout.java:94)
>>>        at
>>> java.awt.GridBagLayoutRTest.testAddLayoutComponent(GridBagLayoutRTest.java:105)
>>>        at java.lang.reflect.VMReflection.invokeMethod(VMReflection.java)
>>>
>>>
>>> I'm not exactly sure what the test is supposed to be doing, but I'm
>>> guessing it is testing for some sort of null/illegal parameter. The
>>> interesting thing or actual issue is that the code check the parameter
>>> for null via an assert.
>>>
>>>           // awt.7F=AddLayoutComponent: attempt to add null component
>>>            assert comp != null : Messages.getString("awt.7F");
>>> //$NON-NLS-1$
>>>
>>> At some point, this test wouldn't have been failing because asserts
>>> weren't enabled, but now they are and an assertion is being thrown.
>>> I'm going to do some digging to figure out what the correct behavior
>>> should be and maybe see about getting this fix into M8.
>>>
>>> The important thing though is that we need to understand the
>>> appropriate way to use asserts and this isn't one of them. An
>>> assertion SHOULD never fail. If an assertion fails, that should
>>> indicate a programming error within the code; in this case, a Harmony
>>> programming error. Indicating a consumer programming error should be
>>> done via runtime exceptions.
>>>
>>> -Nathan
>>>
>>>
>>> On Fri, Nov 7, 2008 at 12:00 PM, Tim Ellison <t.p.ellison@gmail.com>
>>> wrote:
>>>> chunrong lai wrote:
>>>>>  I am bothly OK with waiting results of the reliability testing or
>>>>> voting
>>>>> with current testing status (to stick to the schedule).
>>>>
>>>> Quality is more important than dates -- so I favor delaying the vote
>>>> until the testing is complete and any issues addressed.
>>>>
>>>> Regards,
>>>> Tim
>>>>
>>>>
>>>>> On Fri, Nov 7, 2008 at 10:29 PM, Sian January
>>>>> <sianjanuary@googlemail.com>wrote:
>>>>>
>>>>>> Sorry, that was an assumption on my part because it's quite late by
>>>>>> CST.
>>>>>>
>>>>>> Shall we aim for Monday, or are people quite keen to stick to the
>>>>>> schedule?
>>>>>>
>>>>>>
>>>>>> 2008/11/7 Tim Ellison <t.p.ellison@gmail.com>:
>>>>>>  > Sian January wrote:
>>>>>>>> Great - thanks Chunrong.  In that case I think we should leave the
>>>>>>>> vote until next week as there is testing still ongoing.
>>>>>>> Why next week?  Will it really take that long for the testing to be
>>>>>>> completed Chunrong?
>>>>>>>
>>>>>>> Regards,
>>>>>>> Tim
>>>>>>>
>>>>>>>
>>>>>>>> 2008/11/7 chunrong lai <chunronglai@gmail.com>:
>>>>>>>>>  I will upload the snapshot testing result page soon.
>>>>>>>>>  The reliability testing has not been finished.
>>>>>>>>>
>>>>>>>>>
>>>>>>>>> On Fri, Nov 7, 2008 at 7:12 PM, Sian January <
>>>>>> sianjanuary@googlemail.com>wrote:
>>>>>>>>>> Are there any objections to starting a vote for M8 later today?
>>>>>>>>>>
>>>>>>>>>> --
>>>>>>>>>>  Unless stated otherwise above:
>>>>>>>>>> IBM United Kingdom Limited - Registered in England and Wales with
>>>>>> number
>>>>>>>>>> 741598.
>>>>>>>>>> Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire
>>>>>>>>>> PO6
>>>>>> 3AU
>>>>>>>>
>>>>>>>>
>>>>>>
>>>>>>
>>>>>> --
>>>>>>  Unless stated otherwise above:
>>>>>> IBM United Kingdom Limited - Registered in England and Wales with
>>>>>> number
>>>>>> 741598.
>>>>>> Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6
>>>>>> 3AU
>>>>>>
>>>>>
>>>>
>>>
>>
>