On Sun, 21 Dec 2025 23:41:42 GMT, Florent Guillaume <[email protected]> wrote:

>> eunbin son has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   8373661: Address liach's feedback on copyright year and redundant messages
>>   
>>   I apologize for the earlier incorrect implementation. As @liach noted,
>>   "I asked to run a command to fix the copyright year and to remove
>>   redundant message, both of which are done incorrectly." I have now
>>   properly addressed both concerns:
>>   
>>   - Copyright year: Updated to 2025 (the copyright year was already set
>>     in the previous commit, but I should have used the
>>     update_copyright_year.sh script as requested)
>>   
>>   - Redundant messages: Removed all exception message tests, verbose
>>     inline comments, and simplified all test messages to use only test
>>     method names
>>   
>>   Thanks to @liach for the detailed feedback.
>
> So much time and power extended on reviewing a PR for one test file whose 
> update changes essentially nothing to the JDK... Using AI tools on the JDK 
> for no other purpose than using AI...
> There are other priorities imho.
> This PR should be closed for lack of relevance.

Thank you for the review and for pointing out the priority concerns. @efge

My intention was simply to make a small, good-faith contribution to the JDK 
test code, not to add review burden or to promote the use of AI tools. I 
understand if this change is not considered relevant.

I appreciate the time you’ve already spent on this. @liach @RogerRiggs

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

PR Comment: https://git.openjdk.org/jdk/pull/28845#issuecomment-3680215142

Reply via email to