Parameter "condition" in WARNX macro is used vice versa in
TModule::LoadCursor - module.cpp(872) and
TModule::LoadIcon - module.cpp(909).
Moreover, line with CHECKX macro should be placed just before return (IMHO).
'HBITMAP TModule::LoadBitmap(TResId id) const' should also move the CHECKX(...); to just before the return.
'HACCEL TModule::LoadAccelerators(TResId id) const': uses a WARNX instead (just before the return), we should use a CHECKX to keep consistence (or change all CHECKX to WARNX).
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
Logged In: YES
user_id=873706
Originator: NO
Thanks for finding this, will be included in the next patch
'HBITMAP TModule::LoadBitmap(TResId id) const' should also move the CHECKX(...); to just before the return.
'HACCEL TModule::LoadAccelerators(TResId id) const': uses a WARNX instead (just before the return), we should use a CHECKX to keep consistence (or change all CHECKX to WARNX).
Fixed in 6.21.9 / 6.30
Solved in 6.21.9 / 6.30.