From 60db80295108d84d444ea5df7f512bf4f92ef598 Mon Sep 17 00:00:00 2001 From: Egon Elbre Date: Sat, 27 Feb 2021 23:23:03 +0200 Subject: [PATCH] cmd/gogio: fix test work group handling Signed-off-by: Egon Elbre --- cmd/gogio/windows_test.go | 2 +- cmd/gogio/x11_test.go | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/cmd/gogio/windows_test.go b/cmd/gogio/windows_test.go index 86fa48e1..02e8df47 100644 --- a/cmd/gogio/windows_test.go +++ b/cmd/gogio/windows_test.go @@ -56,7 +56,7 @@ func (d *WineTestDriver) Start(path string) { d.Cleanup(wg.Wait) // Add 50x50px to the display dimensions, as discussed earlier. - d.startServer(wg, d.width+50, d.height+50) + d.startServer(&wg, d.width+50, d.height+50) // Then, start our program via Wine on the X server above. { diff --git a/cmd/gogio/x11_test.go b/cmd/gogio/x11_test.go index 4549b419..9bb3174b 100644 --- a/cmd/gogio/x11_test.go +++ b/cmd/gogio/x11_test.go @@ -39,7 +39,7 @@ func (d *X11TestDriver) Start(path string) { var wg sync.WaitGroup d.Cleanup(wg.Wait) - d.startServer(wg, d.width, d.height) + d.startServer(&wg, d.width, d.height) // Then, start our program on the X server above. { @@ -69,7 +69,7 @@ func (d *X11TestDriver) Start(path string) { d.waitForFrame() } -func (d *X11TestDriver) startServer(wg sync.WaitGroup, width, height int) { +func (d *X11TestDriver) startServer(wg *sync.WaitGroup, width, height int) { // Pick a random display number between 1 and 100,000. Most machines // will only be using :0, so there's only a 0.001% chance of two // concurrent test runs to run into a conflict.