This patch looks fine in general.
Does @inheritDoc inherit @deprecated? What does the javadoc of
javax.imageio.stream.* look like?
+ * @deprecated Finalization is deprecated and should not be used.
+ * See {@link Object#finalize} for the explanation and
recommended replacement.
Also, the second line is rather long and might be good to break into two lines?
Mandy
> On Mar 10, 2017, at 1:40 PM, Roger Riggs <[email protected]> wrote:
>
> Finalizers are inherently problematic and their use can lead to performance
> issues,
> deadlocks, hangs, and other problematic behavior.
>
> The problems have been accumulating for many years and the first step to
> deprecate Object.finalize and the overrides in the JDK to communicate the
> issues, recommend alternatives, and motivate changes where finalization is
> currently used.
>
> The behavior of finalization nor any uses of finalize are not modified by
> this change.
> Most of the changes are to suppress compilation warnings within the JDK.
>
> Please review and comment.
>
> Webrev:
> http://cr.openjdk.java.net/~rriggs/webrev-finalize-deprecate-8165641/
>
> Issue:
> https://bugs.openjdk.java.net/browse/JDK-8165641
>
> Thanks, Roger
>
>
