RFR: JDK-8334217 : [AIX] Misleading error messages after JDK-8320005 [v2]

Suchismith Roy sroy at openjdk.org
Tue Jul 9 07:19:38 UTC 2024


On Sat, 29 Jun 2024 13:27:45 GMT, Thomas Stuefe <stuefe at openjdk.org> wrote:

>>> But by using a central common definition for the string you at least minimize the chance of this breaking if someone changes the error messages, and signal too the code reader that the message may be used somewhere else.
>> 
>> Great point. Thanks, thats good learning. 
>> We had worked on checking error numbers before, but there were limitations to that. 
>> Is the current #define code i pushed ,ok ?
>
>> > But by using a central common definition for the string you at least minimize the chance of this breaking if someone changes the error messages, and signal too the code reader that the message may be used somewhere else.
>> 
>> Great point. Thanks, thats good learning. We had worked on checking error numbers before, but there were limitations to that. Is the current #define code i pushed ,ok ?
> 
> No, since you still only use it in one place, and in porting_aix.cpp we still use a literal string.

Thanks.  @tstuefe  Could you kindly provide a review.

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

PR Comment: https://git.openjdk.org/jdk/pull/19887#issuecomment-2216789549


More information about the hotspot-runtime-dev mailing list