🚀 go-pugleaf

Thank you

Using DestroyIcon() instead DeleteObject (hicon) helped.


"Bob Gonder" <none@nowhere.invalid> wrote in message 
news:s7uf44518nadtjc1g41dp996vb2na00ker@4ax.com...
> Renald Yazmir wrote:
>>Hello
>
> Please edit your posts.
> You have way too much whitespace, making it difficult
> to see enough code on one screen to understand it.
>
>>The function LoadImage() fails when it is called multiple times(around 
>>3325
>>times).
>>
>>void test(void)
>>{
>>    int i = 0;
>>    HIMAGELIST image_list = NULL;
>>    HICON hicon;
>>    int ret = 0;
>>    // Create the imagelist.
>>    image_list = ImageList_Create( 14, 14, ILC_COLOR32 | ILC_MASK, 0, 0);
>>
>>    if (image_list == NULL)
>>                        return;
>>    for (i = 0; i < 10000; i++)
>>    {
>>        hicon = (HICON)LoadImage(
>>                     npw_hInstance,
>>                     MAKEINTRESOURCE(PICT_TEXT),
>>                     IMAGE_ICON,
>>                     0,
>>                     0,
>>                     LR_CREATEDIBSECTION | LR_LOADTRANSPARENT);
>>         if (hicon == NULL)
>>         {
>
> Check the reason for failure:
> http://msdn.microsoft.com/en-us/library/ms648045(VS.85).aspx
> "If the function fails, the return value is NULL. To get
> extended error information, call GetLastError."
>
>>               return;
>>         }else{
>>               DeleteObject (hicon);
> You should be using DestroyIcon() instead.
>>         }
>>         ret = ImageList_AddIcon(image_list, hicon);
>>         if (ret < -1)
> You are not checking that failure properly: if( ret == -1 )
>>         {
>>             return;
>>         }
>>     }
>>      ImageList_Destroy(image_list);
>>}
>>
>>In the code above LoadImage() fails (hicon is NULL) when "i" equals to 
>>3325.
>
> It really should fail at i==0 as you are passing
> an invalid handle to ImageList_AddIcon.
>
> Help:
> "The DeleteObject function deletes a logical pen, brush, font, bitmap, 
> region, or palette,
> freeing all system resources associated with the object. After the object 
> is deleted, the
> specified handle is no longer valid. "
>
>
>