Request for review: 8010833: Test7116786.java is failing on most configs after fix for 8010667

[Vladimir Kozlov]

It is against OpenJDK rules to have binary file (.jar) in open sources. 
It should be moved to closed tests. Or .jar file should be constructed 
during test run (which is impossible based on your comment).

I agree that in such situation (no sources) disabling case58 test is 
acceptable solution.

Thanks,
Vladimir

On 3/26/13 2:38 PM, harold seigel wrote:
> Hi Vladimir,
>
> The test consists of a .java file and a .jar file.  The .java file just
> calls the various test cases that are in the .jar file.  The sources for
> these test cases, including the one for nonzero padding bytes, are not
> provided.  So, they won't get recompiled.
>
> Harold
>
> On 3/26/2013 5:24 PM, Vladimir Kozlov wrote:
>> I am asking this because compiling jdk (jtreg var COMPILEJAVA) could
>> be different from tested jdk (jtreg var TESTJAVA). So you may get old
>> class version of the test file while testing jdk8.
>>
>> Thanks,
>> Vladimir
>>
>> On 3/26/13 2:13 PM, Vladimir Kozlov wrote:
>>> Harold,
>>>
>>> Is it possible to set 'testable' parameter for case58 based on class
>>> file version?
>>>
>>> Thanks,
>>> Vladimir
>>>
>>> On 3/26/13 1:15 PM, harold seigel wrote:
>>>> Please review the following small change to fix bug 8010833
>>>>
>>>> Summary: The fix for 8010667 allows non-zero padding bytes for
>>>> lookupswitch and tableswitch opcodes.  Previously, these had to be
>>>> zero.  Test7116786 is now incorrect because it expects a VerifyError
>>>> exception for non-zero padding bytes for lookupswitch and tableswitch
>>>> opcodes.
>>>>
>>>> Open webrev at http://cr.openjdk.java.net/~hseigel/bug_8010833/
>>>> <http://cr.openjdk.java.net/%7Ehseigel/bug_8010833/>
>>>>
>>>> Bug link at http://bugs.sun.com/view_bug.do?bug_id=8010833
>>>>
>>>> The change was tested by running the test and seeing that it passes.
>>>>
>>>> Thanks, Harold
>