Mikhail,

I already got several personal feedbacks about these comments. I accept
that they are not good. 

I tried to express an idea that JIRAs marked with this comment prevent
getting 100% pass rate on DRLVM. When I search for
Unit_Tests_Pass_on_DRLVM in comments I will get a list of bugs to fix.

Any suggestions how the comment should like? My only requirement is to
keep Unit_Tests_Pass_on_DRLVM lexem in this comment.

With best regards,
Alexei Fedotov,
Intel Middleware Products Division

>-----Original Message-----
>From: Mikhail Loenko [mailto:[EMAIL PROTECTED]
>Sent: Wednesday, October 11, 2006 2:37 PM
>To: harmony-dev@incubator.apache.org
>Subject: Re: [jira] Commented: (HARMONY-1670) [classlib][luni] invalid
>testcase in ClassTest
>
>Hi Alexei
>
>I'm sorry I comlpetely don't understand your comment (as well as a
>number of similar comments in other JIRAs)
>
>Thanks,
>Mikhail
>
>2006/10/7, Alexei Fedotov (JIRA) <[EMAIL PROTECTED]>:
>>    [ http://issues.apache.org/jira/browse/HARMONY-
>1670?page=comments#action_12440591 ]
>>
>> Alexei Fedotov commented on HARMONY-1670:
>> -----------------------------------------
>>
>> [drlvm][unit] Blocks
>http://wiki.apache.org/harmony/Unit_Tests_Pass_on_DRLVM
>>
>> This is a low risk test fix.
>>
>> > [classlib][luni] invalid testcase in ClassTest
>> > ----------------------------------------------
>> >
>> >                 Key: HARMONY-1670
>> >                 URL:
http://issues.apache.org/jira/browse/HARMONY-1670
>> >             Project: Harmony
>> >          Issue Type: Bug
>> >          Components: Classlib
>> >            Reporter: Alexey Varlamov
>> >            Priority: Trivial
>> >         Attachments: H-1670.patch
>> >
>> >
>> > The test_getDeclaredMethodLjava_lang_String$Ljava_lang_Class of
>modules/luni/src/test/java/org/apache/harmony/luni/tests/java/lang/Clas
sTes
>t.java
>> > checks if IllegalAccessException is thrown during invokation of
private
>method of inner class. This is bad due to:
>> > 1) the check is invalid, see HARMONY-1309 for details;
>> > 2) The ClassTest is not appropriate for testing reflection, and
this is
>already covered in j.l.reflect.* tests.
>>
>> --
>> This message is automatically generated by JIRA.
>> -
>> If you think it was sent incorrectly contact one of the
administrators:
>http://issues.apache.org/jira/secure/Administrators.jspa
>> -
>> For more information on JIRA, see:
http://www.atlassian.com/software/jira
>>
>>
>>
>
>---------------------------------------------------------------------
>Terms of use : http://incubator.apache.org/harmony/mailing.html
>To unsubscribe, e-mail: [EMAIL PROTECTED]
>For additional commands, e-mail: [EMAIL PROTECTED]

---------------------------------------------------------------------
Terms of use : http://incubator.apache.org/harmony/mailing.html
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to