9.80:Strange behaviour CXTButton::SetIcon

In version 9.80, the implementation of CXTButton::SetIcon has changed.

It is internally routed to BOOL CXTPImageManagerIconHandle::CreateIconFromResource(LPCTSTR lpszResourceName, CSize szIcon) which internally has the following code:

...
01: HINSTANCE hInst = AfxFindResourceHandle(lpszResourceName, RT_GROUP_ICON);
02: HRSRC hRsrc = ::FindResource(hInst, lpszResourceName, RT_GROUP_ICON);
03:
04: if (hRsrc != NULL)
05: {
06:  // Load and Lock to get a pointer to a GRPICONDIR
07:  HGLOBAL hGlobal = LoadResource( hInst, hRsrc );
08:
09:  CXTPImageManagerIcon::GRPICONDIR* lpGrpIconDir = (CXTPImageManagerIcon::GRPICONDIR*)LockResource( hGlobal );
10:
11:  for (int i = (int)lpGrpIconDir->idCount - 1; i >= 0; i--)
12:  {
13:   if (lpGrpIconDir->idEntries[ i ].bWidth == szIcon.cx && lpGrpIconDir->idEntries[i].bHeight == szIcon.cy)
14:   {
15:    if (CreateIconFromResource(MAKEINTRESOURCE(lpGrpIconDir->idE ntries[i].nID), szIcon))
16:     return TRUE;
17:   }
18:  }
19:
20:  return FALSE;
21: }
22:
23: hInst = AfxFindResourceHandle(lpszResourceName, RT_ICON);
24: hRsrc = ::FindResource(hInst, lpszResourceName, RT_ICON);
25:
26: if (hRsrc == NULL)
27:  return FALSE;
...

Now I call the CXTButton::SetIcon  from within my code and the result is that the main application icon is loaded instead of the icon that is present and even found by the lines above.

Let me explain:

• Line 01 and 02 do return a non-NULL value.
• Then the code is executed until line 15, calling itself recusively  with a value of 1 for lpGrpIconDir->idEntries[i].nID.
• Then the recursive call of the function with a value of 1 for the lpszResourceName evalutes hRsrc to NULL and thus does not branch into the block starting on line 04 (because a resource ID of 6254 was requested and now the strange recursion passes a resource ID of 1 which actually does not exist).
• Instead, it goes directly to line 23 where the ID of 1 is passed and (probably by Windows default) loads the applications icon.

Question: could you please comment on this? I do believe this is an error, because in the version prior to 9.7 everything works correctly.

Thanks
Uwe

Thank you, I will try the second approach.

BTW: My IDR_MAINFRAME is not 1 but 150. Probably returning the  IDR_MAINFRAME (i.e. the lowest icon) when requesting with ID 1 is a Windows default behaviour.

Just tried, still did not work.

The reason is that it still passes the "1" from lpGrpIconDir->idEntries[i].nID to the first recursion of the function (instead of my real resource ID).

I do not fully understand what the function does, so I'll try it now and see how I can fix it.

Maybe you can tell me another option, too, if I find no solution by myself.

Thanks
Uwe

The problem now that LoadIcon loads 32*32 icon instead of 16*16 so CXTPImageManagerIcon::SetIcon works as expected.

To load corrrect Icon you can remove 32*32 layout from ico file, or use LoadImage instead:

HICON hIcon = (HICON)LoadImage(..,..,IMAGE_ICON, 16, 16, LR_DEFAULTCOLOR);