On Tue, 7 Mar 2023 01:36:56 GMT, David Holmes wrote:
>> Kim Barrett has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> make Debugging::_enabled a nesting counter
>
> Okay other than the one outstanding minor syntax query, I have nothing
>
The message from this sender included one or more files
which could not be scanned for virus detection; do not
open these files unless you are certain of the sender's intent.
--
On Tue, 7 Mar 2023 13:08:22 GMT, Coleen Phillimore
On Sat, 4 Mar 2023 11:23:58 GMT, Kim Barrett wrote:
>> Also 8302799: Refactor Debugging variable usage for noreturn crash reporting
>
> Kim Barrett has updated the pull request incrementally with one additional
> commit since the last revision:
>
> make Debugging::_enabled a nesting counter
On Tue, 7 Mar 2023 05:58:57 GMT, Kim Barrett wrote:
>> No I'm comparing the syntax used elsewhere:
>>
>> DebuggingContext debugging{};
>>
>> with that used here:
>>
>> DebuggingContext debugging;
>>
>> why the braces in most cases but not here?
>
> "syntax used elsewhere" are declarations of
The message from this sender included one or more files
which could not be scanned for virus detection; do not
open these files unless you are certain of the sender's intent.
--
On Tue, 7 Mar 2023 01:25:47 GMT, David Holmes wrote
On Sat, 4 Mar 2023 11:23:58 GMT, Kim Barrett wrote:
>> Also 8302799: Refactor Debugging variable usage for noreturn crash reporting
>
> Kim Barrett has updated the pull request incrementally with one additional
> commit since the last revision:
>
> make Debugging::_enabled a nesting counter
On Sat, 4 Mar 2023 11:16:10 GMT, Kim Barrett wrote:
>> src/hotspot/share/utilities/debug.cpp line 85:
>>
>>> 83: if (is_enabled()) {
>>> 84: fatal("Multiple Debugging contexts");
>>> 85: }
>>
>> This seems too restrictive as you could hit different DebuggingContexts in
>> different thr
On Sun, 5 Mar 2023 15:36:30 GMT, Julian Waters wrote:
>> I don't understand the question? These are member variable declarations.
>
> I believe David is asking about why these particular member names were changed
No I'm comparing the syntax used elsewhere:
DebuggingContext debugging{};
with t
On Sat, 4 Mar 2023 11:16:03 GMT, Kim Barrett wrote:
>> make/hotspot/lib/CompileJvm.gmk line 103:
>>
>>> 101: DISABLED_WARNINGS_xlc := tautological-compare shift-negative-value
>>> 102:
>>> 103: DISABLED_WARNINGS_microsoft := 4624 4244 4291 4146 4127 4722
>>
>> It is annoying that we don't docu
> Also 8302799: Refactor Debugging variable usage for noreturn crash reporting
Kim Barrett has updated the pull request incrementally with one additional
commit since the last revision:
make Debugging::_enabled a nesting counter
-
Changes:
- all: https://git.openjdk.org/jdk/pul
10 matches
Mail list logo
