[crac] RFR: 8375044: [CRaC] Structured engine options info [v4]

[Radim Vansa]

On Mon, 19 Jan 2026 09:26:45 GMT, Timofei Pushkin <tpushkin at openjdk.org> wrote:

>> Radim Vansa has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Fix windows build
>
> src/hotspot/share/include/crlib/crlib_description.h line 50:
> 
>> 48:   const char *value_type;
>> 49:   const char *default_value;
>> 50:   const char *description;
> 
> I'd prefer to have nullability specified for the strings. I think all but `default_value` must be non-nullable. `default_value` probably should be nullable with null representing options without which the engine will always fail C/R (e.g. image and engine locations in case of crexec).

I don't think we need the special `null` case for must-be-set option; it's better to document that there's no default value.

-------------

PR Review Comment: https://git.openjdk.org/crac/pull/286#discussion_r2709091431