From e78b625172199f77989015cc7ff817530eecaafd Mon Sep 17 00:00:00 2001 From: Pietro Gagliardi Date: Thu, 9 Apr 2015 19:07:41 -0400 Subject: Changed onWM_NOTIFY() to only pass the NMHDR * in. The LPARAM is more useful as a NMHDR *, and the WPARAM contains duplicate data (http://blogs.msdn.com/b/oldnewthing/archive/2013/12/04/10473637.aspx). Ideally we wouldn't even have the NMHDR.idFrom (see previous commit message), but oh well :/ --- new/ui_windows.h | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) (limited to 'new/ui_windows.h') diff --git a/new/ui_windows.h b/new/ui_windows.h index b89af3a..cfea287 100644 --- a/new/ui_windows.h +++ b/new/ui_windows.h @@ -22,9 +22,8 @@ struct uiWindowsNewControlParams { // ui redirects the message back and calls these functions. // Store the result in *lResult and return any non-FALSE value (such as TRUE) to return the given result; return FALSE to pass the notification up to your window procedure. // Note that these are only issued if they come from the uiControl itself; notifications from children of the uiControl (such as a header control) will be received normally. - // TODO don't give WPARAM/LPARAM raw BOOL (*onWM_COMMAND)(uiControl *c, WORD code, LRESULT *lResult); - BOOL (*onWM_NOTIFY)(uiControl *c, WPARAM wParam, LPARAM lParam, LRESULT *lResult); + BOOL (*onWM_NOTIFY)(uiControl *c, NMHDR *nm, LRESULT *lResult); // This is called in WM_DESTROY. void (*onWM_DESTROY)(uiControl *c); }; -- cgit v1.2.3