Turns out we don't need to track filename in Recording

[srec] / srec.py

diff --git a/srec.py b/srec.py
index 4a9d5b95fdbb96ee39ed1cd207d10509aab60ed4..35efcaf19e451f101eb77a8a024c241c41c13a7d 100644 (file)
--- a/srec.py
+++ b/srec.py
@@ -8,16 +8,18 @@ from dataclasses import dataclass
 from datetime import datetime
 import os
 import subprocess
-from typing import Callable
+from typing import Any, Callable
 
 import gi
 gi.require_version("Gtk", "4.0")
-from gi.repository import Gtk  # nopep8 pylint: disable=wrong-import-position
+gi.require_version("GLib", "2.0")
+
+from gi.repository import Gtk   # nopep8 pylint: disable=wrong-import-position
+from gi.repository import GLib  # nopep8 pylint: disable=wrong-import-position
 
 
 @dataclass
 class Recording:
-    filename: str
     process: subprocess.Popen[bytes]
 
 
@@ -39,15 +41,41 @@ def video_source(stack: Gtk.Stack) -> list[str]:
     return ['-f', 'v4l2', '-i', '/dev/video0']
 
 
+def find_size_display(stack: Gtk.Stack) -> Gtk.Label:
+    return stack.get_child_by_name(
+        'recording').get_first_child().get_next_sibling()
+
+
+def summarize_size(n: int) -> str:
+    if n > 100_000_000:
+        m = int(n / (1024 * 1024))
+        return f'{m}M'
+    if n > 100_000:
+        k = int(n / 1024)
+        return f'{k}K'
+    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 update_size_display() -> Any:
+        try:
+            size = summarize_size(os.stat(filename).st_size)
+        except FileNotFoundError:
+            size = '--'
+        size_display.set_label(f'<big>{size}</big>')
+        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(
-        filename=filename,
         process=subprocess.Popen(command, stdin=subprocess.PIPE))
     stack.set_visible_child_name("recording")
 
@@ -76,7 +104,7 @@ def make_button(label: str, action: Callable[[
 
 
 def make_share_control() -> Gtk.CheckButton:
-    can_share = os.path.exists('/sys/module/v4l2looback')
+    can_share = os.path.exists('/sys/module/v4l2loopback')
     control = Gtk.CheckButton(
         label='Share Webcam', sensitive=can_share, active=can_share)
     control.set_margin_start(20)
@@ -102,6 +130,7 @@ def on_activate(app: Gtk.Application) -> None:
     r_box = Gtk.Box()
     r_box.set_orientation(Gtk.Orientation.VERTICAL)
     r_box.append(make_button("Stop Recording", on_stop_recording, stack))
+    r_box.append(Gtk.Label(use_markup=True, justify=Gtk.Justification.CENTER))
     stack.add_named(r_box, "recording")
 
     win.set_child(stack)