> I'm fine if you want to move ahead with this patch.

Thanks Mandy - much appreciated! We can continue this refinement for 14.

I have filed JDK-8227229 to deprecate and remove Xdebug - for history see:

https://bugs.openjdk.java.net/browse/JDK-6272174

BTW thanks for the reference to JEP-293, I wasn't aware of it. Would be nice to expand it to give more guidelines on handling deprecation etc.

Cheers,
David
-----

On 4/07/2019 3:36 am, Mandy Chung wrote:


On 7/2/19 11:10 PM, David Holmes wrote:
Hi Mandy,

Thanks for taking a look.

On 3/07/2019 8:57 am, Mandy Chung wrote:
On 7/2/19 3:44 PM, David Holmes wrote:
webrev: http://cr.openjdk.java.net/~dholmes/8227055/webrev/
bug: https://bugs.openjdk.java.net/browse/JDK-8227055

The launcher help text needs some minor updates/corrections
- -verbose needs more info
- -Xdebug needs to say it does nothing

Should this be removed instead?

There is some strange history/status with this flag. I expect the flag and thus documentation should be removed, but that would be a seperate issue for 14 (or later). For 13 I'm just aligning the help text with the manpage description.


I would propose  to take -Xdebug out from the launcher help message and man page but -Xdebug is still accepted but do nothing.

- -Xloggc is deprecated and replaced by -Xlog:gc:filename

I think it can just list -Xlog:gc<file> (the replacement) and no need
to mention the deprecated option in the launcher help message.

Again this aligns with the manpage which does list it as deprecated and its replacement. Otherwise the launcher help doesn't document specific -Xlog usages.


-Xloggc was deprecated since 9 (or earlier [1]).  I think documenting the recommended option instead is the right thing to do.  I think it worths the discussion and includes some guideline in JEP 293 (Guidelines for JDK CLI) [2] w.r.t. deprecated options in the help message vs man page.

I'm fine if you want to move ahead with this patch.

- -Xshare:on needs warning about use

Reference to Mac OS X should be macOS.

The --disable-@files is documented as both a standard and extra option.


I think this is intended to be an extra option.

This is from Alan:

"I'm not sure how `-disable-@files` ended up in both the `-help` and `-X` output, it probably make sense to drop it from the `-X` output. That way, `@argument` and `-disable-@files` are listed together, not in separate usage pages."

(The bug explains how it ended up in both places - and yes it started as a -X option.)


Hmm... Alan and Jon are both on vacation.  We can follow up this separately.

Mandy
[1] https://bugs.openjdk.java.net/browse/JDK-8170636
[2] https://openjdk.java.net/jeps/293

Reply via email to