Hi Henry,

On 23/05/2019 11:07 am, Henry Jen wrote:
Hi,

Please review a webrev[1] to deprecate the -Xfuture option per CSR-8224524[2].

src/hotspot/share/Xusage.txt

Ignoring the usefulness, or otherwise of this file, the entry for Xfuture should not be deleted (that happens when an option is actually removed) but just have "(deprecated)" added to it.

---

In hotspot we include the version of the JDK that deprecated the option:

"Option %s was deprecated in version %s and will likely be removed in a future release."

Perhaps you could do the same for the launcher?

Otherwise seems fine.

Thanks,
David
-----

Cheers,
Henry
[1] https://cr.openjdk.java.net/~henryjen/jdk13/8215156/open/webrev/
[2] https://bugs.openjdk.java.net/browse/JDK-8224524

Reply via email to