RFR: 8145461 changes to @Deprecated annotation

[Martin Buchholz]

For similar work in guava, compare e.g. the clarity of
http://docs.guava-libraries.googlecode.com/git/javadoc/com/google/common/io/OutputSupplier.html
"""Deprecated.
For OutputSupplier<? extends OutputStream>, use ByteSink instead. For
OutputSupplier<? extends Writer>, use CharSink. Implementations of
OutputSupplier that don't fall into one of those categories do not
benefit from any of the methods in common.io and should use a
different interface. This interface is scheduled for removal in
December 2015."""

(OTOH, it's already 2016 ...)


On Tue, Mar 29, 2016 at 3:54 PM, Stephen Colebourne
<scolebourne at joda.org> wrote:
> I am pleased to see the simpler form of the API move forward.
>
> However, I feel that condemed() as a boolean does not provide the key
> piece of information necessary - the expected version of removal. I
> would counter-propose that condemed() be replaced by
> expectedDeletion(), a String with the same version semantics as
> since(). While occasionally, the deletion version will be wrong or
> need to be pushed back, in most cases it will be correct. What it does
> do is provides a much better sense to users as to what is going to
> happen.
>
> @Deprecated(since="9", expectedDeletion="11)
> public void myBadBadMethod() {...}
>
> Stephen
>
>
>
> On 29 March 2016 at 23:38, Stuart Marks <stuart.marks at oracle.com> wrote:
>> Hi all,
>>
>> Please review these changes to the @Deprecated annotation:
>>
>>     http://cr.openjdk.java.net/~smarks/reviews/8145461/webrev.0/
>>
>> This is the first of a set of changes for JEP 277, which I hope will become
>> Targeted to JDK 9 late this week. Changes to the deprecation status of
>> specific APIs will be in subsequent changesets, along with some deprecation
>> tooling.
>>
>> Note that this is considerably simplified from previous drafts. In
>> particular, there is no "reason" enum. See the JEP [1] for background and
>> rationale.
>>
>> Thanks,
>>
>> s'marks
>>
>> [1] http://openjdk.java.net/jeps/277
>>