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