Added updateAll() and resolved WM_SETFONT to begin the final reasoning-through of update().

1 files changed, 4 insertions, 9 deletions

diff --git a/wintable/api.h b/wintable/api.h
index 3d36379..f6895e5 100644
--- a/wintable/api.h
+++ b/wintable/api.h
@@ -15,22 +15,17 @@ static void addColumn(struct table *t, WPARAM wParam, LPARAM lParam)
 HANDLER(apiHandlers)
 {
 	intptr_t *rcp;
-	BOOL tfbool;
 
 	switch (uMsg) {
 	case WM_SETFONT:
 		// don't free the old font; see http://blogs.msdn.com/b/oldnewthing/archive/2008/09/12/8945692.aspx
 		t->font = (HFONT) wParam;
 		SendMessageW(t->header, WM_SETFONT, wParam, lParam);
-		// let's ensure the values are strictly TRUE and FALSE just to be safe
-		tfbool = FALSE;
+		// if we redraw, we have to redraw ALL of it; after all, the font changed!
 		if (LOWORD(lParam) != FALSE)
-			tfbool = TRUE;
-		update(t, tfbool);
-		// TODO is this needed?
-		if (tfbool != FALSE)
-			// TODO check error
-			InvalidateRect(t->hwnd, NULL, TRUE);
+			updateAll(t);			// DONE
+		else
+			update(t, FALSE);		// DONE
 		*lResult = 0;
 		return TRUE;
 	case WM_GETFONT: