urwid 3.0.4__tar.gz → 4.0.0__tar.gz

{urwid-3.0.4 → urwid-4.0.0}/.github/workflows/documentation.yml

@@ -44,7 +44,7 @@ jobs:
         run: pip install -e .
       - name: Build documentation
         run: sphinx-build docs build/documentation
-      - uses: actions/upload-artifact@v5
+      - uses: actions/upload-artifact@v7
         with:
           path: build/documentation
           name: documentation

{urwid-3.0.4 → urwid-4.0.0}/.github/workflows/pythonpackage.yml

@@ -172,13 +172,13 @@ jobs:
       - name: Build dist
         run: python -m build
 
-      - uses: actions/upload-artifact@v5
+      - uses: actions/upload-artifact@v7
         with:
           path: dist/*.tar.gz
           name: built-sdist
           retention-days: 3
 
-      - uses: actions/upload-artifact@v5
+      - uses: actions/upload-artifact@v7
         with:
           path: dist/*.whl
           name: built-bdist
@@ -199,7 +199,7 @@ jobs:
         run: |
           python -m pip install --upgrade pip
           pip install --upgrade twine
-      - uses: actions/download-artifact@v6
+      - uses: actions/download-artifact@v8
         with:
           # unpacks default artifact into dist/
           # if `name: wheels` is omitted, the action will create extra parent dir
@@ -224,7 +224,7 @@ jobs:
     permissions:
       id-token: write  # IMPORTANT: this permission is mandatory for trusted publishing
     steps:
-      - uses: actions/download-artifact@v6
+      - uses: actions/download-artifact@v8
         with:
           pattern: built-*
           merge-multiple: true

{urwid-3.0.4 → urwid-4.0.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: urwid
-Version: 3.0.4
+Version: 4.0.0
 Summary: A full-featured console (xterm et al.) user interface library
 Author-email: Ian Ward <ian@excess.org>
 License-Expression: LGPL-2.1-only
@@ -33,7 +33,7 @@ Classifier: Programming Language :: Python :: Implementation :: PyPy
 Requires-Python: >=3.9.0
 Description-Content-Type: text/x-rst
 License-File: COPYING
-Requires-Dist: wcwidth
+Requires-Dist: wcwidth>=0.4
 Provides-Extra: curses
 Requires-Dist: windows-curses; sys_platform == "win32" and extra == "curses"
 Provides-Extra: glib

{urwid-3.0.4 → urwid-4.0.0}/docs/changelog.rst

@@ -2,6 +2,26 @@
 Changelog
 ---------
 
+Urwid 3.0.5
+===========
+
+2026-02-02
+
+Deprecations ⚡
+++++++++++++++
+* Un-deprecate `user_arg` for signals by @penguinolog in https://github.com/urwid/urwid/pull/1092
+Bug fixes 🕷
+++++++++++++
+* Fix terminal scroll TypeError by @penguinolog in https://github.com/urwid/urwid/pull/1098
+* Bugfix some kinds of emojis with grapheme boundry support by @jquast in https://github.com/urwid/urwid/pull/1100
+* Fix: Text.pack line splitting not match Text.render by @penguinolog in https://github.com/urwid/urwid/pull/1099
+
+New Contributors
+++++++++++++++++
+* @jquast made their first contribution in https://github.com/urwid/urwid/pull/1100
+
+**Full Changelog**: https://github.com/urwid/urwid/compare/3.0.4...3.0.5
+
 Urwid 3.0.4
 ===========
 
@@ -14,7 +34,8 @@ Other Changes
 +++++++++++++
 * Refactoring: use official python way for lazy imports by @penguinolog in https://github.com/urwid/urwid/pull/1077
 
-## New Contributors
+New Contributors
+++++++++++++++++
 * @sawdmn made their first contribution in https://github.com/urwid/urwid/pull/1079
 
 **Full Changelog**: https://github.com/urwid/urwid/compare/3.0.3...3.0.4

{urwid-3.0.4 → urwid-4.0.0}/docs/examples/real_edit.py

@@ -48,7 +48,8 @@ class LineWalker(urwid.ListWalker):
         self.focus = 0
 
     def __del__(self) -> None:
-        self.file.close()
+        if self.file is not None:
+            self.file.close()
 
     def get_focus(self):
         return self._get_at_pos(self.focus)

{urwid-3.0.4 → urwid-4.0.0}/examples/edit.py

@@ -48,7 +48,8 @@ class LineWalker(urwid.ListWalker):
         self.focus = 0
 
     def __del__(self) -> None:
-        self.file.close()
+        if self.file is not None:
+            self.file.close()
 
     def get_focus(self):
         return self._get_at_pos(self.focus)

{urwid-3.0.4 → urwid-4.0.0}/pyproject.toml

@@ -30,14 +30,9 @@ platforms=["unix-like"]
 zip-safe = false
 
 [tool.setuptools.packages.find]
-exclude = [
-  "doc",
-  "docs",
-  "examples",
-  "test",
-  "tests",
-  "bin",
-  ".*"
+include = [
+  "urwid",
+  "urwid.*",
 ]
 namespaces = false
 

urwid-4.0.0/requirements.txt

@@ -0,0 +1 @@
+wcwidth>=0.4

urwid-4.0.0/ruff-requirements.txt

@@ -0,0 +1 @@
+ruff==0.15.8

{urwid-3.0.4 → urwid-4.0.0}/tests/test_event_loops.py

@@ -247,6 +247,7 @@ class AsyncioEventLoopTest(unittest.TestCase, EventLoopTestMixin):
         self.assertRaises(ZeroDivisionError, evl.run)
 
 
+@unittest.skip("Unstable test: race conditions happens")
 @unittest.skipUnless(GLIB_AVAILABLE, "GLIB unavailable")
 class GLibEventLoopTest(unittest.TestCase, EventLoopTestMixin):
     def setUp(self):

urwid-4.0.0/tests/test_grapheme_clusters.py

@@ -0,0 +1,210 @@
+from __future__ import annotations
+
+import unittest
+
+from urwid import str_util
+
+# 🇨🇦 regional indicator Canada flag
+FLAG_CA = "\U0001F1E8\U0001F1E6"
+# 👨‍👩‍👧 family (man + ZWJ + woman + ZWJ + girl)
+FAMILY = "\U0001F468\u200D\U0001F469\u200D\U0001F467"
+# 👩🏻‍💻 woman technologist with light skin tone
+TECHNOLOGIST = "\U0001F469\U0001F3FB\u200D\U0001F4BB"
+# 👋🏻 waving hand with light skin tone
+WAVE_SKIN = "\U0001F44B\U0001F3FB"
+# café with combining acute accent on e
+CAFE = "cafe\u0301"
+# ❤️ red heart with VS16 (emoji presentation)
+HEART_VS16 = "\u2764\ufe0f"
+# ❤ red heart (text presentation)
+HEART = "\u2764"
+# 中文 Chinese characters
+CJK = "\u4e2d\u6587"
+
+
+class CalcWidthGraphemeTest(unittest.TestCase):
+    def wtest(self, text, expected):
+        result = str_util.calc_width(text, 0, len(text))
+        assert result == expected, f"got:{result!r} expected:{expected!r}"
+
+    def test_basic(self):
+        self.wtest("hello", 5)
+        self.wtest("", 0)
+        self.wtest(CJK, 4)
+
+    def test_flags(self):
+        self.wtest(FLAG_CA, 2)
+        self.wtest(FLAG_CA + FLAG_CA, 4)
+
+    def test_zwj_sequences(self):
+        self.wtest(FAMILY, 2)
+        self.wtest(TECHNOLOGIST, 2)
+
+    def test_skin_tones(self):
+        self.wtest(WAVE_SKIN, 2)
+
+    def test_combining(self):
+        self.wtest(CAFE, 4)
+        self.wtest("e\u0301", 1)
+
+    def test_variation_selectors(self):
+        self.wtest(HEART_VS16, 2)
+        self.wtest(HEART, 1)
+
+
+class CalcTextPosGraphemeTest(unittest.TestCase):
+    def ptest(self, text, pref_col, expected_pos):
+        pos, _ = str_util.calc_text_pos(text, 0, len(text), pref_col)
+        assert pos == expected_pos, f"got:{pos!r} expected:{expected_pos!r}"
+
+    def test_basic(self):
+        self.ptest("hello", 3, 3)
+
+    def test_flag_boundaries(self):
+        text = f"A{FLAG_CA}B"
+        self.ptest(text, 1, 1)
+        self.ptest(text, 2, 1)
+        self.ptest(text, 3, 3)
+
+    def test_zwj_boundaries(self):
+        text = f"A{FAMILY}B"
+        self.ptest(text, 2, 1)
+        self.ptest(text, 3, 6)
+
+
+class MoveNextCharGraphemeTest(unittest.TestCase):
+    def test_move_through_family_emoji(self):
+        text = f"A{FAMILY}B"
+        pos = str_util.move_next_char(text, 1, len(text))
+        assert pos == 6
+
+    def test_move_through_flag(self):
+        text = f"A{FLAG_CA}B"
+        pos = str_util.move_next_char(text, 1, len(text))
+        assert pos == 3
+
+    def test_move_through_combining(self):
+        pos = str_util.move_next_char(CAFE, 3, len(CAFE))
+        assert pos == 5
+
+    def test_move_through_skin_tone(self):
+        text = f"A{WAVE_SKIN}B"
+        pos = str_util.move_next_char(text, 1, len(text))
+        assert pos == 3
+
+    def test_basic_ascii(self):
+        pos = str_util.move_next_char("hello", 0, 5)
+        assert pos == 1
+
+
+class MovePrevCharGraphemeTest(unittest.TestCase):
+    def test_move_back_through_family_emoji(self):
+        text = f"A{FAMILY}B"
+        pos = str_util.move_prev_char(text, 0, 6)
+        assert pos == 1
+
+    def test_move_back_through_flag(self):
+        text = f"A{FLAG_CA}B"
+        pos = str_util.move_prev_char(text, 0, 3)
+        assert pos == 1
+
+    def test_move_back_through_combining(self):
+        pos = str_util.move_prev_char(CAFE, 0, 5)
+        assert pos == 3
+
+    def test_move_back_through_skin_tone(self):
+        text = f"A{WAVE_SKIN}B"
+        pos = str_util.move_prev_char(text, 0, 3)
+        assert pos == 1
+
+    def test_basic_ascii(self):
+        pos = str_util.move_prev_char("hello", 0, 5)
+        assert pos == 4
+
+
+class IsWideCharGraphemeTest(unittest.TestCase):
+    def test_family_emoji_is_wide(self):
+        assert str_util.is_wide_char(FAMILY, 0)
+
+    def test_flag_is_wide(self):
+        assert str_util.is_wide_char(FLAG_CA, 0)
+
+    def test_cjk_is_wide(self):
+        assert str_util.is_wide_char(CJK, 0)
+
+    def test_ascii_not_wide(self):
+        assert not str_util.is_wide_char("hello", 0)
+
+    def test_skin_tone_is_wide(self):
+        assert str_util.is_wide_char(WAVE_SKIN, 0)
+
+
+# UTF-8 bytes test constants
+FLAG_CA_BYTES = FLAG_CA.encode("utf-8")  # 8 bytes
+FAMILY_BYTES = FAMILY.encode("utf-8")  # 18 bytes
+
+
+class Utf8BytesGraphemeTest(unittest.TestCase):
+    """Tests for grapheme clustering with UTF-8 bytes."""
+
+    def setUp(self):
+        self._orig_encoding = str_util.get_byte_encoding()
+        str_util.set_byte_encoding("utf8")
+
+    def tearDown(self):
+        str_util.set_byte_encoding(self._orig_encoding)
+
+    def test_calc_width_flag_bytes(self):
+        result = str_util.calc_width(FLAG_CA_BYTES, 0, len(FLAG_CA_BYTES))
+        assert result == 2
+
+    def test_calc_width_family_emoji_bytes(self):
+        result = str_util.calc_width(FAMILY_BYTES, 0, len(FAMILY_BYTES))
+        assert result == 2
+
+    def test_move_next_through_family_emoji_bytes(self):
+        # After 'A' (1 byte), should skip entire family emoji
+        text = f"A{FAMILY}B".encode("utf-8")
+        pos = str_util.move_next_char(text, 1, len(text))
+        assert pos == 1 + len(FAMILY_BYTES)
+
+    def test_move_next_through_flag_bytes(self):
+        # After 'A' (1 byte), should skip entire flag
+        text = f"A{FLAG_CA}B".encode("utf-8")
+        pos = str_util.move_next_char(text, 1, len(text))
+        assert pos == 1 + len(FLAG_CA_BYTES)
+
+    def test_move_prev_through_family_emoji_bytes(self):
+        # From position after family emoji, should go back to after 'A'
+        text = f"A{FAMILY}B".encode("utf-8")
+        end_pos = 1 + len(FAMILY_BYTES)
+        pos = str_util.move_prev_char(text, 0, end_pos)
+        assert pos == 1
+
+    def test_move_prev_through_flag_bytes(self):
+        # From position after flag, should go back to after 'A'
+        text = f"A{FLAG_CA}B".encode("utf-8")
+        end_pos = 1 + len(FLAG_CA_BYTES)
+        pos = str_util.move_prev_char(text, 0, end_pos)
+        assert pos == 1
+
+    def test_is_wide_family_emoji_bytes(self):
+        assert str_util.is_wide_char(FAMILY_BYTES, 0)
+
+    def test_is_wide_flag_bytes(self):
+        assert str_util.is_wide_char(FLAG_CA_BYTES, 0)
+
+    def test_calc_text_pos_flag_boundaries_bytes(self):
+        text = f"A{FLAG_CA}B".encode("utf-8")
+        # Column 1 should be at position 1 (after 'A')
+        pos, col = str_util.calc_text_pos(text, 0, len(text), 1)
+        assert pos == 1
+        assert col == 1
+        # Column 2 should still be at position 1 (inside flag grapheme)
+        pos, col = str_util.calc_text_pos(text, 0, len(text), 2)
+        assert pos == 1
+        assert col == 1
+        # Column 3 should be at position after flag
+        pos, col = str_util.calc_text_pos(text, 0, len(text), 3)
+        assert pos == 1 + len(FLAG_CA_BYTES)
+        assert col == 3

{urwid-3.0.4 → urwid-4.0.0}/tests/test_scrollable.py

@@ -38,7 +38,8 @@ class TestScrollable(unittest.TestCase):
         widget = urwid.Scrollable(long_content)
         self.assertEqual(frozenset((urwid.BOX,)), widget.sizing())
 
-        cropped_content_canvas = urwid.CompositeCanvas(long_content.render((reduced_size[0],)))
+        long_content_canvas = long_content.render((reduced_size[0],))
+        cropped_content_canvas = urwid.CompositeCanvas(long_content_canvas)
         cropped_content_canvas.trim_end(content_size[1] - reduced_size[1])
         top_decoded = cropped_content_canvas.decoded_text
 

{urwid-3.0.4 → urwid-4.0.0}/tests/test_text_layout.py

@@ -169,15 +169,27 @@ class CalcTranslateTest(unittest.TestCase):
     def check(self, text, mode, width, result_left, result_center, result_right) -> None:
         with self.subTest("left"):
             result = urwid.default_layout.layout(text, width, "left", mode)
-            self.assertEqual(result_left, result)
+            self.assertEqual(
+                result_left,
+                result,
+                f"For {text=} {width=} {mode=} got {result!r} instead of {result_left!r}",
+            )
 
         with self.subTest("center"):
             result = urwid.default_layout.layout(text, width, "center", mode)
-            self.assertEqual(result_center, result)
+            self.assertEqual(
+                result_center,
+                result,
+                f"For {text=} {width=} {mode=} got {result!r} instead of {result_center!r}",
+            )
 
         with self.subTest("right"):
             result = urwid.default_layout.layout(text, width, "right", mode)
-            self.assertEqual(result_right, result)
+            self.assertEqual(
+                result_right,
+                result,
+                f"For {text=} {width=} {mode=} got {result!r} instead of {result_right!r}",
+            )
 
     def test_calc_translate_char(self):
         self.check(

{urwid-3.0.4 → urwid-4.0.0}/urwid/__init__.py

@@ -255,15 +255,7 @@ VERSION = __version_tuple__
 __locals: dict[str, typing.Any] = locals()  # use mutable access for pure lazy loading
 
 # Backward compatible lazy load with deprecation warnings
-_moved_warn: dict[str, str] = {
-    "lcd_display": "urwid.display.lcd",
-    "html_fragment": "urwid.display.html_fragment",
-    "web_display": "urwid.display.web",
-    "monitored_list": "urwid.widget.monitored_list",
-    "listbox": "urwid.widget.listbox",
-    "treetools": "urwid.widget.treetools",
-}
-# Backward compatible lazy load without any warnings
+_moved_warn: dict[str, str] = {}
 # Before DeprecationWarning need to start PendingDeprecationWarning process.
 
 

{urwid-3.0.4 → urwid-4.0.0}/urwid/display/_posix_raw_display.py

@@ -315,7 +315,10 @@ class Screen(_raw_display_base.Screen):
             selector.register(fd, selectors.EVENT_READ)
             input_ready = selector.select(0)
             while input_ready:
-                chars.extend(os.read(fd, 1024))
+                chunk = os.read(fd, 1024)
+                if len(chunk) == 0:
+                    raise RuntimeError("stdin has been closed")
+                chars.extend(chunk)
                 input_ready = selector.select(0)
 
             return chars

{urwid-3.0.4 → urwid-4.0.0}/urwid/display/common.py

@@ -1018,7 +1018,8 @@ class BaseScreen(abc.ABC, metaclass=signals.MetaSignals):
         Deprecated in favor of calling `start` as a context manager.
         """
         warnings.warn(
-            "run_wrapper is deprecated in favor of calling `start` as a context manager.",
+            "run_wrapper is deprecated in favor of calling `start` as a context manager."
+            "API will be removed in version 5.0.",
             DeprecationWarning,
             stacklevel=3,
         )

{urwid-3.0.4 → urwid-4.0.0}/urwid/event_loop/main_loop.py

@@ -28,7 +28,6 @@ import os
 import sys
 import time
 import typing
-import warnings
 from contextlib import suppress
 
 from urwid import display, signals
@@ -169,16 +168,6 @@ class MainLoop:
         else:
             self._topmost_widget = self._widget
 
-    def _set_widget(self, widget: Widget) -> None:
-        warnings.warn(
-            f"method `{self.__class__.__name__}._set_widget` is deprecated, "
-            f"please use `{self.__class__.__name__}.widget` property."
-            "API will be removed in version 4.0.",
-            DeprecationWarning,
-            stacklevel=2,
-        )
-        self.widget = widget
-
     @property
     def pop_ups(self) -> bool:
         return self._pop_ups
@@ -191,16 +180,6 @@ class MainLoop:
         else:
             self._topmost_widget = self._widget
 
-    def _set_pop_ups(self, pop_ups: bool) -> None:
-        warnings.warn(
-            f"method `{self.__class__.__name__}._set_pop_ups` is deprecated, "
-            f"please use `{self.__class__.__name__}.pop_ups` property."
-            "API will be removed in version 4.0.",
-            DeprecationWarning,
-            stacklevel=2,
-        )
-        self.pop_ups = pop_ups
-
     def set_alarm_in(self, sec: float, callback: Callable[[Self, _T], typing.Any], user_data: _T = None):
         """
         Schedule an alarm in *sec* seconds that will call *callback* from the

{urwid-3.0.4 → urwid-4.0.0}/urwid/font.py

@@ -24,15 +24,15 @@ import typing
 import warnings
 from pprint import pformat
 
+import wcwidth
+
 from urwid.canvas import CanvasError, TextCanvas
 from urwid.display.escape import SAFE_ASCII_DEC_SPECIAL_RE
-from urwid.util import apply_target_encoding, str_util
+from urwid.util import apply_target_encoding
 
 if typing.TYPE_CHECKING:
     from collections.abc import Iterator, Sequence
 
-    from typing_extensions import Literal
-
 
 def separate_glyphs(gdata: str, height: int) -> tuple[dict[str, tuple[int, list[str]]], bool]:
     """return (dictionary of glyphs, utf8 required)"""
@@ -60,7 +60,7 @@ def separate_glyphs(gdata: str, height: int) -> tuple[dict[str, tuple[int, list[
             character = key_line[key_index]
 
         if key_index < len(key_line) and key_line[key_index] == character:
-            end_col += str_util.get_char_width(character)
+            end_col += wcwidth.width(character, control_codes="ignore")
             key_index += 1
             continue
 
@@ -76,7 +76,7 @@ def separate_glyphs(gdata: str, height: int) -> tuple[dict[str, tuple[int, list[
                 if j >= len(line):
                     fill = end_col - start_col - y
                     break
-                y += str_util.get_char_width(line[j])
+                y += wcwidth.width(line[j], control_codes="ignore")
                 j += 1
             if y + fill != end_col - start_col:
                 raise ValueError(repr((y, fill, end_col)))
@@ -98,7 +98,8 @@ def add_font(name: str, cls: FontRegistry) -> None:
     warnings.warn(
         "`add_font` is deprecated, please set 'name' attribute to the font class,"
         " use metaclass keyword argument 'font_name'"
-        " or use `Font.register(<name>)`",
+        " or use `Font.register(<name>)`"
+        "API will be removed in version 5.0.",
         DeprecationWarning,
         stacklevel=2,
     )
@@ -219,22 +220,6 @@ class Font(metaclass=FontRegistry):
         """Font description."""
         return f"{self.__class__.__name__}():\n  {self.height!r}\n  {pformat(self.data, indent=4)}"
 
-    @staticmethod
-    def _to_text(
-        obj: str,
-        encoding: str = "utf-8",
-        errors: Literal["strict", "ignore", "replace"] = "strict",
-    ) -> str:
-        warnings.warn(
-            "_to_text is deprecated: only text fonts are supported. API will be removed in version 4.0.",
-            DeprecationWarning,
-            stacklevel=3,
-        )
-        if isinstance(obj, str):
-            return obj
-
-        raise TypeError(f"{obj!r} is not str")
-
     def add_glyphs(self, gdata: str) -> None:
         d, utf8_required = separate_glyphs(gdata, self.height)
         self.char.update(d)

{urwid-3.0.4 → urwid-4.0.0}/urwid/signals.py

@@ -23,7 +23,6 @@ from __future__ import annotations
 import abc
 import itertools
 import typing
-import warnings
 import weakref
 
 if typing.TYPE_CHECKING:
@@ -99,9 +98,9 @@ class Signals:
         :type name: signal name
         :param callback: the function to call when that signal is sent
         :type callback: function
-        :param user_arg: deprecated additional argument to callback (appended
-                         after the arguments passed when the signal is
-                         emitted). If None no arguments will be added.
+        :param user_arg: additional argument to callback
+                         (appended  after the arguments passed when the signal is emitted).
+                         If None no arguments will be added.
                          Don't use this argument, use user_args instead.
         :param weak_args: additional arguments passed to the callback
                           (before any arguments passed when the signal
@@ -167,12 +166,6 @@ class Signals:
         handler can also be disconnected by calling
         urwid.disconnect_signal, which doesn't need this key.
         """
-        if user_arg is not None:
-            warnings.warn(
-                "Don't use user_arg argument, use user_args instead.",
-                DeprecationWarning,
-                stacklevel=2,
-            )
 
         sig_cls = obj.__class__
         if name not in self._supported.get(sig_cls, ()):
@@ -195,9 +188,8 @@ class Signals:
         # their callbacks) from existing.
         obj_weak = weakref.ref(obj)
 
-        def weakref_callback(weakref):  # pylint: disable=redefined-outer-name  # bad, but not changing API
-            o = obj_weak()
-            if o:
+        def weakref_callback(_ref: weakref.ReferenceType[typing.Any]) -> None:
+            if o := obj_weak():
                 self.disconnect_by_key(o, name, key)
 
         user_args = self._prepare_user_args(weak_args, user_args, weakref_callback)
@@ -209,7 +201,7 @@ class Signals:
         self,
         weak_args: Iterable[typing.Any] = (),
         user_args: Iterable[typing.Any] = (),
-        callback: Callable[..., typing.Any] | None = None,
+        callback: Callable[[weakref.ReferenceType[typing.Any]], typing.Any] | None = None,
     ) -> tuple[Collection[weakref.ReferenceType], Collection[typing.Any]]:
         # Turn weak_args into weakrefs and prepend them to user_args
         w_args = tuple(weakref.ref(w_arg, callback) for w_arg in weak_args)
@@ -303,14 +295,13 @@ class Signals:
         self,
         callback,
         user_arg: typing.Any,
-        weak_args: Collection[weakref.ReferenceType],
-        user_args: Collection[typing.Any],
+        weak_args: Iterable[weakref.ReferenceType],
+        user_args: Iterable[typing.Any],
         emit_args: Iterable[typing.Any],
     ) -> bool:
         args_to_pass = []
         for w_arg in weak_args:
-            real_arg = w_arg()
-            if real_arg is not None:
+            if (real_arg := w_arg()) is not None:
                 args_to_pass.append(real_arg)
             else:
                 # de-referenced