Review Request : JDK-8031670: remove unneeded -source option from javadoc tests

Joseph Darcy joe.darcy at oracle.com
Wed Feb 19 15:42:29 PST 2014


Hi Neil,

On the removal of

     test/tools/javadoc/sourceOption/SourceOption.java

after your patch is there some remaining javadoc test which uses the 
-source option? We probably want to keep one test which uses to the 
option to make sure it is supported. If there is no such remaining test, 
SourceOption.java could be resurrected and make to check that a JDK 8 
feature is rejected under source 7, etc.

Otherwise, the removal of the other -source uses looks fine.

Thanks,

-Joe

On 2/19/2014 7:37 AM, Vicente-Arturo Romero-Zaldivar wrote:
> Hi Neil,
>
> there is a typo at test/tools/javadoc/6964914/JavacWarning.java:
>
> "It *amy* be deprecated in JDK8"
>
> at test/com/sun/javadoc/testLambdaFeature/TestLambdaFeature.java:
>
> I would add the note as an independent comment after the
>
> * @test
> * @bug      8004893 8022738
> section not as part of it.
>
> Vicente
>
> On 18/02/14 21:10, Neil Toda wrote:
>>
>> A small set of javadoc tests contain the -source parameter.  In most 
>> cases, the parameter is
>> not required for the test.  In several cases, the -source parameter 
>> is being explicitly tested,
>> but relies on a JDK version that will be removed from JDK9.
>>
>> This changeset removes unnecessary -source specification when not 
>> needed, or changes the
>> test/source-version requested to one that will work in JDK9.
>>
>> http://cr.openjdk.java.net/~darcy/neiltoda/8031670.0/
>>
>> Thanks
>>
>> -neil
>>
>>
>>
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.openjdk.java.net/pipermail/compiler-dev/attachments/20140219/f27ef68b/attachment.html 


More information about the compiler-dev mailing list