Moved to use contextlib suppress pattern than empty exception blocks

src/app.py

@@ -1,4 +1,5 @@
 # Python imports
+from contextlib import suppress
 import signal
 import os
 
@@ -52,18 +53,15 @@ class Application:
         except Exception:
             ipc_server.send_test_ipc_message()
 
-        try:
+        with suppress(Exception):
             ipc_server.create_ipc_listener()
-        except Exception as e:
-            ...
 
     def setup_debug_hook(self):
-        try:
+        # Typically: ValueError: signal only works in main thread
+        with suppress(ValueError):
             # kill -SIGUSR2 <pid> from Linux/Unix or SIGBREAK signal from Windows
             signal.signal(
                 vars(signal).get("SIGBREAK") or vars(signal).get("SIGUSR2"),
                 debug_signal_handler
             )
-        except ValueError:
+
-            # Typically: ValueError: signal only works in main thread
-            ...

src/core/controllers/base_controller_data.py

@@ -46,7 +46,8 @@ class BaseControllerData:
 
             logger.info(f"Not a File: {arg}")
 
-        if len(files) > 0:
+        if len(files) == 0: return
+
         settings_manager.set_is_starting_with_file(True)
         settings_manager.set_starting_files(files)
 
@@ -84,24 +85,22 @@ class BaseControllerData:
             widget.remove(child)
 
     def get_clipboard_data(self, encoding = "utf-8") -> str:
-        if which("xclip"):
+        if not which("xclip"):
-            command = ['xclip','-selection','clipboard']
-        else:
             logger.info('xclip not found...')
             return
 
+        command = ['xclip','-selection','clipboard']
         proc    = subprocess.Popen(['xclip','-selection', 'clipboard', '-o'], stdout = subprocess.PIPE)
         retcode = proc.wait()
         data    = proc.stdout.read()
         return data.decode(encoding).strip()
 
     def set_clipboard_data(self, data: type, encoding = "utf-8") -> None:
-        if which("xclip"):
+        if not which("xclip"):
-            command = ['xclip','-selection','clipboard']
-        else:
             logger.info('xclip not found...')
             return
 
+        command = ['xclip','-selection','clipboard']
         proc = subprocess.Popen(command, stdin = subprocess.PIPE)
         proc.stdin.write(data.encode(encoding))
         proc.stdin.close()

src/core/widgets/controls/open_files_button.py

@@ -1,4 +1,5 @@
 # Python imports
+from contextlib import suppress
 import os
 
 # Lib imports
@@ -58,11 +59,9 @@ class OpenFilesButton(Gtk.Button):
 
         chooser.set_select_multiple(True)
 
-        try:
+        with suppress(Exception):
             folder = widget.get_current_file().get_parent() if not start_dir else start_dir
             chooser.set_current_folder( folder.get_path() )
-        except Exception as e:
-            ...
 
         response = chooser.run()
         if not response == Gtk.ResponseType.OK:

src/core/widgets/vte_widget.py

@@ -99,7 +99,7 @@ class VteWidget(Vte.Terminal):
         try:
             command_ran  = lines[-1].split("-->:")[1].strip()
         except VteWidgetException as e:
-            logger.debud(e)
+            logger.debug(e)
             return
 
         if not command_ran[0:3].encode() in self.cd_cmd_prefix:
@@ -114,12 +114,12 @@ class VteWidget(Vte.Terminal):
         event = Event("pty_path_updated", "", target_path)
         event_system.emit("handle_bridge_event", (event,))
 
-    def update_term_path(self, fpath):
+    def update_term_path(self, fpath: str):
         self.dont_process = True
 
         cmds = [f"cd '{fpath}'\n", "clear\n"]
-        for i in cmds:
+        for cmd in cmds:
-            self.run_command(i)
+            self.run_command(cmd)
 
-    def run_command(self, cmd):
+    def run_command(self, cmd: str):
         self.feed_child_binary(bytes(cmd, 'utf8'))

src/libs/singleton.py

@@ -12,13 +12,11 @@ class SingletonError(Exception):
 
 
 class Singleton:
-    ccount = 0
+    _instance = None
 
     def __new__(cls, *args, **kwargs):
-        obj        = super(Singleton, cls).__new__(cls)
+        if cls._instance:
-        cls.ccount += 1
+            raise SingletonError(f"'{cls.__name__}' is a Singleton. Cannot create a new instance...")
 
-        if cls.ccount == 2:
+        cls._instance = super(Singleton, cls).__new__(cls)
-            raise SingletonError(f"Exceeded {cls.__name__} instantiation limit...")
+        return cls._instance
-
-        return obj