diff --git a/redo/windows/parent.c b/redo/windows/parent.c
index aa107d53..df91bd72 100644
--- a/redo/windows/parent.c
+++ b/redo/windows/parent.c
@@ -105,9 +105,6 @@ void paintContainerBackground(HWND hwnd, HDC dc, RECT *paintRect)
 // why have this to begin with? http://blogs.msdn.com/b/oldnewthing/archive/2010/03/16/9979112.aspx
 BOOL handleParentMessages(HWND hwnd, UINT uMsg, WPARAM wParam, LPARAM lParam, LRESULT *lResult)
 {
-	HWND control;
-	NMHDR *nm = (NMHDR *) lParam;
-
 	switch (uMsg) {
 	case WM_COMMAND:
 		return runWM_COMMAND(wParam, lParam, lResult);
diff --git a/redo/windows/separator.c b/redo/windows/separator.c
index 0282e6ee..26ca71f6 100644
--- a/redo/windows/separator.c
+++ b/redo/windows/separator.c
@@ -20,7 +20,7 @@ static uintptr_t separatorHandle(uiControl *c)
 {
 	struct separator *s = (struct separator *) c;
 
-	return s->hwnd;
+	return (uintptr_t) (s->hwnd);
 }
 
 static void separatorPreferredSize(uiControl *c, uiSizing *d, intmax_t *width, intmax_t *height)
diff --git a/redo/windows/window.c b/redo/windows/window.c
index 808c57b8..ae0cd6b1 100644
--- a/redo/windows/window.c
+++ b/redo/windows/window.c
@@ -24,7 +24,6 @@ static LRESULT CALLBACK windowWndProc(HWND hwnd, UINT uMsg, WPARAM wParam, LPARA
 	struct window *w;
 	CREATESTRUCTW *cs = (CREATESTRUCTW *) lParam;
 	WINDOWPOS *wp = (WINDOWPOS *) lParam;
-	RECT r;
 	LRESULT lResult;
 
 	w = (struct window *) GetWindowLongPtrW(hwnd, GWLP_USERDATA);
@@ -193,9 +192,9 @@ static void windowSetMargined(uiWindow *ww, int margined)
 // from https://msdn.microsoft.com/en-us/library/windows/desktop/dn742486.aspx#sizingandspacing
 #define windowMargin 7
 
-static void windowResizeChild(uiControl *c)
+static void windowResizeChild(uiWindow *ww)
 {
-	struct window *w = (struct window *) c;
+	struct window *w = (struct window *) ww;
 	RECT r;
 	uiSizing *d;