Big change: with the current event model, it's safe to call Area.RepaintAll() (and the future Area.Repaint()) from within AreaHandler.Key() and AreaHandler.Mouse(); get rid of the bool returns from each. The future Area.Repaint() is important; the bool returns repainted the whole Area, which may not be optimal.

1 files changed, 2 insertions, 8 deletions

diff --git a/redo/area_darwin.go b/redo/area_darwin.go
index 471ad58..1d715ac 100644
--- a/redo/area_darwin.go
+++ b/redo/area_darwin.go
@@ -126,10 +126,7 @@ func areaMouseEvent(self C.id, e C.id, click bool, up bool, data unsafe.Pointer)
 		}
 		held >>= 1
 	}
-	repaint := a.handler.Mouse(me)
-	if repaint {
-		C.display(self)
-	}
+	a.handler.Mouse(me)
 }
 
 //export areaView_mouseMoved_mouseDragged
@@ -152,10 +149,7 @@ func areaView_mouseUp(self C.id, e C.id, data unsafe.Pointer) {
 
 func sendKeyEvent(self C.id, ke KeyEvent, data unsafe.Pointer) {
 	a := (*area)(data)
-	repaint := a.handler.Key(ke)
-	if repaint {
-		C.display(self)
-	}
+	a.handler.Key(ke)
 }
 
 func areaKeyEvent(self C.id, e C.id, up bool, data unsafe.Pointer) {