linting: pylint: Enable `broad-except`, Improve logging

Handle exceptions explicitly to prevent unexpected errors from causing
problems.

Improve exception and notification logging.

40 files changed, 290 insertions, 305 deletions

diff --git a/ranger/api/commands.py b/ranger/api/commands.py
index 25353b5b..6b761de8 100644
--- a/ranger/api/commands.py
+++ b/ranger/api/commands.py
@@ -29,18 +29,14 @@ class CommandContainer(object):
         return self.commands[key]
 
     def alias(self, name, full_command):
-        try:
-            cmd = type(name, (AliasCommand, ), dict())
-            # pylint: disable=protected-access
-            cmd._based_function = name
-            cmd._function_name = name
-            cmd._object_name = name
-            cmd._line = full_command
-            # pylint: enable=protected-access
-            self.commands[name] = cmd
-
-        except Exception:
-            pass
+        cmd = type(name, (AliasCommand, ), dict())
+        # pylint: disable=protected-access
+        cmd._based_function = name
+        cmd._function_name = name
+        cmd._object_name = name
+        cmd._line = full_command
+        # pylint: enable=protected-access
+        self.commands[name] = cmd
 
     def load_commands_from_module(self, module):
         for var in vars(module).values():
@@ -392,13 +388,13 @@ class FunctionCommand(Command):
             value = arg if (equal_sign is -1) else arg[equal_sign + 1:]
             try:
                 value = int(value)
-            except Exception:
+            except ValueError:
                 if value in ('True', 'False'):
                     value = (value == 'True')
                 else:
                     try:
                         value = float(value)
-                    except Exception:
+                    except ValueError:
                         pass
 
             if equal_sign == -1:
diff --git a/ranger/config/commands.py b/ranger/config/commands.py
index bb20ddcb..eba94e19 100755
--- a/ranger/config/commands.py
+++ b/ranger/config/commands.py
@@ -598,9 +598,10 @@ class console(Command):
         if self.arg(1)[0:2] == '-p':
             try:
                 position = int(self.arg(1)[2:])
-                self.shift()
-            except Exception:
+            except ValueError:
                 pass
+            else:
+                self.shift()
         self.fm.open_console(self.rest(1), position=position)
 
 
@@ -614,10 +615,10 @@ class load_copy_buffer(Command):
     def execute(self):
         from ranger.container.file import File
         from os.path import exists
+        fname = self.fm.confpath(self.copy_buffer_filename)
         try:
-            fname = self.fm.confpath(self.copy_buffer_filename)
             fobj = open(fname, 'r')
-        except Exception:
+        except OSError:
             return self.fm.notify(
                 "Cannot open %s" % (fname or self.copy_buffer_filename), bad=True)
         self.fm.copy_buffer = set(File(g)
@@ -635,10 +636,10 @@ class save_copy_buffer(Command):
 
     def execute(self):
         fname = None
+        fname = self.fm.confpath(self.copy_buffer_filename)
         try:
-            fname = self.fm.confpath(self.copy_buffer_filename)
             fobj = open(fname, 'w')
-        except Exception:
+        except OSError:
             return self.fm.notify("Cannot open %s" %
                                   (fname or self.copy_buffer_filename), bad=True)
         fobj.write("\n".join(fobj.path for fobj in self.fm.copy_buffer))
@@ -744,7 +745,7 @@ class eval_(Command):
             else:
                 if result and not quiet:
                     p(result)
-        except Exception as err:
+        except Exception as err:  # pylint: disable=broad-except
             p(err)
 
 
@@ -841,15 +842,12 @@ class chmod(Command):
         for fobj in self.fm.thistab.get_selection():
             try:
                 os.chmod(fobj.path, mode)
-            except Exception as ex:
+            except OSError as ex:
                 self.fm.notify(ex)
 
-        try:
-            # reloading directory.  maybe its better to reload the selected
-            # files only.
-            self.fm.thisdir.load_content()
-        except Exception:
-            pass
+        # reloading directory.  maybe its better to reload the selected
+        # files only.
+        self.fm.thisdir.content_outdated = True
 
 
 class bulkrename(Command):
@@ -1270,7 +1268,7 @@ class scout(Command):
         # pylint: enable=no-member
         try:
             self._regex = re.compile(regex, options)
-        except Exception:
+        except re.error:
             self._regex = re.compile("")
         return self._regex
 
diff --git a/ranger/config/pylintrc b/ranger/config/pylintrc
index adec41f1..7cac80d1 100644
--- a/ranger/config/pylintrc
+++ b/ranger/config/pylintrc
@@ -5,4 +5,4 @@ class-rgx=[a-z][a-z0-9_]{1,30}$
 [FORMAT]
 max-line-length = 99
 max-module-lines=3000
-disable=locally-disabled,locally-enabled,missing-docstring,duplicate-code,fixme,broad-except
+disable=locally-disabled,locally-enabled,missing-docstring,duplicate-code,fixme
diff --git a/ranger/container/bookmarks.py b/ranger/container/bookmarks.py
index edffa5ae..305ac4c2 100644
--- a/ranger/container/bookmarks.py
+++ b/ranger/container/bookmarks.py
@@ -155,12 +155,8 @@ class Bookmarks(object):
         if os.access(self.path, os.W_OK):
             fobj = open(self.path + ".new", 'w')
             for key, value in self.dct.items():
-                if isinstance(key, str)\
-                        and key in ALLOWED_KEYS:
-                    try:
-                        fobj.write("{0}:{1}\n".format(str(key), str(value)))
-                    except Exception:
-                        pass
+                if isinstance(key, str) and key in ALLOWED_KEYS:
+                    fobj.write("{0}:{1}\n".format(str(key), str(value)))
 
             fobj.close()
             old_perms = os.stat(self.path)
@@ -181,8 +177,8 @@ class Bookmarks(object):
         if not os.path.exists(self.path):
             try:
                 fobj = open(self.path, 'w')
-            except Exception:
-                raise OSError('Cannot read the given path')
+            except OSError:
+                raise
             fobj.close()
 
         if os.access(self.path, os.R_OK):
diff --git a/ranger/container/directory.py b/ranger/container/directory.py
index 4fbc2e29..22901006 100644
--- a/ranger/container/directory.py
+++ b/ranger/container/directory.py
@@ -331,25 +331,25 @@ class Directory(  # pylint: disable=too-many-instance-attributes,too-many-public
                             file_stat = os_stat(name)
                         else:
                             file_stat = file_lstat
+                    except OSError:
+                        file_lstat = None
+                        file_stat = None
+                    if file_lstat and file_stat:
                         stats = (file_stat, file_lstat)
                         is_a_dir = file_stat.st_mode & 0o170000 == 0o040000
-                    except Exception:
+                    else:
                         stats = None
                         is_a_dir = False
+
                     if is_a_dir:
-                        try:
-                            item = self.fm.get_directory(name)
-                            item.load_if_outdated()
-                        except Exception:
-                            item = Directory(name, preload=stats, path_is_abs=True,
-                                             basename_is_rel_to=basename_is_rel_to)
-                            item.load()
+                        item = self.fm.get_directory(name, preload=stats, path_is_abs=True,
+                                                     basename_is_rel_to=basename_is_rel_to)
+                        item.load_if_outdated()
+                        if self.flat:
+                            item.relative_path = os.path.relpath(item.path, self.path)
                         else:
-                            if self.flat:
-                                item.relative_path = os.path.relpath(item.path, self.path)
-                            else:
-                                item.relative_path = item.basename
-                            item.relative_path_lower = item.relative_path.lower()
+                            item.relative_path = item.basename
+                        item.relative_path_lower = item.relative_path.lower()
                         if item.vcs and item.vcs.track:
                             if item.vcs.is_root_pointer:
                                 has_vcschild = True
@@ -454,7 +454,7 @@ class Directory(  # pylint: disable=too-many-instance-attributes,too-many-public
 
         try:
             sort_func = self.sort_dict[self.settings.sort]
-        except Exception:
+        except KeyError:
             sort_func = sort_by_basename
 
         if self.settings.sort_case_insensitive and \
@@ -494,9 +494,9 @@ class Directory(  # pylint: disable=too-many-instance-attributes,too-many-public
                         stat = os_stat(realpath(dirpath + "/" + fname))
                     else:
                         stat = os_stat(dirpath + "/" + fname)
-                    cum += stat.st_size
-                except Exception:
-                    pass
+                except OSError:
+                    continue
+                cum += stat.st_size
         return cum
 
     def look_up_cumulative_size(self):
@@ -548,7 +548,7 @@ class Directory(  # pylint: disable=too-many-instance-attributes,too-many-public
     def move_to_obj(self, arg, attr=None):
         try:
             arg = arg.path
-        except Exception:
+        except AttributeError:
             pass
         self.load_content_once(schedule=False)
         if self.empty():
@@ -594,11 +594,11 @@ class Directory(  # pylint: disable=too-many-instance-attributes,too-many-public
         """Make sure the pointer is in the valid range"""
         Accumulator.correct_pointer(self)
 
-        try:
-            if self == self.fm.thisdir:
+        if self == self.fm.thisdir:
+            try:
                 self.fm.thisfile = self.pointed_obj
-        except Exception:
-            pass
+            except AttributeError:
+                pass
 
     def load_content_once(self, *a, **k):
         """Load the contents of the directory if not done yet"""
diff --git a/ranger/container/file.py b/ranger/container/file.py
index e1284c1f..e0b9922c 100644
--- a/ranger/container/file.py
+++ b/ranger/container/file.py
@@ -54,14 +54,11 @@ class File(FileSystemObject):
     def firstbytes(self):
         if self._firstbytes is None:
             try:
-                fobj = open(self.path, 'r')
-                self._firstbytes = fobj.read(N_FIRST_BYTES)
-                fobj.close()
-                return self._firstbytes
-            except Exception:
-                pass
-        else:
-            return self._firstbytes
+                with open(self.path, 'r') as fobj:
+                    self._firstbytes = fobj.read(N_FIRST_BYTES)
+            except OSError:
+                return None
+        return self._firstbytes
 
     def is_binary(self):
         if self.firstbytes and control_characters & set(self.firstbytes):
diff --git a/ranger/container/fsobject.py b/ranger/container/fsobject.py
index fbb75e6c..0447d5a6 100644
--- a/ranger/container/fsobject.py
+++ b/ranger/container/fsobject.py
@@ -18,6 +18,7 @@ from ranger.ext import spawn
 from ranger.ext.lazy_property import lazy_property
 from ranger.ext.human_readable import human_readable
 
+# Python 2 compatibility
 if hasattr(str, 'maketrans'):
     maketrans = str.maketrans  # pylint: disable=invalid-name,no-member
 else:
@@ -164,14 +165,14 @@ class FileSystemObject(  # pylint: disable=too-many-instance-attributes
     def user(self):
         try:
             return getpwuid(self.stat.st_uid)[0]
-        except Exception:
+        except KeyError:
             return str(self.stat.st_uid)
 
     @lazy_property
     def group(self):
         try:
             return getgrgid(self.stat.st_gid)[0]
-        except Exception:
+        except KeyError:
             return str(self.stat.st_gid)
 
     for attr in ('video', 'audio', 'image', 'media', 'document', 'container'):
@@ -220,7 +221,7 @@ class FileSystemObject(  # pylint: disable=too-many-instance-attributes
     def mimetype(self):
         try:
             return self._mimetype
-        except Exception:
+        except AttributeError:
             self.set_mimetype()
             return self._mimetype
 
@@ -228,7 +229,7 @@ class FileSystemObject(  # pylint: disable=too-many-instance-attributes
     def mimetype_tuple(self):
         try:
             return self._mimetype_tuple
-        except Exception:
+        except AttributeError:
             self.set_mimetype()
             return self._mimetype_tuple
 
@@ -245,7 +246,7 @@ class FileSystemObject(  # pylint: disable=too-many-instance-attributes
         if self.is_link:
             try:
                 return realpath(self.path)
-            except Exception:
+            except OSError:
                 return None  # it is impossible to get the link destination
         return self.path
 
@@ -279,7 +280,7 @@ class FileSystemObject(  # pylint: disable=too-many-instance-attributes
                 if self.is_link:
                     new_stat = stat(path)
                 self.exists = True
-            except Exception:
+            except OSError:
                 self.exists = False
 
         # Set some attributes
diff --git a/ranger/container/history.py b/ranger/container/history.py
index 31830b55..b3a1943e 100644
--- a/ranger/container/history.py
+++ b/ranger/container/history.py
@@ -35,7 +35,7 @@ class History(object):
         if self.unique:
             try:
                 self._history.remove(item)
-            except Exception:
+            except ValueError:
                 pass
         else:
             if self._history and self._history[-1] == item:
@@ -51,9 +51,10 @@ class History(object):
         if self._history and unique:
             try:
                 self._history.remove(item)
-                self._index -= 1
-            except Exception:
+            except ValueError:
                 pass
+            else:
+                self._index -= 1
         try:
             self._history[self._index] = item
         except IndexError:
@@ -101,13 +102,13 @@ class History(object):
         try:
             return self._history[-1]
         except IndexError:
-            raise HistoryEmptyException()
+            raise HistoryEmptyException
 
     def bottom(self):
         try:
             return self._history[0]
         except IndexError:
-            raise HistoryEmptyException()
+            raise HistoryEmptyException
 
     def back(self):
         self._index -= 1
diff --git a/ranger/container/settings.py b/ranger/container/settings.py
index c42f50f2..012b5a2c 100644
--- a/ranger/container/settings.py
+++ b/ranger/container/settings.py
@@ -166,7 +166,7 @@ class Settings(SignalDispatcher, FileManagerAware):
         else:
             try:
                 localpath = self.fm.thisdir.path
-            except Exception:
+            except AttributeError:
                 localpath = path
 
         if localpath:
@@ -236,8 +236,7 @@ class Settings(SignalDispatcher, FileManagerAware):
             if path not in self._localsettings:
                 try:
                     regex = re.compile(path)
-                except Exception:
-                    # Bad regular expression
+                except re.error:  # Bad regular expression
                     return
                 self._localregexes[path] = regex
                 self._localsettings[path] = dict()
diff --git a/ranger/core/actions.py b/ranger/core/actions.py
index 5e840058..baf98563 100644
--- a/ranger/core/actions.py
+++ b/ranger/core/actions.py
@@ -33,6 +33,7 @@ from ranger.container.file import File
 from ranger.core.loader import CommandLoader, CopyLoader
 from ranger.container.settings import ALLOWED_SETTINGS, ALLOWED_VALUES
 
+
 MACRO_FAIL = "<\x01\x01MACRO_HAS_NO_VALUE\x01\01>"
 
 LOG = getLogger(__name__)
@@ -57,7 +58,7 @@ class Actions(  # pylint: disable=too-many-instance-attributes,too-many-public-m
 
         Exit the program.
         """
-        raise SystemExit()
+        raise SystemExit
 
     def reset(self):
         """:reset
@@ -143,25 +144,35 @@ class Actions(  # pylint: disable=too-many-instance-attributes,too-many-public-m
         """
         try:
             cwd = self.thisdir
-        except Exception:
+        except AttributeError:
             pass
         else:
             cwd.unload()
             cwd.load_content()
 
-    def notify(self, text, duration=4, bad=False):
+    def notify(self, obj, duration=4, bad=False, exception=None):
         """:notify <text>
 
         Display the text in the statusbar.
         """
-        if isinstance(text, Exception):
+        if isinstance(obj, Exception):
             if ranger.args.debug:
-                raise text
+                raise obj
+            exception = obj
             bad = True
-        elif bad is True and ranger.args.debug:
-            raise Exception(str(text))
-        text = str(text)
-        LOG.debug("Command notify invoked: [Bad: %s, Text: '%s']", bad, text)
+        elif bad and ranger.args.debug:
+            raise Exception(str(obj))
+
+        text = str(obj)
+
+        text_log = 'Notification: {0}'.format(text)
+        if bad:
+            LOG.error(text_log)
+        else:
+            LOG.info(text_log)
+        if exception:
+            LOG.exception(exception)
+
         if self.ui and self.ui.is_on:
             self.ui.status.notify("  ".join(text.split("\n")),
                                   duration=duration, bad=bad)
@@ -175,7 +186,7 @@ class Actions(  # pylint: disable=too-many-instance-attributes,too-many-public-m
         """
         try:
             item = self.loader.queue[0]
-        except Exception:
+        except IndexError:
             self.notify("Type Q or :quit<Enter> to exit ranger")
         else:
             self.notify("Aborting: " + item.get_description())
@@ -230,7 +241,7 @@ class Actions(  # pylint: disable=too-many-instance-attributes,too-many-public-m
                     return self.notify(ex)
         try:
             cmd_class(string, quantifier=quantifier).execute()
-        except Exception as ex:
+        except Exception as ex:  # pylint: disable=broad-except
             if ranger.args.debug:
                 raise
             else:
@@ -296,7 +307,7 @@ class Actions(  # pylint: disable=too-many-instance-attributes,too-many-public-m
         for i in range(1, 10):
             try:
                 tab = self.fm.tabs[i]
-            except Exception:
+            except KeyError:
                 continue
             tabdir = tab.thisdir
             if not tabdir:
@@ -365,7 +376,7 @@ class Actions(  # pylint: disable=too-many-instance-attributes,too-many-public-m
                     continue
                 try:
                     self.execute_console(line)
-                except Exception as ex:
+                except Exception as ex:  # pylint: disable=broad-except
                     if ranger.args.debug:
                         raise
                     else:
@@ -396,7 +407,7 @@ class Actions(  # pylint: disable=too-many-instance-attributes,too-many-public-m
                     fobj.path + "\n" for fobj in self.fm.thistab.get_selection()))
 
             if ranger.args.choosefile or ranger.args.choosefiles:
-                raise SystemExit()
+                raise SystemExit
 
         if isinstance(files, set):
             files = list(files)
@@ -440,10 +451,7 @@ class Actions(  # pylint: disable=too-many-instance-attributes,too-many-public-m
             steps = direction.left()
             if narg is not None:
                 steps *= narg
-            try:
-                directory = os.path.join(*(['..'] * steps))
-            except Exception:
-                return
+            directory = os.path.join(*(['..'] * steps))
             self.thistab.enter_dir(directory)
             self.change_mode('normal')
         if cwd and cwd.accessible and cwd.content_loaded:
@@ -468,7 +476,7 @@ class Actions(  # pylint: disable=too-many-instance-attributes,too-many-public-m
                 if self.mode == 'visual':
                     try:
                         startpos = cwd.index(self._visual_start)
-                    except Exception:
+                    except ValueError:
                         self._visual_start = None
                         startpos = min(self._visual_start_pos, len(cwd))
                     # The files between here and _visual_start_pos
@@ -703,7 +711,7 @@ class Actions(  # pylint: disable=too-many-instance-attributes,too-many-public-m
         if isinstance(text, str) and regexp:
             try:
                 text = re.compile(text, re.UNICODE | re.IGNORECASE)  # pylint: disable=no-member
-            except Exception:
+            except re.error:
                 return False
         self.thistab.last_search = text
         self.search_next(order='search', offset=offset)
@@ -839,7 +847,7 @@ class Actions(  # pylint: disable=too-many-instance-attributes,too-many-public-m
     def draw_possible_programs(self):
         try:
             target = self.thistab.get_selection()[0]
-        except Exception:
+        except IndexError:
             self.ui.browser.draw_info = []
             return
         programs = [program for program in self.rifle.list_commands([target.path], None)]
@@ -860,7 +868,7 @@ class Actions(  # pylint: disable=too-many-instance-attributes,too-many-public-m
     def display_command_help(self, console_widget):
         try:
             command = console_widget._get_cmd_class()  # pylint: disable=protected-access
-        except Exception:
+        except KeyError:
             self.notify("Feature not available!", bad=True)
             return
 
@@ -912,9 +920,10 @@ class Actions(  # pylint: disable=too-many-instance-attributes,too-many-public-m
     def update_preview(self, path):
         try:
             del self.previews[path]
-            self.ui.need_redraw = True
-        except Exception:
+        except KeyError:
             return False
+        self.ui.need_redraw = True
+        return True
 
     @staticmethod
     def sha1_encode(path):
@@ -941,7 +950,7 @@ class Actions(  # pylint: disable=too-many-instance-attributes,too-many-public-m
             # PDF file.
             try:
                 data = self.previews[path]
-            except Exception:
+            except KeyError:
                 data = self.previews[path] = {'loading': False}
             else:
                 if data['loading']:
@@ -959,7 +968,7 @@ class Actions(  # pylint: disable=too-many-instance-attributes,too-many-public-m
             if found is False:
                 try:
                     stat_ = os.stat(self.settings.preview_script)
-                except Exception:
+                except OSError:
                     self.fm.notify(
                         "Preview Script `%s' doesn't exist!" % self.settings.preview_script,
                         bad=True,
@@ -1047,7 +1056,7 @@ class Actions(  # pylint: disable=too-many-instance-attributes,too-many-public-m
                 def on_destroy(signal):  # pylint: disable=unused-argument
                     try:
                         del self.previews[path]
-                    except Exception:
+                    except KeyError:
                         pass
                 loadable.signal_bind('after', on_after)
                 loadable.signal_bind('destroy', on_destroy)
@@ -1058,7 +1067,7 @@ class Actions(  # pylint: disable=too-many-instance-attributes,too-many-public-m
         else:
             try:
                 return codecs.open(path, 'r', errors='ignore')
-            except Exception:
+            except OSError:
                 return None
 
     # --------------------------
@@ -1317,31 +1326,34 @@ class Actions(  # pylint: disable=too-many-instance-attributes,too-many-public-m
     def paste_symlink(self, relative=False):
         copied_files = self.copy_buffer
         for fobj in copied_files:
-            self.notify(next_available_filename(fobj.basename))
+            new_name = next_available_filename(fobj.basename)
+            self.notify(new_name)
             try:
-                new_name = next_available_filename(fobj.basename)
                 if relative:
                     relative_symlink(fobj.path, join(getcwd(), new_name))
                 else:
                     symlink(fobj.path, join(getcwd(), new_name))
-            except Exception as ex:
-                self.notify(ex)
+            except OSError as ex:
+                self.notify('Failed to paste symlink: View log for more info',
+                            bad=True, exception=ex)
 
     def paste_hardlink(self):
         for fobj in self.copy_buffer:
+            new_name = next_available_filename(fobj.basename)
             try:
-                new_name = next_available_filename(fobj.basename)
                 link(fobj.path, join(getcwd(), new_name))
-            except Exception as ex:
-                self.notify(ex)
+            except OSError as ex:
+                self.notify('Failed to paste hardlink: View log for more info',
+                            bad=True, exception=ex)
 
     def paste_hardlinked_subtree(self):
         for fobj in self.copy_buffer:
             try:
                 target_path = join(getcwd(), fobj.basename)
                 self._recurse_hardlinked_tree(fobj.path, target_path)
-            except Exception as ex:
-                self.notify(ex)
+            except OSError as ex:
+                self.notify('Failed to paste hardlinked subtree: View log for more info',
+                            bad=True, exception=ex)
 
     def _recurse_hardlinked_tree(self, source_path, target_path):
         if isdir(source_path):
diff --git a/ranger/core/fm.py b/ranger/core/fm.py
index 87c8b9f1..f524bbc4 100644
--- a/ranger/core/fm.py
+++ b/ranger/core/fm.py
@@ -7,7 +7,6 @@ from __future__ import (absolute_import, division, print_function)
 
 from time import time
 from collections import deque
-import logging
 import mimetypes
 import os.path
 import pwd
@@ -33,9 +32,6 @@ from ranger.core.loader import Loader
 from ranger.ext import logutils
 
 
-LOG = logging.getLogger(__name__)
-
-
 class FM(Actions,  # pylint: disable=too-many-instance-attributes
          SignalDispatcher):
     input_blocked = False
@@ -74,7 +70,7 @@ class FM(Actions,  # pylint: disable=too-many-instance-attributes
 
         try:
             self.username = pwd.getpwuid(os.geteuid()).pw_name
-        except Exception:
+        except KeyError:
             self.username = 'uid:' + str(os.geteuid())
         self.hostname = socket.gethostname()
         self.home_path = os.path.expanduser('~')
@@ -195,13 +191,13 @@ class FM(Actions,  # pylint: disable=too-many-instance-attributes
         if self.ui:
             try:
                 self.ui.destroy()
-            except Exception:
+            except Exception:  # pylint: disable=broad-except
                 if debug:
                     raise
         if self.loader:
             try:
                 self.loader.destroy()
-            except Exception:
+            except Exception:  # pylint: disable=broad-except
                 if debug:
                     raise
 
@@ -209,10 +205,10 @@ class FM(Actions,  # pylint: disable=too-many-instance-attributes
     def get_log():
         """Return the current log
 
-        The log is returned as a list of string
+        The log is returned as a generator over its entries' lines
         """
-        for log in logutils.log_queue:
-            for line in log.split('\n'):
+        for entry in logutils.QUEUE:
+            for line in entry.splitlines():
                 yield line
 
     def _get_image_displayer(self):
@@ -270,10 +266,10 @@ class FM(Actions,  # pylint: disable=too-many-instance-attributes
                         print(ranger.args.confdir)
                         print("To run ranger without the need for configuration")
                         print("files, use the --clean option.")
-                        raise SystemExit()
+                        raise SystemExit
                 try:
                     shutil.copy(self.relpath(src), self.confpath(dest))
-                except Exception as ex:
+                except OSError as ex:
                     sys.stderr.write("  ERROR: %s\n" % str(ex))
         if which == 'rifle' or which == 'all':
             copy('config/rifle.conf', 'rifle.conf')
@@ -313,13 +309,13 @@ class FM(Actions,  # pylint: disable=too-many-instance-attributes
         """returns the path relative to rangers library directory"""
         return os.path.join(ranger.RANGERDIR, *paths)
 
-    def get_directory(self, path):
+    def get_directory(self, path, **dir_kwargs):
         """Get the directory object at the given path"""
         path = os.path.abspath(path)
         try:
             return self.directories[path]
         except KeyError:
-            obj = Directory(path)
+            obj = Directory(path, **dir_kwargs)
             self.directories[path] = obj
             return obj
 
diff --git a/ranger/core/loader.py b/ranger/core/loader.py
index 022dfe6b..59ce7391 100644
--- a/ranger/core/loader.py
+++ b/ranger/core/loader.py
@@ -18,7 +18,7 @@ from ranger.ext.human_readable import human_readable
 try:
     import chardet  # pylint: disable=import-error
     HAVE_CHARDET = True
-except Exception:
+except ImportError:
     HAVE_CHARDET = False
 
 
@@ -40,7 +40,7 @@ class Loadable(object):
     def unpause(self):
         try:
             del self.paused
-        except Exception:
+        except AttributeError:
             pass
 
     def destroy(self):
@@ -74,7 +74,7 @@ class CopyLoader(Loadable, FileManagerAware):  # pylint: disable=too-many-instan
             else:
                 try:
                     fstat = os.stat(fname)
-                except Exception:
+                except OSError:
                     continue
                 size += max(step, math.ceil(fstat.st_size / step) * step)
         return size
@@ -239,7 +239,7 @@ class CommandLoader(  # pylint: disable=too-many-instance-attributes
                 return
             try:
                 self.process.send_signal(20)
-            except Exception:
+            except OSError:
                 pass
             Loadable.pause(self)
             self.signal_emit('pause', process=self.process, loader=self)
@@ -248,7 +248,7 @@ class CommandLoader(  # pylint: disable=too-many-instance-attributes
         if not self.finished and self.paused:
             try:
                 self.process.send_signal(18)
-            except Exception:
+            except OSError:
                 pass
             Loadable.unpause(self)
             self.signal_emit('unpause', process=self.process, loader=self)
@@ -409,9 +409,6 @@ class Loader(FileManagerAware):
                 self.fm.ui.status.request_redraw()
         except StopIteration:
             self._remove_current_process(item)
-        except Exception as err:
-            self.fm.notify(err)
-            self._remove_current_process(item)
 
     def _remove_current_process(self, item):
         item.load_generator = None
diff --git a/ranger/core/main.py b/ranger/core/main.py
index bd837aa1..23debc76 100644
--- a/ranger/core/main.py
+++ b/ranger/core/main.py
@@ -39,7 +39,7 @@ def main(
 
     try:
         locale.setlocale(locale.LC_ALL, '')
-    except Exception:
+    except locale.Error:
         print("Warning: Unable to set locale.  Expect encoding problems.")
 
     # so that programs can know that ranger spawned them:
@@ -66,7 +66,7 @@ def main(
                 fobj = open(fm.confpath('tagged'), 'r', errors='replace')
             else:
                 fobj = open(fm.confpath('tagged'), 'r')
-        except Exception:
+        except OSError:
             pass
         else:
             for line in fobj.readlines():
@@ -164,7 +164,7 @@ def main(
             profile = pstats.Stats(tempfile.gettempdir() + '/ranger_profile', stream=sys.stderr)
         else:
             fm.loop()
-    except Exception:
+    except Exception:  # pylint: disable=broad-except
         import traceback
         crash_traceback = traceback.format_exc()
     except SystemExit as error:
@@ -173,7 +173,7 @@ def main(
         if crash_traceback:
             try:
                 filepath = fm.thisfile.path if fm.thisfile else "None"
-            except Exception:
+            except AttributeError:
                 filepath = "None"
         try:
             fm.ui.destroy()
@@ -187,7 +187,7 @@ def main(
             print("Locale: %s" % '.'.join(str(s) for s in locale.getlocale()))
             try:
                 print("Current file: %s" % filepath)
-            except Exception:
+            except NameError:
                 pass
             print(crash_traceback)
             print("ranger crashed.  "
@@ -318,13 +318,14 @@ def load_settings(  # pylint: disable=too-many-locals,too-many-branches,too-many
         allow_access_to_confdir(ranger.args.confdir, True)
 
         # XXX Load plugins (experimental)
+        plugindir = fm.confpath('plugins')
         try:
-            plugindir = fm.confpath('plugins')
-            plugins = [p[:-3] for p in os.listdir(plugindir)
-                       if p.endswith('.py') and not p.startswith('_')]
-        except Exception:
-            pass
+            plugin_files = os.listdir(plugindir)
+        except OSError:
+            LOG.debug('Unable to access plugin directory: %s', plugindir)
         else:
+            plugins = [p[:-3] for p in plugin_files
+                       if p.endswith('.py') and not p.startswith('_')]
             if not os.path.exists(fm.confpath('plugins', '__init__.py')):
                 LOG.debug("Creating missing '__init__.py' file in plugin folder")
                 fobj = open(fm.confpath('plugins', '__init__.py'), 'w')
@@ -344,7 +345,7 @@ def load_settings(  # pylint: disable=too-many-locals,too-many-branches,too-many
                         module = importlib.import_module('plugins.' + plugin)
                         fm.commands.load_commands_from_module(module)
                     LOG.debug("Loaded plugin '%s'", plugin)
-                except Exception as ex:
+                except Exception as ex:  # pylint: disable=broad-except
                     ex_msg = "Error while loading plugin '{0}'".format(plugin)
                     LOG.error(ex_msg)
                     LOG.exception(ex)
@@ -389,7 +390,7 @@ def allow_access_to_confdir(confdir, allow):
                 print(confdir)
                 print("To run ranger without the need for configuration")
                 print("files, use the --clean option.")
-                raise SystemExit()
+                raise SystemExit
         else:
             LOG.debug("Created config directory '%s'", confdir)
         if confdir not in sys.path:
diff --git a/ranger/core/runner.py b/ranger/core/runner.py
index 6fc7b60d..8c3e3162 100644
--- a/ranger/core/runner.py
+++ b/ranger/core/runner.py
@@ -24,6 +24,7 @@ t: run application in a new terminal window
 
 from __future__ import (absolute_import, division, print_function)
 
+import logging
 import os
 import sys
 from subprocess import Popen, PIPE
@@ -31,6 +32,9 @@ from ranger.ext.get_executables import get_executables, get_term
 from ranger.ext.popen_forked import Popen_forked
 
 
+LOG = logging.getLogger(__name__)
+
+
 # TODO: Remove unused parts of runner.py
 # ALLOWED_FLAGS = 'sdpwcrtSDPWCRT'
 ALLOWED_FLAGS = 'cfrtCFRT'
@@ -81,7 +85,7 @@ class Context(object):  # pylint: disable=too-many-instance-attributes
     def filepaths(self):
         try:
             return [f.path for f in self.files]
-        except Exception:
+        except AttributeError:
             return []
 
     def __iter__(self):
@@ -117,13 +121,15 @@ class Runner(object):  # pylint: disable=too-few-public-methods
             if boolean:
                 try:
                     self.ui.initialize()
-                except Exception:
+                except Exception as ex:  # pylint: disable=broad-except
                     self._log("Failed to initialize UI")
+                    LOG.exception(ex)
             else:
                 try:
                     self.ui.suspend()
-                except Exception:
+                except Exception as ex:  # pylint: disable=broad-except
                     self._log("Failed to suspend UI")
+                    LOG.exception(ex)
 
     def __call__(
             # pylint: disable=too-many-branches,too-many-statements
@@ -235,7 +241,7 @@ class Runner(object):  # pylint: disable=too-few-public-methods
                     Popen_forked(**popen_kws)
                 else:
                     process = Popen(**popen_kws)
-            except Exception as ex:
+            except OSError as ex:
                 error = ex
                 self._log("Failed to run: %s\n%s" % (str(action), str(ex)))
             else:
diff --git a/ranger/core/tab.py b/ranger/core/tab.py
index 26f634da..160f590f 100644
--- a/ranger/core/tab.py
+++ b/ranger/core/tab.py
@@ -130,7 +130,7 @@ class Tab(FileManagerAware, SettingsAware):  # pylint: disable=too-many-instance
 
         try:
             os.chdir(path)
-        except Exception:
+        except OSError:
             return True
         self.path = path
         self.thisdir = new_thisdir
diff --git a/ranger/ext/cached_function.py b/ranger/ext/cached_function.py
index 643b6304..4255443c 100644
--- a/ranger/ext/cached_function.py
+++ b/ranger/ext/cached_function.py
@@ -10,7 +10,7 @@ def cached_function(fnc):
     def inner_cached_function(*args):
         try:
             return cache[args]
-        except Exception:
+        except KeyError:
             value = fnc(*args)
             cache[args] = value
             return value
diff --git a/ranger/ext/direction.py b/ranger/ext/direction.py
index ecde944a..7268d5d6 100644
--- a/ranger/ext/direction.py
+++ b/ranger/ext/direction.py
@@ -38,19 +38,19 @@ class Direction(dict):
     def _get_bool(self, first, second, fallback=None):
         try:
             return self[first]
-        except Exception:
+        except KeyError:
             try:
                 return not self[second]
-            except Exception:
+            except KeyError:
                 return fallback
 
     def _get_direction(self, first, second, fallback=0):
         try:
             return self[first]
-        except Exception:
+        except KeyError:
             try:
                 return -self[second]
-            except Exception:
+            except KeyError:
                 return fallback
 
     def up(self):  # pylint: disable=invalid-name
@@ -98,7 +98,7 @@ class Direction(dict):
         for key in ('up', 'right', 'down', 'left'):
             try:
                 self[key] *= n
-            except Exception:
+            except KeyError:
                 pass
 
     def set(self, n):
diff --git a/ranger/ext/get_executables.py b/ranger/ext/get_executables.py
index 86296a2a..1646cb56 100644
--- a/ranger/ext/get_executables.py
+++ b/ranger/ext/get_executables.py
@@ -37,13 +37,13 @@ def get_executables_uncached(*paths):
     for path in paths:
         try:
             content = listdir(path)
-        except Exception:
+        except OSError:
             continue
         for item in content:
             abspath = path + '/' + item
             try:
                 filestat = stat(abspath)
-            except Exception:
+            except OSError:
                 continue
             if filestat.st_mode & (S_IXOTH | S_IFREG):
                 executables.add(item)
diff --git a/ranger/ext/img_display.py b/ranger/ext/img_display.py
index 1b20cc7b..b4deaa39 100644
--- a/ranger/ext/img_display.py
+++ b/ranger/ext/img_display.py
@@ -35,6 +35,10 @@ W3MIMGDISPLAY_PATHS = [
 ]
 
 
+class ImageDisplayError(Exception):
+    pass
+
+
 class ImgDisplayUnsupportedException(Exception):
     pass
 
@@ -83,9 +87,9 @@ class W3MImageDisplayer(ImageDisplayer):
         for path in paths:
             if path is not None and os.path.exists(path):
                 return path
-        raise RuntimeError("No w3mimgdisplay executable found.  Please set "
-                           "the path manually by setting the %s environment variable.  (see "
-                           "man page)" % W3MIMGDISPLAY_ENV)
+        raise ImageDisplayError("No w3mimgdisplay executable found.  Please set "
+                                "the path manually by setting the %s environment variable.  (see "
+                                "man page)" % W3MIMGDISPLAY_ENV)
 
     def _get_font_dimensions(self):
         # Get the height and width of a character displayed in the terminal in
@@ -110,7 +114,11 @@ class W3MImageDisplayer(ImageDisplayer):
     def draw(self, path, start_x, start_y, width, height):
         if not self.is_initialized or self.process.poll() is not None:
             self.initialize()
-        self.process.stdin.write(self._generate_w3m_input(path, start_x, start_y, width, height))
+        try:
+            input_gen = self._generate_w3m_input(path, start_x, start_y, width, height)
+        except ImageDisplayError:
+            raise
+        self.process.stdin.write(input_gen)
         self.process.stdin.flush()
         self.process.stdout.readline()
 
@@ -146,7 +154,7 @@ class W3MImageDisplayer(ImageDisplayer):
         """
         fontw, fonth = self._get_font_dimensions()
         if fontw == 0 or fonth == 0:
-            raise ImgDisplayUnsupportedException()
+            raise ImgDisplayUnsupportedException
 
         max_width_pixels = max_width * fontw
         max_height_pixels = max_height * fonth - 2
@@ -161,7 +169,7 @@ class W3MImageDisplayer(ImageDisplayer):
         output = self.process.stdout.readline().split()
 
         if len(output) != 2:
-            raise Exception('Failed to execute w3mimgdisplay', output)
+            raise ImageDisplayError('Failed to execute w3mimgdisplay', output)
 
         width = int(output[0])
         height = int(output[1])
@@ -257,13 +265,8 @@ class ITerm2ImageDisplayer(ImageDisplayer, FileManagerAware):
     @staticmethod
     def _encode_image_content(path):
         """Read and encode the contents of path"""
-        fobj = open(path, 'rb')
-        try:
+        with open(path, 'rb') as fobj:
             return base64.b64encode(fobj.read())
-        except Exception:
-            return ""
-        finally:
-            fobj.close()
 
     @staticmethod
     def _get_image_dimensions(path):
@@ -296,7 +299,7 @@ class ITerm2ImageDisplayer(ImageDisplayer, FileManagerAware):
                     size = struct.unpack('>H', file_handle.read(2))[0] - 2
                 file_handle.seek(1, 1)
                 height, width = struct.unpack('>HH', file_handle.read(4))
-            except Exception:
+            except OSError:
                 file_handle.close()
                 return 0, 0
         else:
diff --git a/ranger/ext/keybinding_parser.py b/ranger/ext/keybinding_parser.py
index 34282767..e5fb6a86 100644
--- a/ranger/ext/keybinding_parser.py
+++ b/ranger/ext/keybinding_parser.py
@@ -153,8 +153,11 @@ def _unbind_traverse(pointer, keys, pos=0):
     elif len(keys) == pos + 1:
         try:
             del pointer[keys[pos]]
+        except KeyError:
+            pass
+        try:
             keys.pop()
-        except Exception:
+        except IndexError:
             pass
 
 
@@ -174,7 +177,7 @@ class KeyMaps(dict):
     def _clean_input(self, context, keys):
         try:
             pointer = self[context]
-        except Exception:
+        except KeyError:
             self[context] = pointer = dict()
         if PY3:
             keys = keys.encode('utf-8').decode('latin-1')
@@ -191,7 +194,7 @@ class KeyMaps(dict):
                     pointer = pointer[key]
                 else:
                     pointer[key] = pointer = dict()
-            except Exception:
+            except KeyError:
                 pointer[key] = pointer = dict()
         pointer[last_key] = leaf
 
@@ -202,7 +205,7 @@ class KeyMaps(dict):
         for key in clean_source:
             try:
                 pointer = pointer[key]
-            except Exception:
+            except KeyError:
                 raise KeyError("Tried to copy the keybinding `%s',"
                                " but it was not found." % source)
         self.bind(context, target, copy.deepcopy(pointer))
diff --git a/ranger/ext/logutils.py b/ranger/ext/logutils.py
index b00aa605..38fc7cda 100644
--- a/ranger/ext/logutils.py
+++ b/ranger/ext/logutils.py
@@ -3,10 +3,6 @@ from __future__ import (absolute_import, division, print_function)
 import logging
 from collections import deque
 
-LOG_FORMAT = "[%(levelname)s] %(message)s"
-LOG_FORMAT_EXT = "%(asctime)s,%(msecs)d [%(name)s] |%(levelname)s| %(message)s"
-LOG_DATA_FORMAT = "%H:%M:%S"
-
 
 class QueueHandler(logging.Handler):
     """
@@ -30,11 +26,11 @@ class QueueHandler(logging.Handler):
         self.enqueue(self.format(record))
 
 
-# pylint: disable=invalid-name
-log_queue = deque(maxlen=1000)
-concise_formatter = logging.Formatter(fmt=LOG_FORMAT, datefmt=LOG_DATA_FORMAT)
-extended_formatter = logging.Formatter(fmt=LOG_FORMAT_EXT, datefmt=LOG_DATA_FORMAT)
-# pylint: enable=invalid-name
+QUEUE = deque(maxlen=1000)
+FMT_NORMAL = logging.Formatter(
+    fmt='%(asctime)s %(levelname).4s %(message)s', datefmt='%H:%M:%S')
+FMT_DEBUG = logging.Formatter(
+    fmt='%(asctime)s.%(msecs)03d %(levelname).4s [%(name)s] %(message)s', datefmt='%H:%M:%S')
 
 
 def setup_logging(debug=True, logfile=None):
@@ -57,17 +53,14 @@ def setup_logging(debug=True, logfile=None):
     root_logger = logging.getLogger()
 
     if debug:
-        # print all logging in extended format
         log_level = logging.DEBUG
-        formatter = extended_formatter
+        formatter = FMT_DEBUG
     else:
-        # print only warning and critical message
-        # in a concise format
         log_level = logging.INFO
-        formatter = concise_formatter
+        formatter = FMT_NORMAL
 
     handlers = []
-    handlers.append(QueueHandler(log_queue))
+    handlers.append(QueueHandler(QUEUE))
     if logfile:
         if logfile == '-':
             handlers.append(logging.StreamHandler())
diff --git a/ranger/ext/rifle.py b/ranger/ext/rifle.py
index 551dd288..c6b96f73 100755
--- a/ranger/ext/rifle.py
+++ b/ranger/ext/rifle.py
@@ -94,7 +94,7 @@ def _is_terminal():
         os.ttyname(0)
         os.ttyname(1)
         os.ttyname(2)
-    except Exception:
+    except OSError:
         return False
     return True
 
@@ -168,17 +168,13 @@ class Rifle(object):  # pylint: disable=too-many-instance-attributes
             line = line.strip()
             if line.startswith('#') or line == '':
                 continue
-            try:
-                if self.delimiter1 not in line:
-                    raise Exception("Line without delimiter")
-                tests, command = line.split(self.delimiter1, 1)
-                tests = tests.split(self.delimiter2)
-                tests = tuple(tuple(f.strip().split(None, 1)) for f in tests)
-                command = command.strip()
-                self.rules.append((command, tests))
-            except Exception as ex:
-                self.hook_logger(
-                    "Syntax error in %s line %d (%s)" % (config_file, lineno, str(ex)))
+            if self.delimiter1 not in line:
+                raise ValueError("Line without delimiter")
+            tests, command = line.split(self.delimiter1, 1)
+            tests = tests.split(self.delimiter2)
+            tests = tuple(tuple(f.strip().split(None, 1)) for f in tests)
+            command = command.strip()
+            self.rules.append((command, tests))
         fobj.close()
 
     def _eval_condition(self, condition, files, label):
diff --git a/ranger/ext/shutil_generatorized.py b/ranger/ext/shutil_generatorized.py
index 338ed5b8..f1cf5a46 100644
--- a/ranger/ext/shutil_generatorized.py
+++ b/ranger/ext/shutil_generatorized.py
@@ -95,17 +95,17 @@ def copystat(src, dst):
     if hasattr(os, 'utime'):
         try:
             os.utime(dst, (fstat.st_atime, fstat.st_mtime))
-        except Exception:
+        except OSError:
             pass
     if hasattr(os, 'chmod'):
         try:
             os.chmod(dst, mode)
-        except Exception:
+        except OSError:
             pass
     if hasattr(os, 'chflags') and hasattr(fstat, 'st_flags'):
         try:
             os.chflags(dst, fstat.st_flags)  # pylint: disable=no-member
-        except Exception:
+        except OSError:
             pass
 
 
@@ -182,7 +182,7 @@ def copytree(src, dst,  # pylint: disable=too-many-locals,too-many-branches
     errors = []
     try:
         os.makedirs(dst)
-    except Exception as err:
+    except OSError:
         if not overwrite:
             dst = get_safe_path(dst)
             os.makedirs(dst)
@@ -212,10 +212,10 @@ def copytree(src, dst,  # pylint: disable=too-many-locals,too-many-branches
                 done += n
         # catch the Error from the recursive copytree so that we can
         # continue with other files
-        except Error as err:
-            errors.extend(err.args[0])
-        except EnvironmentError as why:
-            errors.append((srcname, dstname, str(why)))
+        except Error as ex:
+            errors.extend(ex.args[0])
+        except EnvironmentError as ex:
+            errors.append((srcname, dstname, str(ex)))
     try:
         copystat(src, dst)
     except OSError as why:
diff --git a/ranger/ext/signals.py b/ranger/ext/signals.py
index 4d2ee25e..50141565 100644
--- a/ranger/ext/signals.py
+++ b/ranger/ext/signals.py
@@ -137,7 +137,7 @@ class SignalDispatcher(object):
         assert isinstance(weak, bool)
         try:
             handlers = self._signals[signal_name]
-        except Exception:
+        except KeyError:
             handlers = self._signals[signal_name] = []
         nargs = function.__code__.co_argcount
 
@@ -180,13 +180,13 @@ class SignalDispatcher(object):
         try:
             handlers = self._signals[
                 signal_handler._signal_name]  # pylint: disable=protected-access
-        except Exception:
+        except KeyError:
             pass
         else:
+            signal_handler._function = None  # pylint: disable=protected-access
             try:
-                signal_handler._function = None  # pylint: disable=protected-access
                 handlers.remove(signal_handler)
-            except Exception:
+            except IndexError:
                 pass
 
     def signal_garbage_collect(self):
diff --git a/ranger/ext/vcs/vcs.py b/ranger/ext/vcs/vcs.py
index 8b13badd..00f51152 100644
--- a/ranger/ext/vcs/vcs.py
+++ b/ranger/ext/vcs/vcs.py
@@ -9,22 +9,14 @@ import os
 import subprocess
 import threading
 import time
-from logging import getLogger
 
 from ranger.ext import spawn
 
-# Python2 compatibility
+# Python 2 compatibility
 try:
     import queue
 except ImportError:
     import Queue as queue  # pylint: disable=import-error
-try:
-    FileNotFoundError
-except NameError:
-    FileNotFoundError = OSError  # pylint: disable=redefined-builtin
-
-
-LOG = getLogger(__name__)
 
 
 class VcsError(Exception):
@@ -133,7 +125,7 @@ class Vcs(object):  # pylint: disable=too-many-instance-attributes
             else:
                 with open(os.devnull, mode='w') as fd_devnull:
                     subprocess.check_call(cmd, cwd=path, stdout=fd_devnull, stderr=fd_devnull)
-        except (subprocess.CalledProcessError, FileNotFoundError):
+        except (subprocess.CalledProcessError, OSError):
             raise VcsError('{0:s}: {1:s}'.format(str(cmd), path))
 
     def _get_repotype(self, path):
@@ -238,9 +230,8 @@ class VcsRoot(Vcs):  # pylint: disable=abstract-method
             self.branch = self.data_branch()
             self.obj.vcsremotestatus = self.data_status_remote()
             self.obj.vcsstatus = self.data_status_root()
-        except VcsError as error:
-            LOG.exception(error)
-            self.obj.fm.notify('VCS Exception: View log for more info', bad=True)
+        except VcsError as ex:
+            self.obj.fm.notify('VCS Exception: View log for more info', bad=True, exception=ex)
             return False
         self.rootinit = True
         return True
@@ -253,9 +244,8 @@ class VcsRoot(Vcs):  # pylint: disable=abstract-method
             self.status_subpaths = self.data_status_subpaths()
             self.obj.vcsremotestatus = self.data_status_remote()
             self.obj.vcsstatus = self._status_root()
-        except VcsError as error:
-            LOG.exception(error)
-            self.obj.fm.notify('VCS Exception: View log for more info', bad=True)
+        except VcsError as ex:
+            self.obj.fm.notify('VCS Exception: View log for more info', bad=True, exception=ex)
             return False
         self.rootinit = True
         self.updatetime = time.time()
@@ -479,9 +469,8 @@ class VcsThread(threading.Thread):  # pylint: disable=too-many-instance-attribut
                             column.need_redraw = True
                     self.ui.status.need_redraw = True
                     self.ui.redraw()
-            except Exception as error:  # pylint: disable=broad-except
-                LOG.exception(error)
-                self.ui.fm.notify('VCS Exception: View log for more info', bad=True)
+            except Exception as ex:  # pylint: disable=broad-except
+                self.ui.fm.notify('VCS Exception: View log for more info', bad=True, exception=ex)
 
     def pause(self):
         """Pause thread"""
diff --git a/ranger/ext/widestring.py b/ranger/ext/widestring.py
index 23e3d050..ed66f3c7 100644
--- a/ranger/ext/widestring.py
+++ b/ranger/ext/widestring.py
@@ -62,10 +62,7 @@ class WideString(object):  # pylint: disable=too-few-public-methods
         except UnicodeEncodeError:
             # Here I assume that string is a "unicode" object, because why else
             # would str(string) raise a UnicodeEncodeError?
-            try:
-                self.string = string.encode('latin-1', 'ignore')
-            except Exception:
-                self.string = ""
+            self.string = string.encode('latin-1', 'ignore')
         if chars is None:
             self.chars = string_to_charlist(string)
         else:
diff --git a/ranger/gui/ansi.py b/ranger/gui/ansi.py
index 4bfaa0a0..5cb2ab90 100644
--- a/ranger/gui/ansi.py
+++ b/ranger/gui/ansi.py
@@ -51,7 +51,7 @@ def text_with_fg_bg_attr(ansi_text):  # pylint: disable=too-many-branches,too-ma
                         n = int(arg)
                     else:                         # empty code means reset
                         n = 0
-                except Exception:
+                except ValueError:
                     continue
 
                 if n == 0:                        # reset colors and attributes
diff --git a/ranger/gui/color.py b/ranger/gui/color.py
index 7f2b73af..45f983e8 100644
--- a/ranger/gui/color.py
+++ b/ranger/gui/color.py
@@ -30,7 +30,7 @@ def get_color(fg, bg):
         size = len(COLOR_PAIRS)
         try:
             curses.init_pair(size, fg, bg)
-        except Exception:
+        except curses.error:
             # If curses.use_default_colors() failed during the initialization
             # of curses, then using -1 as fg or bg will fail as well, which
             # we need to handle with fallback-defaults:
@@ -41,7 +41,7 @@ def get_color(fg, bg):
 
             try:
                 curses.init_pair(size, fg, bg)
-            except Exception:
+            except curses.error:
                 # If this fails too, colors are probably not supported
                 pass
         COLOR_PAIRS[key] = size
diff --git a/ranger/gui/colorscheme.py b/ranger/gui/colorscheme.py
index c5aa0e31..75a82146 100644
--- a/ranger/gui/colorscheme.py
+++ b/ranger/gui/colorscheme.py
@@ -37,6 +37,10 @@ from ranger.ext.cached_function import cached_function
 from ranger.ext.iter_tools import flatten
 
 
+class ColorSchemeError(Exception):
+    pass
+
+
 class ColorScheme(object):
     """This is the class that colorschemes must inherit from.
 
@@ -96,7 +100,7 @@ def _colorscheme_name_to_class(signal):  # pylint: disable=too-many-branches
     def is_scheme(cls):
         try:
             return issubclass(cls, ColorScheme)
-        except Exception:
+        except TypeError:
             return False
 
     # create ~/.config/ranger/colorschemes/__init__.py if it doesn't exist
@@ -120,7 +124,7 @@ def _colorscheme_name_to_class(signal):  # pylint: disable=too-many-branches
             signal.value = signal.previous
         else:
             signal.value = ColorScheme()
-        raise Exception("Cannot locate colorscheme `%s'" % scheme_name)
+        raise ColorSchemeError("Cannot locate colorscheme `%s'" % scheme_name)
     else:
         if usecustom:
             allow_access_to_confdir(ranger.args.confdir, True)
@@ -137,7 +141,7 @@ def _colorscheme_name_to_class(signal):  # pylint: disable=too-many-branches
                     signal.value = var()
                     break
             else:
-                raise Exception("The module contains no valid colorscheme!")
+                raise ColorSchemeError("The module contains no valid colorscheme!")
 
 
 def get_all_colorschemes():
diff --git a/ranger/gui/curses_shortcuts.py b/ranger/gui/curses_shortcuts.py
index 2c411058..648292c5 100644
--- a/ranger/gui/curses_shortcuts.py
+++ b/ranger/gui/curses_shortcuts.py
@@ -36,13 +36,13 @@ class CursesShortcuts(SettingsAware):
 
         try:
             self.win.addstr(*args)
-        except Exception:
+        except curses.error:
             if len(args) > 1:
                 self.win.move(y, x)
 
                 try:
                     self.win.addstr(*_fix_surrogates(args))
-                except Exception:
+                except curses.error:
                     pass
 
     def addnstr(self, *args):
@@ -50,13 +50,13 @@ class CursesShortcuts(SettingsAware):
 
         try:
             self.win.addnstr(*args)
-        except Exception:
+        except curses.error:
             if len(args) > 2:
                 self.win.move(y, x)
 
                 try:
                     self.win.addnstr(*_fix_surrogates(args))
-                except Exception:
+                except curses.error:
                     pass
 
     def addch(self, *args):
@@ -64,7 +64,7 @@ class CursesShortcuts(SettingsAware):
             args = [args[1], args[0]] + list(args[2:])
         try:
             self.win.addch(*args)
-        except Exception:
+        except curses.error:
             pass
 
     def color(self, *keys):
diff --git a/ranger/gui/displayable.py b/ranger/gui/displayable.py
index 2ce23165..926fd857 100644
--- a/ranger/gui/displayable.py
+++ b/ranger/gui/displayable.py
@@ -3,6 +3,8 @@
 
 from __future__ import (absolute_import, division, print_function)
 
+import curses
+
 from ranger.core.shared import FileManagerAware
 from ranger.gui.curses_shortcuts import CursesShortcuts
 
@@ -176,13 +178,13 @@ class Displayable(  # pylint: disable=too-many-instance-attributes
             window_is_cleared = True
             try:
                 self.win.resize(hei, wid)
-            except Exception:
+            except curses.error:
                 # Not enough space for resizing...
                 try:
                     self.win.mvderwin(0, 0)
                     do_move = True
                     self.win.resize(hei, wid)
-                except Exception:
+                except curses.error:
                     pass
                     # raise ValueError("Resizing Failed!")
 
@@ -195,7 +197,7 @@ class Displayable(  # pylint: disable=too-many-instance-attributes
             # log("moving " + str(self))
             try:
                 self.win.mvderwin(y, x)
-            except Exception:
+            except curses.error:
                 pass
 
             self.paryx = self.win.getparyx()
diff --git a/ranger/gui/mouse_event.py b/ranger/gui/mouse_event.py
index 160351d3..0eccbf57 100644
--- a/ranger/gui/mouse_event.py
+++ b/ranger/gui/mouse_event.py
@@ -32,7 +32,7 @@ class MouseEvent(object):
         """Returns whether the mouse key n is pressed"""
         try:
             return (self.bstate & MouseEvent.PRESSED[n]) != 0
-        except Exception:
+        except IndexError:
             return False
 
     def mouse_wheel_direction(self):
diff --git a/ranger/gui/ui.py b/ranger/gui/ui.py
index 6c84580b..20e01536 100644
--- a/ranger/gui/ui.py
+++ b/ranger/gui/ui.py
@@ -91,12 +91,12 @@ class UI(  # pylint: disable=too-many-instance-attributes,too-many-public-method
         curses.halfdelay(20)
         try:
             curses.curs_set(int(bool(self.settings.show_cursor)))
-        except Exception:
+        except curses.error:
             pass
         curses.start_color()
         try:
             curses.use_default_colors()
-        except Exception:
+        except curses.error:
             pass
 
         self.settings.signal_bind('setopt.mouse_enabled', _setup_mouse)
@@ -130,7 +130,7 @@ class UI(  # pylint: disable=too-many-instance-attributes,too-many-public-method
         curses.echo()
         try:
             curses.curs_set(1)
-        except Exception:
+        except curses.error:
             pass
         if self.settings.mouse_enabled:
             _setup_mouse(dict(value=False))
@@ -346,14 +346,16 @@ class UI(  # pylint: disable=too-many-instance-attributes,too-many-public-method
             try:
                 fixed_cwd = cwd.encode('utf-8', 'surrogateescape'). \
                     decode('utf-8', 'replace')
-                sys.stdout.write("%sranger:%s%s" % (
+                fmt_tup = (
                     curses.tigetstr('tsl').decode('latin-1'),
                     fixed_cwd,
                     curses.tigetstr('fsl').decode('latin-1'),
-                ))
-                sys.stdout.flush()
-            except Exception:
+                )
+            except UnicodeError:
                 pass
+            else:
+                sys.stdout.write("%sranger:%s%s" % fmt_tup)
+                sys.stdout.flush()
 
         self.win.refresh()
 
diff --git a/ranger/gui/widgets/browsercolumn.py b/ranger/gui/widgets/browsercolumn.py
index 6f890dab..b0435785 100644
--- a/ranger/gui/widgets/browsercolumn.py
+++ b/ranger/gui/widgets/browsercolumn.py
@@ -5,6 +5,7 @@
 
 from __future__ import (absolute_import, division, print_function)
 
+import curses
 import stat
 from time import time
 from os.path import splitext
@@ -76,13 +77,14 @@ class BrowserColumn(Pager):  # pylint: disable=too-many-instance-attributes
                 elif event.pressed(3):
                     try:
                         clicked_file = self.target.files[index]
+                    except IndexError:
+                        pass
+                    else:
                         if clicked_file.is_directory:
                             self.fm.enter_dir(clicked_file.path)
                         elif self.level == 0:
                             self.fm.thisdir.move_to_obj(clicked_file)
                             self.fm.execute_file(clicked_file)
-                    except Exception:
-                        pass
 
         else:
             if self.level > 0 and not direction:
@@ -102,7 +104,7 @@ class BrowserColumn(Pager):  # pylint: disable=too-many-instance-attributes
         """
         try:
             self.win.move(line, 0)
-        except Exception:
+        except curses.error:
             return
         for entry in commands:
             text, attr = entry
@@ -363,11 +365,10 @@ class BrowserColumn(Pager):  # pylint: disable=too-many-instance-attributes
             predisplay_left += textstring
             space -= textstringlen
 
+            assert space >= 0, "Error: there is not enough space to write the text. " \
+                "I have computed spaces wrong."
             if space > 0:
                 predisplay_left.append([' ' * space, []])
-            elif space < 0:
-                raise Exception("Error: there is not enough space to write "
-                                "the text. I have computed spaces wrong.")
 
             # Computing display data. Now we compute the display_data list
             # ready to display in curses. It is a list of lists [string, attr]
diff --git a/ranger/gui/widgets/console.py b/ranger/gui/widgets/console.py
index 0fa941d4..a2e058ad 100644
--- a/ranger/gui/widgets/console.py
+++ b/ranger/gui/widgets/console.py
@@ -42,8 +42,8 @@ class Console(Widget):  # pylint: disable=too-many-instance-attributes,too-many-
             self.historypath = self.fm.confpath('history')
             try:
                 fobj = open(self.historypath, 'r')
-            except Exception:
-                pass
+            except OSError as ex:
+                self.fm.notify('Unable to read history file', bad=True, exception=ex)
             else:
                 for line in fobj:
                     self.history.add(line[:-1])
@@ -70,8 +70,8 @@ class Console(Widget):  # pylint: disable=too-many-instance-attributes,too-many-
         if self.historypath:
             try:
                 fobj = open(self.historypath, 'w')
-            except Exception:
-                pass
+            except OSError as ex:
+                self.fm.notify('Unable to write history file', bad=True, exception=ex)
             else:
                 for entry in self.history_backup:
                     try:
@@ -102,13 +102,13 @@ class Console(Widget):  # pylint: disable=too-many-instance-attributes,too-many-
         if self.question_queue:
             try:
                 move(self.y, len(self.question_queue[0][0]))
-            except Exception:
+            except curses.error:
                 pass
         else:
             try:
                 pos = uwid(self.line[0:self.pos]) + len(self.prompt)
                 move(self.y, self.x + min(self.wid - 1, pos))
-            except Exception:
+            except curses.error:
                 pass
 
     def open(self, string='', prompt=None, position=None):
@@ -121,7 +121,7 @@ class Console(Widget):  # pylint: disable=too-many-instance-attributes,too-many-
         if self.last_cursor_mode is None:
             try:
                 self.last_cursor_mode = curses.curs_set(1)
-            except Exception:
+            except curses.error:
                 pass
         self.allow_close = False
         self.tab_deque = None
@@ -150,14 +150,11 @@ class Console(Widget):  # pylint: disable=too-many-instance-attributes,too-many-
         if trigger_cancel_function:
             cmd = self._get_cmd(quiet=True)
             if cmd:
-                try:
-                    cmd.cancel()
-                except Exception as error:
-                    self.fm.notify(error)
+                cmd.cancel()
         if self.last_cursor_mode is not None:
             try:
                 curses.curs_set(self.last_cursor_mode)
-            except Exception:
+            except curses.error:
                 pass
             self.last_cursor_mode = None
         self.fm.hide_console_info()
@@ -446,8 +443,6 @@ class Console(Widget):  # pylint: disable=too-many-instance-attributes,too-many-
             if not quiet:
                 error = "Command not found: `%s'" % self.line.split()[0]
                 self.fm.notify(error, bad=True)
-        except Exception:
-            return None
         else:
             return command_class(self.line)
 
diff --git a/ranger/gui/widgets/pager.py b/ranger/gui/widgets/pager.py
index 2ecf65c0..bf7ca5f4 100644
--- a/ranger/gui/widgets/pager.py
+++ b/ranger/gui/widgets/pager.py
@@ -5,6 +5,8 @@
 
 from __future__ import (absolute_import, division, print_function)
 
+import curses
+
 from ranger.gui import ansi
 from ranger.ext.direction import Direction
 from ranger.ext.img_display import ImgDisplayUnsupportedException
@@ -99,7 +101,7 @@ class Pager(Widget):  # pylint: disable=too-many-instance-attributes
                                              self.wid, self.hei)
             except ImgDisplayUnsupportedException:
                 self.fm.settings.preview_images = False
-            except Exception as ex:
+            except Exception as ex:  # pylint: disable=broad-except
                 self.fm.notify(ex, bad=True)
             else:
                 self.image_drawn = True
@@ -110,7 +112,7 @@ class Pager(Widget):  # pylint: disable=too-many-instance-attributes
         elif self.markup == 'ansi':
             try:
                 self.win.move(i, 0)
-            except Exception:
+            except curses.error:
                 pass
             else:
                 for chunk in ansi.text_with_fg_bg_attr(line):
diff --git a/ranger/gui/widgets/statusbar.py b/ranger/gui/widgets/statusbar.py
index c9133415..8e9e259e 100644
--- a/ranger/gui/widgets/statusbar.py
+++ b/ranger/gui/widgets/statusbar.py
@@ -10,6 +10,7 @@ such as the space used by all the files in this directory.
 
 from __future__ import (absolute_import, division, print_function)
 
+import curses
 import os
 from os import getuid, readlink
 from pwd import getpwuid
@@ -77,7 +78,7 @@ class StatusBar(Widget):  # pylint: disable=too-many-instance-attributes
             self.fm.thisfile.load_if_outdated()
             try:
                 ctime = self.fm.thisfile.stat.st_ctime
-            except Exception:
+            except AttributeError:
                 ctime = -1
         else:
             ctime = -1
@@ -131,7 +132,7 @@ class StatusBar(Widget):  # pylint: disable=too-many-instance-attributes
 
             try:
                 self.addnstr(0, starting_point, string, space_left)
-            except Exception:
+            except curses.error:
                 break
             space_left -= len(string)
             starting_point += len(string)
@@ -150,7 +151,7 @@ class StatusBar(Widget):  # pylint: disable=too-many-instance-attributes
                 return
         try:
             stat = target.stat
-        except Exception:
+        except AttributeError:
             return
         if stat is None:
             return
@@ -172,7 +173,7 @@ class StatusBar(Widget):  # pylint: disable=too-many-instance-attributes
             how = 'good' if target.exists else 'bad'
             try:
                 dest = readlink(target.path)
-            except Exception:
+            except OSError:
                 dest = '?'
             left.add(' -> ' + dest, 'link', how)
         else:
diff --git a/ranger/gui/widgets/titlebar.py b/ranger/gui/widgets/titlebar.py
index 425dd64c..a2c54516 100644
--- a/ranger/gui/widgets/titlebar.py
+++ b/ranger/gui/widgets/titlebar.py
@@ -74,10 +74,7 @@ class TitleBar(Widget):
                 elif i == 2:
                     self.fm.enter_dir("/")
                 else:
-                    try:
-                        self.fm.enter_dir(part.directory)
-                    except Exception:
-                        pass
+                    self.fm.enter_dir(part.directory)
                 return True
         return False
 
diff --git a/ranger/gui/widgets/view_base.py b/ranger/gui/widgets/view_base.py
index 8807628b..5cdb2615 100644
--- a/ranger/gui/widgets/view_base.py
+++ b/ranger/gui/widgets/view_base.py
@@ -52,15 +52,15 @@ class ViewBase(Widget, DisplayableContainer):  # pylint: disable=too-many-instan
         if self.pager is not None and self.pager.visible:
             try:
                 self.fm.ui.win.move(self.main_column.y, self.main_column.x)
-            except Exception:
+            except curses.error:
                 pass
         else:
+            col_x = self.main_column.x
+            col_y = self.main_column.y + self.main_column.target.pointer \
+                - self.main_column.scroll_begin
             try:
-                col_x = self.main_column.x
-                col_y = self.main_column.y + self.main_column.target.pointer \
-                    - self.main_column.scroll_begin
                 self.fm.ui.win.move(col_y, col_x)
-            except Exception:
+            except curses.error:
                 pass
 
     def _draw_bookmarks(self):
@@ -127,7 +127,7 @@ class ViewBase(Widget, DisplayableContainer):  # pylint: disable=too-many-instan
         self.addnstr(ystart - 1, 0, "key          command".ljust(self.wid), self.wid)
         try:
             self.win.chgat(ystart - 1, 0, curses.A_UNDERLINE)
-        except Exception:
+        except curses.error:
             pass
         whitespace = " " * self.wid
         i = ystart
diff --git a/ranger/gui/widgets/view_miller.py b/ranger/gui/widgets/view_miller.py
index 569b93fa..e79ade79 100644
--- a/ranger/gui/widgets/view_miller.py
+++ b/ranger/gui/widgets/view_miller.py
@@ -155,7 +155,7 @@ class ViewMiller(ViewBase):  # pylint: disable=too-many-ancestors,too-many-insta
                 self.addch(0, x, curses.ACS_TTEE, 0)
                 self.addch(y, x, curses.ACS_BTEE, 0)
                 # pylint: enable=no-member
-            except Exception:
+            except curses.error:
                 # in case it's off the boundaries
                 pass
 
@@ -186,7 +186,7 @@ class ViewMiller(ViewBase):  # pylint: disable=too-many-ancestors,too-many-insta
                     self.fm.settings.use_preview_script:
                 try:
                     result = not self.fm.previews[target.realpath]['foundpreview']
-                except Exception:
+                except KeyError:
                     return self.old_collapse
 
         self.old_collapse = result