Review request : 7141141 Add 3 new test scenarios for testing Main-Class attribute in jar manifest file

Joe Darcy joe.darcy at oracle.com
Thu Feb 2 19:26:55 UTC 2012


Looks fine,

-Joe

On 2/1/2012 7:47 PM, Kumar Srinivasan wrote:
> Hi Joe, David,
>
> Thanks for the feedback, here is the modified delta webrev:
> http://cr.openjdk.java.net/~ksrini/7141141/webrev.1/webrev.delta/index.html 
>
>
> Changes are:
> * Eliminated typos and fixed the comments
> * Replaced @compile with @build
> * Replaced Throwable with IOException this makes it consistent.
>
> Thanks
> Kumar
>
>
>> Hello,
>>
>> On 2/1/2012 1:37 PM, Kumar Srinivasan wrote:
>>> Hi,
>>>
>>> Here are some improvements to the launcher tests, contributed by Sonali
>>> Goel of the SQE team, please review:
>>>
>>> http://cr.openjdk.java.net/~ksrini/7141141/webrev.0/
>>>
>>> The CR:
>>> http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7141141
>>>
>>> Thanks
>>> Kumar
>>>
>>
>> A few stylistic comments:
>>
>> 29  * @compile -XDignore.symbol.file MainClassAttributeTest.java
>>
>> It might suffice to use @build instead of @compile.
>>
>>   43  * only English is seleced and will pass vacuosly for other 
>> locales.
>>   44  *
>>   45  */
>>
>> Extra line.
>>
>>   98     public static void main(String[] args) throws Throwable {
>>
>> Is it necessary to throw Throwable here?
>>
>> Otherwise looks fine.
>>
>> -Joe
>




More information about the core-libs-dev mailing list