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

Vicente-Arturo Romero-Zaldivar vicente.romero at oracle.com
Fri Feb 21 12:53:21 PST 2014


Hi Neil,

On 21/02/14 01:25, Neil Toda wrote:
>
> There is another test that does a negative check on a feature, 
> testLambdaFeature.
> However, it too will stop working when support for 1.7 is removed from 
> the JDK.
> Granted that will be several years from now, but by then we'll all 
> forget and just
> remove the negative test.

I hope that we won't ever remove negative tests just because they fail.

>
> I think I've made a mistake taking out the ./sourceOption test. It is 
> better that this
> test exist and be specifically designed to make sure -source is working.

Agree.

>
> With that in mind, I'll not remove it, and add to it so that we have a 
> list of features
> in each release that will fail in the next release.  Something like:
>
> Release        Feature
> =======        ===================
> 1.6            List<String> list = ArrayList<String>();
> 1.7            List<String> list = ArrayList<>();
> 1.8            Lambda or default methods
> 1.9            TBD
>
> So for JDK9, the test will specify:
>     -source 1.6
>     A.java will contain : List<String> list = ArrayList<>();
>
>     and the test will look for failure.
>
> In this way, the intent will by conveyed in the test.
>
> Does this all sound okay?

I would prefer to see the proposed change, I'm not sure I fully 
understand what you want to do here.

Thanks,
Vicente
>
> Thanks Joe
>
> -neil
>
>
> On 2/19/2014 3:42 PM, Joseph Darcy wrote:
>> 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/20140221/087335e8/attachment.html 


More information about the compiler-dev mailing list