🚀 go-pugleaf

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. "