On Tue, 21 Apr 2026 11:22:00 GMT, Frederic Parain <[email protected]> wrote:

>> Alright. I do not think I understand exactly what it means that we fail with 
>> preloading these classes. 
>> 
>> I think in both this code patch "app substituted a different version of %s" 
>> -> "app substituted a different version" is probably a better logging. But 
>> not relevant for this fix patch.
>
> +1 on removal of the second printing of the class name.

I removed the second printing of the class name and adjusted the indentation to 
match the first line. I also adjusted the other very similar print in 
`SystemDictionary::preload_from_null_free_field`.

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

PR Review Comment: 
https://git.openjdk.org/valhalla/pull/2344#discussion_r3117160539

Reply via email to