Factor out Recording.stop()

[srec] / srec.py

diff --git a/srec.py b/srec.py
index 35efcaf19e451f101eb77a8a024c241c41c13a7d..50f311c14d1dd6d2ff4e4fac08a31fb9af03d645 100644 (file)
--- a/srec.py
+++ b/srec.py
@@ -22,6 +22,13 @@ from gi.repository import GLib  # nopep8 pylint: disable=wrong-import-position
 class Recording:
     process: subprocess.Popen[bytes]
 
 class Recording:
     process: subprocess.Popen[bytes]
 

+    def stop(self) -> None:
+        stdin = self.process.stdin
+        assert stdin is not None
+        stdin.write(b'q')
+        stdin.flush()
+        self.process.wait()
+

 
 recording: Recording | None = None
 
 
 recording: Recording | None = None
 


@@ -83,11 +90,7 @@ def on_start_recording(_: Gtk.Button, stack: Gtk.Stack) -> None:
 def on_stop_recording(_: Gtk.Button, stack: Gtk.Stack) -> None:
     global recording  # pylint: disable=global-statement
     assert recording is not None
 def on_stop_recording(_: Gtk.Button, stack: Gtk.Stack) -> None:
     global recording  # pylint: disable=global-statement
     assert recording is not None

-    stdin = recording.process.stdin
-    assert stdin is not None
-    stdin.write(b'q')
-    stdin.flush()
-    recording.process.wait()
+    recording.stop()

     recording = None
     stack.set_visible_child_name("not_recording")
 
     recording = None
     stack.set_visible_child_name("not_recording")