From ff53b8efce55f5668af61f13b656fdb54dee7755 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?John=20Ankarstr=C3=B6m?= Date: Tue, 19 Jul 2022 17:26:24 +0200 Subject: Check Windows API calls for errors more consistently. Some of the checks are likely redundant, but the Windows API documentation rarely makes it clear WHICH errors may be returned (and under which circumstances) rather than simply WHETHER errors may be returned (under any circumstances, including those that do not apply in the given case). --- c/listview.cpp | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) (limited to 'c/listview.cpp') diff --git a/c/listview.cpp b/c/listview.cpp index 8f5da65..fa99793 100644 --- a/c/listview.cpp +++ b/c/listview.cpp @@ -13,16 +13,15 @@ ListView::ListView(const HWND hWndParent, const HMENU hMenu, const DWORD dwStyle { m_hWndParent = hWndParent; m_bHeader = !(dwStyle & LVS_NOCOLUMNHEADER); - hWnd = CreateWindowEx( + hWnd = throw_nil( WS_EX_CLIENTEDGE, WC_LISTVIEW, TEXT(""), dwStyle|WS_CHILD|WS_VISIBLE|WS_VSCROLL|WS_TABSTOP|LVS_REPORT|LVS_SHOWSELALWAYS, 0, 0, 0, 0, m_hWndParent, hMenu, GetModuleHandle(NULL), this); - if (!hWnd) throw Win32Error(GetLastError()); - if (SetProp(hWnd, TEXT("this"), (HANDLE)this)) + if (throw_nil(hWnd, TEXT("this"), (HANDLE)this)) m_prevProc = (WNDPROC)SetWindowLongPtr(hWnd, GWLP_WNDPROC, (LONG_PTR)::WndProc); -- cgit v1.2.3