Re: RFR: 8301627: System.exit and Runtime.exit debug logging [v7]

2023-02-20 Thread Roger Riggs
> It can be difficult to find the cause of calls to > `java.lang.System.exit(status)` and `Runtime.exit(status)` because the Java > runtime exits. > The status value and stack trace are logged using the System Logger named > `java.lang.Runtime` with message level `System.Logger.Level.DEBUG`. Ro

Re: RFR: 8301627: System.exit and Runtime.exit debug logging [v6]

2023-02-20 Thread Chris Hegarty
On Fri, 17 Feb 2023 17:27:50 GMT, Roger Riggs wrote: >> It can be difficult to find the cause of calls to >> `java.lang.System.exit(status)` and `Runtime.exit(status)` because the Java >> runtime exits. >> The status value and stack trace are logged using the System Logger named >> `java.lang.

Re: RFR: 8301627: System.exit and Runtime.exit debug logging [v6]

2023-02-17 Thread Alan Bateman
On Fri, 17 Feb 2023 17:27:50 GMT, Roger Riggs wrote: >> It can be difficult to find the cause of calls to >> `java.lang.System.exit(status)` and `Runtime.exit(status)` because the Java >> runtime exits. >> The status value and stack trace are logged using the System Logger named >> `java.lang.

Re: RFR: 8301627: System.exit and Runtime.exit debug logging [v6]

2023-02-17 Thread Roger Riggs
> It can be difficult to find the cause of calls to > `java.lang.System.exit(status)` and `Runtime.exit(status)` because the Java > runtime exits. > The status value and stack trace are logged using the System Logger named > `java.lang.Runtime` with message level `System.Logger.Level.DEBUG`. Ro

Re: RFR: 8301627: System.exit and Runtime.exit debug logging [v5]

2023-02-17 Thread Roger Riggs
On Fri, 17 Feb 2023 07:48:41 GMT, Alan Bateman wrote: >> Roger Riggs has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Improve implNote for Runtime.exit() with review suggestions. > > src/java.base/share/classes/java/lang/Runtime.java line

Re: RFR: 8301627: System.exit and Runtime.exit debug logging [v5]

2023-02-16 Thread Alan Bateman
On Thu, 16 Feb 2023 21:53:00 GMT, Roger Riggs wrote: >> It can be difficult to find the cause of calls to >> `java.lang.System.exit(status)` and `Runtime.exit(status)` because the Java >> runtime exits. >> The status value and stack trace are logged using the System Logger named >> `java.lang.

Re: RFR: 8301627: System.exit and Runtime.exit debug logging [v5]

2023-02-16 Thread Roger Riggs
> It can be difficult to find the cause of calls to > `java.lang.System.exit(status)` and `Runtime.exit(status)` because the Java > runtime exits. > The status value and stack trace are logged using the System Logger named > `java.lang.Runtime` with message level `System.Logger.Level.DEBUG`. Ro

Re: RFR: 8301627: System.exit and Runtime.exit debug logging [v4]

2023-02-16 Thread Roger Riggs
On Tue, 14 Feb 2023 19:02:52 GMT, Stuart Marks wrote: >> Roger Riggs has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Correct System.getLogger link > > src/java.base/share/classes/java/lang/Shutdown.java line 168: > >> 166:

Re: RFR: 8301627: System.exit and Runtime.exit debug logging [v4]

2023-02-16 Thread Roger Riggs
On Thu, 16 Feb 2023 16:52:14 GMT, Sean Mullan wrote: >> Roger Riggs has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Correct System.getLogger link > > src/java.base/share/classes/java/lang/Runtime.java line 160: > >> 158: * >> 159:

Re: RFR: 8301627: System.exit and Runtime.exit debug logging [v4]

2023-02-16 Thread Roger Riggs
On Tue, 14 Feb 2023 13:10:41 GMT, Daniel Fuchs wrote: >> Roger has updated this but it's still a comment on a non-public class. I >> think the main question here is whether there should be a note in the >> System.exit and Runtime.exit to document that these methods log? If not, >> will it be d

Re: RFR: 8301627: System.exit and Runtime.exit debug logging [v4]

2023-02-16 Thread Roger Riggs
On Mon, 13 Feb 2023 11:02:14 GMT, Daniel Fuchs wrote: >> test/jdk/java/lang/runtime/RuntimeExitLogTest.java line 89: >> >>> 87: } >>> 88: cmd.add(this.getClass().getName()); >>> 89: cmd.add(Integer.toString(status)); >> >> Another possibility for testing this is to launc

Re: RFR: 8301627: System.exit and Runtime.exit debug logging [v4]

2023-02-16 Thread Sean Mullan
On Tue, 14 Feb 2023 18:56:29 GMT, Roger Riggs wrote: >> It can be difficult to find the cause of calls to >> `java.lang.System.exit(status)` and `Runtime.exit(status)` because the Java >> runtime exits. >> The status value and stack trace are logged using the System Logger named >> `java.lang.

