Extracted the method parse_window_size

Author: henrikmidtiby

manim/renderer/opengl_renderer_window.py

@@ -32,6 +32,27 @@ def __init__(
         mon_index = config.window_monitor
         monitor = monitors[min(mon_index, len(monitors) - 1)]
 
+        size = self.parse_window_size(window_size, monitor)
+
+        super().__init__(size=size)
+
+        self.title = f"Manim Community {__version__}"
+        self.size = size
+        self.renderer = renderer
+
+        mglw.activate_context(window=self)
+        self.timer = Timer()
+        self.config = mglw.WindowConfig(ctx=self.ctx, wnd=self, timer=self.timer)
+        self.timer.start()
+
+        self.swap_buffers()
+
+        initial_position = self.find_initial_position(size, monitor)
+        self.position = initial_position
+
+    def parse_window_size(
+        self, window_size: str | tuple[int, ...], monitor: Monitor
+    ) -> tuple[int, int]:
         invalid_window_size_error_message = (
             "window_size must be specified either as 'default', a string of the form "
             "'width,height', or a tuple of 2 ints of the form (width, height)."
@@ -58,22 +79,7 @@ def __init__(
             size = (window_width, window_height)
         else:
             raise ValueError(invalid_window_size_error_message)
-
-        super().__init__(size=size)
-
-        self.title = f"Manim Community {__version__}"
-        self.size = size
-        self.renderer = renderer
-
-        mglw.activate_context(window=self)
-        self.timer = Timer()
-        self.config = mglw.WindowConfig(ctx=self.ctx, wnd=self, timer=self.timer)
-        self.timer.start()
-
-        self.swap_buffers()
-
-        initial_position = self.find_initial_position(size, monitor)
-        self.position = initial_position
+        return size
 
     # Delegate event handling to scene.
     def on_mouse_motion(self, x: int, y: int, dx: int, dy: int) -> None: