Window: Use OptionMixin.set_option()

Author: tony

src/libtmux/window.py

@@ -18,7 +18,7 @@
 from libtmux.pane import Pane
 
 from . import exc
-from .common import PaneDict, WindowOptionDict, handle_option_error
+from .common import OptionMixin, PaneDict, WindowOptionDict, handle_option_error
 from .formats import FORMAT_SEPARATOR
 
 if t.TYPE_CHECKING:
@@ -29,7 +29,7 @@
 
 
 @dataclasses.dataclass()
-class Window(Obj):
+class Window(Obj, OptionMixin):
     """:term:`tmux(1)` :term:`Window` [window_manual]_.
 
     Holds :class:`Pane` objects.
@@ -344,89 +344,6 @@ def set_window_option(
 
         return self.set_option(option=option, value=value)
 
-    def set_option(
-        self,
-        option: str,
-        value: t.Union[int, str],
-        _format: t.Optional[bool] = None,
-        unset: t.Optional[bool] = None,
-        unset_panes: t.Optional[bool] = None,
-        prevent_overwrite: t.Optional[bool] = None,
-        suppress_warnings: t.Optional[bool] = None,
-        append: t.Optional[bool] = None,
-        g: t.Optional[bool] = None,
-        scope: t.Optional[OptionScope] = None,
-    ) -> "Window":
-        """Set option for tmux window.
-
-        Wraps ``$ tmux set-option <option> <value>``.
-
-        Parameters
-        ----------
-        option : str
-            option to set, e.g. 'aggressive-resize'
-        value : str
-            window option value. True/False will turn in 'on' and 'off',
-            also accepts string of 'on' or 'off' directly.
-
-        Raises
-        ------
-        :exc:`exc.OptionError`, :exc:`exc.UnknownOption`,
-        :exc:`exc.InvalidOption`, :exc:`exc.AmbiguousOption`
-        """
-        flags: t.List[str] = []
-        if isinstance(value, bool) and value:
-            value = "on"
-        elif isinstance(value, bool) and not value:
-            value = "off"
-
-        if unset is not None and unset:
-            assert isinstance(unset, bool)
-            flags.append("-u")
-
-        if unset_panes is not None and unset_panes:
-            assert isinstance(unset_panes, bool)
-            flags.append("-U")
-
-        if _format is not None and _format:
-            assert isinstance(_format, bool)
-            flags.append("-F")
-
-        if prevent_overwrite is not None and prevent_overwrite:
-            assert isinstance(prevent_overwrite, bool)
-            flags.append("-o")
-
-        if suppress_warnings is not None and suppress_warnings:
-            assert isinstance(suppress_warnings, bool)
-            flags.append("-q")
-
-        if append is not None and append:
-            assert isinstance(append, bool)
-            flags.append("-a")
-
-        if g is not None and g:
-            assert isinstance(g, bool)
-            flags.append("-g")
-
-        if scope is not None:
-            assert scope in OPTION_SCOPE_FLAG_MAP
-            flags.append(
-                OPTION_SCOPE_FLAG_MAP[scope],
-            )
-
-        cmd = self.cmd(
-            "set-option",
-            f"-t{self.session_id}:{self.window_index}",
-            *flags,
-            option,
-            value,
-        )
-
-        if isinstance(cmd.stderr, list) and len(cmd.stderr):
-            handle_option_error(cmd.stderr[0])
-
-        return self
-
     def show_window_options(self, g: t.Optional[bool] = False) -> "WindowOptionDict":
         """Show options for tmux window. Deprecated by :meth:`Window.show_options()`.