vino/rocket
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

editor: fix dialog return

Browse Source 
return the status from EndDialog instead of a constant TRUE. This
is mostly about keeping the code "honest", it doesn't change
anything in reality.

Also fix up some code-style.
This commit is contained in:
Erik Faye-Lund 2011-04-05 10:00:36 +02:00
parent 6d56c23e00
commit d8081fd63c
1 changed files with 6 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

21
editor/editor.cpp
View File

@ -88,14 +88,9 @@ static LRESULT CALLBACK setRowsDialogProc(HWND hDlg, UINT message, WPARAM wParam
SendMessage(GetParent(hDlg), WM_SETROWS, 0, result); SendMessage(GetParent(hDlg), WM_SETROWS, 0, result);
/* end dialog */ /* end dialog */
EndDialog(hDlg, LOWORD(wParam)); return EndDialog(hDlg, LOWORD(wParam));
return TRUE; } else if(LOWORD(wParam) == IDCANCEL)
} return EndDialog( hDlg, LOWORD(wParam));
else if(LOWORD(wParam)== IDCANCEL)
{
EndDialog( hDlg, LOWORD(wParam));
return TRUE;
}
break; break;
case WM_CLOSE: case WM_CLOSE:
@ -127,13 +122,9 @@ static LRESULT CALLBACK biasSelectionDialogProc(HWND hDlg, UINT message, WPARAM
SendMessage(GetParent(hDlg), WM_BIASSELECTION, 0, LPARAM(bias)); SendMessage(GetParent(hDlg), WM_BIASSELECTION, 0, LPARAM(bias));
/* end dialog */ /* end dialog */
EndDialog(hDlg, LOWORD(wParam)); return EndDialog(hDlg, LOWORD(wParam));
return TRUE; } else if(LOWORD(wParam) == IDCANCEL)
} return EndDialog( hDlg, LOWORD(wParam));
else if(LOWORD(wParam)== IDCANCEL)
{
EndDialog( hDlg, LOWORD(wParam));
}
break; break;
case WM_CLOSE: case WM_CLOSE: