Linux: minor optimization when checking for a X extension status

By moving the code not using the thread lock outside of the context manager.

Author: BoboTiG

CHANGELOG.md

@@ -6,6 +6,7 @@ See Git checking messages for full history.
 - removed support for Python 3.8
 - added support for Python 3.14
 - Linux: fixed a threadding issue in `.close()` when calling `XCloseDisplay()` (#251)
+- Linux: minor optimization when checking for a X extension status (#251)
 - :heart: contributors: @
 
 ## 9.0.2 (2024-09-01)

src/mss/linux.py

@@ -352,22 +352,22 @@ def close(self) -> None:
 
     def _is_extension_enabled(self, name: str, /) -> bool:
         """Return True if the given *extension* is enabled on the server."""
-        with lock:
-            major_opcode_return = c_int()
-            first_event_return = c_int()
-            first_error_return = c_int()
+        major_opcode_return = c_int()
+        first_event_return = c_int()
+        first_error_return = c_int()
 
-            try:
+        try:
+            with lock:
                 self.xlib.XQueryExtension(
                     self._handles.display,
                     name.encode("latin1"),
                     byref(major_opcode_return),
                     byref(first_event_return),
                     byref(first_error_return),
                 )
-            except ScreenShotError:
-                return False
-            return True
+        except ScreenShotError:
+            return False
+        return True
 
     def _set_cfunctions(self) -> None:
         """Set all ctypes functions and attach them to attributes."""