[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-2d-dev
Subject:    Re: [OpenJDK 2D-Dev] RFR: 8051519
From:       Mandy Chung <mandy.chung () oracle ! com>
Date:       2016-02-08 21:15:15
Message-ID: C4D88DC4-74F1-4752-B4E7-252D9DE0E0F1 () oracle ! com
[Download RAW message or body]


> On Feb 8, 2016, at 12:40 PM, Phil Race <philip.race@oracle.com> wrote:
> 
> Bug : https://bugs.openjdk.java.net/browse/JDK-8051519
> This internal AP being deprecated in preparation for removal in JDK 10.
> Ordinarily we would remove it directly but it is known to be used by
> non-JDK code, hence the deprecation. CCC approval is already obtained
> 
> diff in-line below
> 
> diff --git a/src/java.desktop/share/classes/sun/java2d/SunGraphicsEnvironment.java \
>                 b/src/java.desktop/share/classes/sun/java2d/SunGraphicsEnvironment.java
>                 
> --- a/src/java.desktop/share/classes/sun/java2d/SunGraphicsEnvironment.java
> +++ b/src/java.desktop/share/classes/sun/java2d/SunGraphicsEnvironment.java
> @@ -208,8 +208,7 @@
> * to use Mincho instead of Gothic for dialoginput in JA locales
> * on windows. Not needed on other platforms.
> *
> -     * DO NOT MOVE OR RENAME OR OTHERWISE ALTER THIS METHOD.
> -     * ITS USED BY SOME NON-JRE INTERNAL CODE.
> +     * @deprecated as of JDK9. To be removed in a future release.
> */
> public static void useAlternateFontforJALocales() {
> getFontManagerForSGE().useAlternateFontforJALocales();
> 

Do you want to annotate the method with @Deprecated to make it clear and also users \
of this internal API will get compiler warning?

FWIW.  When compiled with javac with module system integrated, they will get \
compilation error anyway.  Since the javadoc has @deprecated, annotating with \
@Deprecated makes it clear.

Mandy 


[prev in list] [next in list] [prev in thread] [next in thread]