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 :/

1 files changed, 1 insertions, 2 deletions

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);
 };