From 9688cc0c73f7582302cdc8fa1efdc81e2c472c7e Mon Sep 17 00:00:00 2001 From: Ralf Jung Date: Mon, 15 Apr 2024 15:56:47 +0200 Subject: [PATCH 1/1] prefer extending with tuples over lists --- bubblebox.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/bubblebox.py b/bubblebox.py index 02bcb5a..a806e09 100644 --- a/bubblebox.py +++ b/bubblebox.py @@ -71,7 +71,7 @@ class DbusProxyDirective: os.set_inheritable(bwrap_end, True) # Put this at the usual location for the bus insode the sandbox. # TODO: What if DBUS_SESSION_BUS_ADDRESS says something else? - bwrap.flags.extend(["--bind", filename, XDG_RUNTIME_DIR + "/bus", "--sync-fd", str(bwrap_end)]) + bwrap.flags.extend(("--bind", filename, XDG_RUNTIME_DIR + "/bus", "--sync-fd", str(bwrap_end))) # Constructors that should be used instead of directly mentioning the class above. def bwrap_flags(*flags): @@ -141,7 +141,7 @@ def host_access(dirs): recursive_host_access(path, desc, out) else: # Allow access to this path - out.extend([Access.flag(desc), path, path]) + out.extend((Access.flag(desc), path, path)) # Start the recursive traversal out = [] recursive_host_access("", dirs, out) -- 2.30.2