Bad message handler prototypes

Brought to you by: nenadstefanovic

Bad message handler prototypes

Forum: Open Discussion

Creator: Tsuda Kageyu

Created: 2011-10-15

Updated: 2013-05-13

Tsuda Kageyu - 2011-10-15

Currently, CMenu class is used in some message handler prorotypes. For example,

void OnInitMenuPopup(CMenu menuPopup, UINT nIndex, BOOL bSysMenu)

But I think we should use CMenuHandle instead of CMenu, because CMenu calls DestroyMenu.
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Tsuda Kageyu - 2011-10-16

MSG_WM_SYSCOMMAND hanlder prototype is wrong too.

// void OnSysCommand(UINT nID, LPARAM lParam)

The second parameter should be CPoint.
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Nenad Stefanovic - 2011-10-19

You are correct. We will fix that.

Thanks,
Nenad

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.