oto audio player paused and resumed correctly on state change

this was spotted because running the emulator on the command line would
cause the debugger to run a paused state, but the audio player was busy
looping and pegging a CPU core

Author: JetSetIlly

gui/ebiten/audio.go

@@ -22,6 +22,11 @@ func (a *audioPlayer) setState(state gui.State) {
 	a.crit.Lock()
 	defer a.crit.Unlock()
 	a.state = state
+	if state == gui.StatePaused {
+		a.p.Pause()
+	} else {
+		a.p.Play()
+	}
 }
 
 func (a *audioPlayer) Read(buf []uint8) (int, error) {

gui/ebiten/gui.go

@@ -152,7 +152,11 @@ func (eg *guiEbiten) Update() error {
 				case <-ready:
 					eg.audio.r = s.Read
 					eg.audio.p = ctx.NewPlayer(&eg.audio)
-					eg.audio.p.Play()
+					if eg.state == gui.StatePaused {
+						eg.audio.p.Pause()
+					} else {
+						eg.audio.p.Play()
+					}
 				case <-eg.endGui:
 					return ebiten.Termination
 				}
@@ -305,7 +309,7 @@ func Launch(endGui <-chan bool, g *gui.ChannelsGUI, update func() error) error {
 		endGui:      endGui,
 		state:       gui.StateRunning,
 		audio: audioPlayer{
-			state: gui.StateRunning,
+			state: gui.StatePaused,
 		},
 		lastFrame: time.Now(),
 		update:    update,
@@ -317,7 +321,6 @@ func Launch(endGui <-chan bool, g *gui.ChannelsGUI, update func() error) error {
 	for !done {
 		select {
 		case eg.state = <-g.State:
-			eg.audio.setState(eg.state)
 			done = true
 		case <-endGui:
 			return nil