Windows Template Library (WTL) / Discussion / Open Discussion: CResource::LoadEx bug

CResource::LoadEx bug

Forum: Open Discussion

Creator: thims

Created: 2014-08-28

Updated: 2014-10-02

thims - 2014-08-28

It passes parameters to FindResourceEx() in wrong order.

To fix it without breaking the existing apps, please rename params in function prototype (line 1155 in atluser.h) like this:

bool LoadEx(ATL::_U_STRINGorID ID, ATL::_U_STRINGorID Type, WORD wLanguage)

and change line 1160 like this:

m_hResource = ::FindResourceEx(ModuleHelper::GetResourceInstance(), Type.m_lpstr, ID.m_lpstr, wLanguage);

Last edit: thims 2014-08-28

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Nenad Stefanovic - 2014-09-26

Thanks for reporting this. I will consider your solution.

Cheers,
Nenad

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Nenad Stefanovic - 2014-10-02

What a mess... I hate doing this, but it is the only reasonable way to fix the problem.

Fixed as proposed - atluser.h, 596

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Log in to post a comment.