interesting
/
libui
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Changed onWM_COMMAND() to give our controls only the notification code. We don't want to give the control the ID part of wParam because that's (or that'll be) dynamically assigned based on control parenting and prior controls; we don't want to give the control the LPARAM since that contains the window handle and uiControl already has that.

This commit is contained in:
Pietro Gagliardi 2015-04-09 18:54:14 -04:00
parent d4271c8ceb
commit a538412df2
5 changed files with 8 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
button_windows.c
View File

@ -6,11 +6,11 @@ struct button {
void *onClickedData;
};
static BOOL onWM_COMMAND(uiControl *c, WPARAM wParam, LPARAM lParam, LRESULT *lResult)
static BOOL onWM_COMMAND(uiControl *c, WORD code, LRESULT *lResult)
{
struct button *b = (struct button *) (c->data);
if (HIWORD(wParam) != BN_CLICKED)
if (code != BN_CLICKED)
return FALSE;
(*(b->onClicked))(c, b->onClickedData);
*lResult = 0;

4
checkbox_windows.c
View File

@ -6,13 +6,13 @@ struct checkbox {
void *onToggledData;
};
static BOOL onWM_COMMAND(uiControl *c, WPARAM wParam, LPARAM lParam, LRESULT *lResult)
static BOOL onWM_COMMAND(uiControl *c, WORD code, LRESULT *lResult)
{
struct checkbox *cc = (struct checkbox *) (c->data);
HWND hwnd;
WPARAM check;
if (HIWORD(wParam) != BN_CLICKED)
if (code != BN_CLICKED)
return FALSE;
// we didn't use BS_AUTOCHECKBOX (see controls_windows.go) so we have to manage the check state ourselves

2
entry_windows.c
View File

@ -4,7 +4,7 @@
struct entry {
};
static BOOL onWM_COMMAND(uiControl *c, WPARAM wParam, LPARAM lParam, LRESULT *lResult)
static BOOL onWM_COMMAND(uiControl *c, WORD code, LRESULT *lResult)
{
return FALSE;
}

4
newcontrol_windows.c
View File

@ -5,7 +5,7 @@ typedef struct singleHWND singleHWND;
struct singleHWND {
HWND hwnd;
BOOL (*onWM_COMMAND)(uiControl *, WPARAM, LPARAM, LRESULT *);
BOOL (*onWM_COMMAND)(uiControl *, WORD, LRESULT *);
BOOL (*onWM_NOTIFY)(uiControl *, WPARAM, LPARAM, LRESULT *);
void (*onWM_DESTROY)(uiControl *);
uintptr_t parent;
@ -65,7 +65,7 @@ static LRESULT CALLBACK singleSubclassProc(HWND hwnd, UINT uMsg, WPARAM wParam,
switch (uMsg) {
case msgCOMMAND:
if ((*(s->onWM_COMMAND))(c, wParam, lParam, &lResult) != FALSE)
if ((*(s->onWM_COMMAND))(c, HIWORD(wParam), &lResult) != FALSE)
return lResult;
break;
case msgNOTIFY:

2
ui_windows.h
View File

@ -23,7 +23,7 @@ struct uiWindowsNewControlParams {
// 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, WPARAM wParam, LPARAM lParam, LRESULT *lResult);
BOOL (*onWM_COMMAND)(uiControl *c, WORD code, LRESULT *lResult);
BOOL (*onWM_NOTIFY)(uiControl *c, WPARAM wParam, LPARAM lParam, LRESULT *lResult);
// This is called in WM_DESTROY.
void (*onWM_DESTROY)(uiControl *c);