Remove no longer needed IndexError exceptions

MultiCommand._catch_todo_errors() prevents occuring of those.

topydo/commands/DepriCommand.py

@@ -26,17 +26,13 @@ class DepriCommand(MultiCommand):
             p_args, p_todolist, p_out, p_err, p_prompt)
 
     def execute_multi_specific(self):
-        try:
-            self.printer.add_filter(PrettyPrinterNumbers(self.todolist))
+        self.printer.add_filter(PrettyPrinterNumbers(self.todolist))
 
-            for todo in self.todos:
-                if todo.priority() != None:
-                    self.todolist.set_priority(todo, None)
-                    self.out("Priority removed.")
-                    self.out(self.printer.print_todo(todo))
-
-        except IndexError:
-            self.error(self.usage())
+        for todo in self.todos:
+            if todo.priority() != None:
+                self.todolist.set_priority(todo, None)
+                self.out("Priority removed.")
+                self.out(self.printer.print_todo(todo))
 
     def usage(self):
         return """Synopsis: depri <NUMBER1> [<NUMBER2> ...]"""

topydo/commands/PostponeCommand.py

@@ -17,7 +17,6 @@
 from datetime import date, timedelta
 
 from topydo.lib.MultiCommand import MultiCommand
-from topydo.lib.Command import InvalidCommandArgument
 from topydo.lib.Config import config
 from topydo.lib.PrettyPrinterFilter import PrettyPrinterNumbers
 from topydo.lib.RelativeDate import relative_date_to_date
@@ -52,31 +51,28 @@ class PostponeCommand(MultiCommand):
 
             return offset_date
 
-        try:
-            pattern = self.args[-1]
-            self.printer.add_filter(PrettyPrinterNumbers(self.todolist))
+        pattern = self.args[-1]
+        self.printer.add_filter(PrettyPrinterNumbers(self.todolist))
 
-            for todo in self.todos:
-                offset = _get_offset(todo)
-                new_due = relative_date_to_date(pattern, offset)
-
-                if new_due:
-                    if self.move_start_date and todo.has_tag(config().tag_start()):
-                        length = todo.length()
-                        new_start = new_due - timedelta(length)
-                        # pylint: disable=E1103
-                        todo.set_tag(config().tag_start(), new_start.isoformat())
+        for todo in self.todos:
+            offset = _get_offset(todo)
+            new_due = relative_date_to_date(pattern, offset)
 
+            if new_due:
+                if self.move_start_date and todo.has_tag(config().tag_start()):
+                    length = todo.length()
+                    new_start = new_due - timedelta(length)
                     # pylint: disable=E1103
-                    todo.set_tag(config().tag_due(), new_due.isoformat())
-
-                    self.todolist.set_dirty()
-                    self.out(self.printer.print_todo(todo))
-                else:
-                    self.error("Invalid date pattern given.")
-                    break
-        except (InvalidCommandArgument, IndexError):
-            self.error(self.usage())
+                    todo.set_tag(config().tag_start(), new_start.isoformat())
+
+                # pylint: disable=E1103
+                todo.set_tag(config().tag_due(), new_due.isoformat())
+
+                self.todolist.set_dirty()
+                self.out(self.printer.print_todo(todo))
+            else:
+                self.error("Invalid date pattern given.")
+                break
 
     def usage(self):
         return "Synopsis: postpone [-s] <NUMBER> [<NUMBER2> ...] <PATTERN>"

topydo/commands/PriorityCommand.py

@@ -31,26 +31,23 @@ class PriorityCommand(MultiCommand):
     def execute_multi_specific(self):
         priority = None
 
-        try:
-            priority = self.args[-1]
-            self.printer.add_filter(PrettyPrinterNumbers(self.todolist))
-
-            if is_valid_priority(priority):
-                for todo in self.todos:
-                    old_priority = todo.priority()
-                    self.todolist.set_priority(todo, priority)
-
-                    if old_priority and priority and old_priority != priority:
-                        self.out("Priority changed from {} to {}".format(
-                            old_priority, priority))
-                    elif not old_priority:
-                        self.out("Priority set to {}.".format(priority))
-
-                    self.out(self.printer.print_todo(todo))
-            else:
-                self.error("Invalid priority given.")
-        except IndexError:
-            self.error(self.usage())
+        priority = self.args[-1]
+        self.printer.add_filter(PrettyPrinterNumbers(self.todolist))
+
+        if is_valid_priority(priority):
+            for todo in self.todos:
+                old_priority = todo.priority()
+                self.todolist.set_priority(todo, priority)
+
+                if old_priority and priority and old_priority != priority:
+                    self.out("Priority changed from {} to {}".format(
+                        old_priority, priority))
+                elif not old_priority:
+                    self.out("Priority set to {}.".format(priority))
+
+                self.out(self.printer.print_todo(todo))
+        else:
+            self.error("Invalid priority given.")
 
     def usage(self):
         return """Synopsis: pri <NUMBER1> [<NUMBER2> ...] <PRIORITY>"""