Looks good.
Thanks,
David
On 23/05/2019 12:44 pm, Henry Jen wrote:
On May 22, 2019, at 7:04 PM, David Holmes <[email protected]> wrote:
On 23/05/2019 11:56 am, Henry Jen wrote:
Thanks for the quick review,
On May 22, 2019, at 6:34 PM, David Holmes <[email protected]> wrote:
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.
I was wondering if I should prefix (deprecated) or append to the content. It
make the line over the 80 so I decided just remove it since this document is
not official we don’t want to advocate that option any more.
So, we have couple options,
-Xfuture enable strictest checks, anticipating future default
(deprecated)
-Xrs reduce use of OS signals by Java/VM (see documentation)
I prefer option 1.
OK, updated[1].
[1] https://cr.openjdk.java.net/~henryjen/jdk13/8215156/open/1/webrev/
Cheers,
Henry
