Factor out begin_monitoring_file_size()

[srec] / srec.py

diff --git a/srec.py b/srec.py
index e9054838af18172c5e785162eab43c62dd7c1934..64f2d07e89d4c98c1e93d06892cdc89a035da929 100644 (file)
--- a/srec.py
+++ b/srec.py
@@ -4,6 +4,8 @@
 # under the terms of the GNU General Public License as published by the
 # Free Software Foundation, version 3.
 
 # under the terms of the GNU General Public License as published by the
 # Free Software Foundation, version 3.
 

+from __future__ import annotations
+

 from dataclasses import dataclass
 from datetime import datetime
 import os
 from dataclasses import dataclass
 from datetime import datetime
 import os


@@ -20,9 +22,14 @@ from gi.repository import GLib  # nopep8 pylint: disable=wrong-import-position
 
 
 @dataclass
 
 
 @dataclass

-class Recording:
+class Stream:

     process: subprocess.Popen[bytes]
 
     process: subprocess.Popen[bytes]
 

+    @staticmethod
+    def start(command: list[str]) -> Stream:
+        # pylint: disable=consider-using-with
+        return Stream(process=subprocess.Popen(command, stdin=subprocess.PIPE))
+

     def stop(self) -> None:
         stdin = self.process.stdin
         assert stdin is not None
     def stop(self) -> None:
         stdin = self.process.stdin
         assert stdin is not None


@@ -34,7 +41,7 @@ class Recording:
         self.process.wait()
 
 
         self.process.wait()
 
 

-recording: Recording | None = None
+recording: Stream | None = None

 
 
 def make_filename() -> str:
 
 
 def make_filename() -> str:


@@ -67,13 +74,7 @@ def summarize_size(n: int) -> str:
     return str(n)
 
 
     return str(n)
 
 

-def on_start_recording(_: Gtk.Button, stack: Gtk.Stack) -> None:
-    global recording  # pylint: disable=global-statement
-    assert recording is None
-
-    filename = make_filename()
-    size_display = find_size_display(stack)
-
+def begin_monitoring_file_size(size_display: Gtk.Label, filename: str) -> None:

     def update_size_display() -> Any:
         try:
             size = summarize_size(os.stat(filename).st_size)
     def update_size_display() -> Any:
         try:
             size = summarize_size(os.stat(filename).st_size)


@@ -83,11 +84,17 @@ def on_start_recording(_: Gtk.Button, stack: Gtk.Stack) -> None:
         return GLib.SOURCE_REMOVE if recording is None else GLib.SOURCE_CONTINUE
     GLib.timeout_add_seconds(1, update_size_display)
 
         return GLib.SOURCE_REMOVE if recording is None else GLib.SOURCE_CONTINUE
     GLib.timeout_add_seconds(1, update_size_display)
 

-    command = (['ffmpeg', '-framerate', '25'] + video_source(stack) +
-               ['-f', 'pulse', '-ac', '2', '-i', 'default', filename])
-    # pylint: disable=consider-using-with
-    recording = Recording(
-        process=subprocess.Popen(command, stdin=subprocess.PIPE))
+
+def on_start_recording(_: Gtk.Button, stack: Gtk.Stack) -> None:
+    global recording  # pylint: disable=global-statement
+    assert recording is None
+
+    filename = make_filename()
+    begin_monitoring_file_size(find_size_display(stack), filename)
+
+    recording = Stream.start(
+        ['ffmpeg', '-framerate', '25'] + video_source(stack) +
+        ['-f', 'pulse', '-ac', '2', '-i', 'default', filename])

     stack.set_visible_child_name("recording")
 
 
     stack.set_visible_child_name("recording")