On Fri, 17 Jan 2025 17:54:07 GMT, Rajat Mahajan <rmaha...@openjdk.org> wrote:

>> **Issue:** 
>> AwtWindow::SetIconData leaks the old icon handles in hOldIcon and hOldIconSm 
>> if CreateIconFromRaster raises an exception. Additionally, an exception is 
>> checked only after the first call to CreateIconFromRaster.
>> 
>> **Solution:**
>> I have added the exception handling code to take care that the handles are 
>> properly destroyed and not leaked.
>> 
>> **Testing:**
>> I have tested the code to make sure there are no regressions caused by this.
>
> Rajat Mahajan has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   leave opening bracre as is

Marked as reviewed by honkar (Reviewer).

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

PR Review: https://git.openjdk.org/jdk/pull/22932#pullrequestreview-2559628474

Reply via email to