RFR: 8327138: Clean up status management in cdsConfig.hpp and CDS.java
Ioi Lam
iklam at openjdk.org
Wed Mar 6 02:00:47 UTC 2024
On Tue, 5 Mar 2024 23:49:02 GMT, Calvin Cheung <ccheung at openjdk.org> wrote:
>> Do you mean aligning like this:
>>
>>
>> static bool is_dumping_heap() NOT_CDS_JAVA_HEAP_RETURN_(false);
>> static void stop_dumping_full_module_graph(const char* reason = nullptr) NOT_CDS_JAVA_HEAP_RETURN;
>> static bool is_dumping_full_module_graph() { return CDS_ONLY(_is_dumping_full_module_graph) NOT_CDS(false); }
>> static void stop_using_full_module_graph(const char* reason = nullptr) NOT_CDS_JAVA_HEAP_RETURN;
>> static bool is_using_full_module_graph() NOT_CDS_JAVA_HEAP_RETURN_(false);
>>
>>
>> I think this is more messy, and you can't easily tell that these four functions are all related to the same feature (full_module_graph).
>
> I meant the following. Just the last line #94 needs to be changed - shift one space to the left after `bool`.
>
> static bool is_dumping_heap() NOT_CDS_JAVA_HEAP_RETURN_(false);
> static void stop_dumping_full_module_graph(const char* reason = nullptr) NOT_CDS_JAVA_HEAP_RETURN;
> static bool is_dumping_full_module_graph() { return CDS_ONLY(_is_dumping_full_module_graph) NOT_CDS(false); }
> static void stop_using_full_module_graph(const char* reason = nullptr) NOT_CDS_JAVA_HEAP_RETURN;
> static bool is_using_full_module_graph() NOT_CDS_JAVA_HEAP_RETURN_(false);
I wanted line 94 to align the "using_full_module_graph" part with the function at line 93.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/18095#discussion_r1513713821
More information about the core-libs-dev
mailing list