Improve internal apis

vdirsyncer/cli/__init__.py

@@ -16,8 +16,8 @@ def catch_errors(f):
         try:
             f(*a, **kw)
         except:
-            if not handle_cli_error():
-                sys.exit(1)
+            handle_cli_error()
+            sys.exit(1)
 
     return inner
 

vdirsyncer/cli/tasks.py

@@ -70,8 +70,8 @@ def sync_collection(wq, pair_name, collection, config_a, config_b,
             force_delete=force_delete
         )
     except:
-        if not handle_cli_error(status_name):
-            raise JobFailed()
+        handle_cli_error(status_name)
+        raise JobFailed()
 
     save_status(general['status_path'], pair_name, collection,
                 data_type='items', data=status)

vdirsyncer/cli/utils.py

@@ -44,6 +44,13 @@ class JobFailed(RuntimeError):
 
 
 def handle_cli_error(status_name='sync'):
+    '''
+    Print a useful error message for the current exception.
+
+    This is supposed to catch all exceptions, and should never raise any
+    exceptions itself.
+    '''
+
     try:
         raise
     except CliError as e:
@@ -480,8 +487,8 @@ class WorkerQueue(object):
             try:
                 func(wq=self)
             except Exception as e:
-                if not _handle_cli_error():
-                    self._exceptions.append(e)
+                _handle_cli_error()
+                self._exceptions.append(e)
             finally:
                 self._queue.task_done()