newer syntax is just better

Author: totaam

packaging/MSWindows/BUILD.py

@@ -388,7 +388,7 @@ def build_service() -> None:
     os.rename(os.path.join(SERVICE_SRC_DIR, XPRA_SERVICE_EXE), XPRA_SERVICE_EXE)
 
 
-VersionInfo = namedtuple("VersionInfo", "string,value,revision,full_string,arch_info,extra,padded")
+VersionInfo = namedtuple("VersionInfo", ("string", "value", "revision", "full_string", "arch_info", "extra", "padded"))
 version_info = VersionInfo("invalid", (0, 0), 0, "invalid", "arch", "extra", (0, 0, 0, 0))
 
 

xpra/audio/wrapper.py

@@ -390,7 +390,7 @@ def start_sending_audio(plugins, audio_source_plugin: str, device: str, codec: s
          remote_decoders, remote_pulseaudio_server, remote_pulseaudio_id))
     with log.trap_error("Error setting up %r audio source" % (audio_source_plugin or "auto")):
         # info about the remote end:
-        PAInfo = namedtuple("PAInfo", "pulseaudio_server,pulseaudio_id,remote_decoders")
+        PAInfo = namedtuple("PAInfo", ("pulseaudio_server", "pulseaudio_id", "remote_decoders"))
         remote = PAInfo(pulseaudio_server=remote_pulseaudio_server,
                         pulseaudio_id=remote_pulseaudio_id,
                         remote_decoders=remote_decoders)

xpra/client/gtk3/opengl/client_window.py

@@ -14,8 +14,8 @@
 
 log = Logger("opengl", "window")
 
-Rectangle = namedtuple("Rectangle", "x,y,width,height")
-DrawEvent = namedtuple("DrawEvent", "area")
+Rectangle = namedtuple("Rectangle", ("x", "y", "width", "height"))
+DrawEvent = namedtuple("DrawEvent", ("area", ))
 
 MONITOR_REINIT = envbool("XPRA_OPENGL_MONITOR_REINIT", False)
 

xpra/net/quic/asyncio_thread.py

@@ -24,7 +24,7 @@
 
 UVLOOP = envbool("XPRA_UVLOOP", not WIN32)
 
-ExceptionWrapper = namedtuple("ExceptionWrapper", "exception,args")
+ExceptionWrapper = namedtuple("ExceptionWrapper", ("exception", "args"))
 
 
 class ThreadedAsyncioLoop:

xpra/server/dbus/server.py

@@ -21,7 +21,7 @@
 
 log = Logger("dbus", "server")
 
-Rectangle = namedtuple("Rectangle", "x,y,width,height")
+Rectangle = namedtuple("Rectangle", ("x", "y", "width", "height"))
 
 
 def n(*args):

xpra/server/shadow/root_window_model.py

@@ -188,7 +188,7 @@ def notify(self, prop: str) -> None:
         if listeners is None:
             log.warn(f"Warning: ignoring notify for {prop!r}")
             return
-        PSpec = namedtuple("PSpec", "name")
+        PSpec = namedtuple("PSpec", ("name", ))
         pspec = PSpec(name=prop)
         for listener, *args in listeners:
             with log.trap_error(f"Error on {prop!r} signal listener {listener}"):

xpra/x11/desktop/monitor_model.py

@@ -20,7 +20,7 @@
 MIN_SIZE = 640, 350
 MAX_SIZE = 8192, 8192
 
-MonitorDamageNotify = namedtuple("MonitorDamageNotify", "x,y,width,height")
+MonitorDamageNotify = namedtuple("MonitorDamageNotify", ("x", "y", "width", "height"))
 
 
 class MonitorDesktopModel(DesktopModelBase):