Re: RFR: 8301627: System.exit and Runtime.exit debug logging [v4]

2023-02-14 Thread Stuart Marks
On Tue, 14 Feb 2023 18:56:29 GMT, Roger Riggs wrote: >> It can be difficult to find the cause of calls to >> `java.lang.System.exit(status)` and `Runtime.exit(status)` because the Java >> runtime exits. >> The status value and stack trace are logged using the System Logger named >> `java.lang.

Re: RFR: 8301627: System.exit and Runtime.exit debug logging [v4]

2023-02-14 Thread Roger Riggs
> It can be difficult to find the cause of calls to > `java.lang.System.exit(status)` and `Runtime.exit(status)` because the Java > runtime exits. > The status value and stack trace are logged using the System Logger named > `java.lang.Runtime` with message level `System.Logger.Level.DEBUG`. Ro

Re: RFR: 8301627: System.exit and Runtime.exit debug logging [v3]

2023-02-14 Thread Roger Riggs
On Tue, 14 Feb 2023 17:15:28 GMT, Daniel Fuchs wrote: >> Roger Riggs has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Add an @implNote to Runtime.exit to describe the java.lang.Runtime logging. > > src/java.base/share/classes/java/lang/Ru

Re: RFR: 8301627: System.exit and Runtime.exit debug logging [v3]

2023-02-14 Thread Daniel Fuchs
On Tue, 14 Feb 2023 16:46:29 GMT, Roger Riggs wrote: >> It can be difficult to find the cause of calls to >> `java.lang.System.exit(status)` and `Runtime.exit(status)` because the Java >> runtime exits. >> The status value and stack trace are logged using the System Logger named >> `java.lang.

Re: RFR: 8301627: System.exit and Runtime.exit debug logging [v3]

2023-02-14 Thread Roger Riggs
> It can be difficult to find the cause of calls to > `java.lang.System.exit(status)` and `Runtime.exit(status)` because the Java > runtime exits. > The status value and stack trace are logged using the System Logger named > `java.lang.Runtime` with message level `System.Logger.Level.DEBUG`. Ro

Re: RFR: 8301627: System.exit and Runtime.exit debug logging [v2]

2023-02-14 Thread Daniel Fuchs
On Tue, 14 Feb 2023 07:45:07 GMT, Alan Bateman wrote: >> FINE is not a level supported by the System.Logger, it is the level to which >> DEBUG is mapped when the backend is java.util.logging. I suggest to remove >> FINE from this description and add an `{@link Loger.Level#DEBUG DEBUG}` >> arou

Re: RFR: 8301627: System.exit and Runtime.exit debug logging [v2]

2023-02-13 Thread Alan Bateman
On Mon, 13 Feb 2023 10:57:35 GMT, Daniel Fuchs wrote: >> src/java.base/share/classes/java/lang/Shutdown.java line 162: >> >>> 160: * If the system logger {@code java.lang.Runtime} is enabled for >>> logging level DEBUG/FINE >>> 161: * the stack trace of the call to {@code Runtime.exit

Re: RFR: 8301627: System.exit and Runtime.exit debug logging [v2]

2023-02-13 Thread Roger Riggs
> It can be difficult to find the cause of calls to > `java.lang.System.exit(status)` and `Runtime.exit(status)` because the Java > runtime exits. > The status value and stack trace are logged using the System Logger named > `java.lang.Runtime` with message level `System.Logger.Level.DEBUG`. Ro

Re: RFR: 8301627: System.exit and Runtime.exit debug logging

2023-02-13 Thread Daniel Fuchs
On Sun, 12 Feb 2023 18:15:25 GMT, Alan Bateman wrote: >> It can be difficult to find the cause of calls to >> `java.lang.System.exit(status)` and `Runtime.exit(status)` because the Java >> runtime exits. >> The status value and stack trace are logged using the System Logger named >> `java.lang

Re: RFR: 8301627: System.exit and Runtime.exit debug logging

2023-02-12 Thread Alan Bateman
On Fri, 10 Feb 2023 20:33:39 GMT, Roger Riggs wrote: > It can be difficult to find the cause of calls to > `java.lang.System.exit(status)` and `Runtime.exit(status)` because the Java > runtime exits. > The status value and stack trace are logged using the System Logger named > `java.lang.Runti

Re: RFR: 8301627: System.exit and Runtime.exit debug logging

2023-02-12 Thread Alan Bateman
On Fri, 10 Feb 2023 20:33:39 GMT, Roger Riggs wrote: > It can be difficult to find the cause of calls to > `java.lang.System.exit(status)` and `Runtime.exit(status)` because the Java > runtime exits. > The status value and stack trace are logged using the System Logger named > `java.lang.Runti