From 433a44f9bdbd8d01254559815b5dc349d44e46f8 Mon Sep 17 00:00:00 2001
From: hut <hut@hut.pm>
Date: Sun, 16 Jul 2017 05:49:09 +0200
Subject: container.fsobject: faster loading with lazy_property

---
 ranger/container/fsobject.py | 56 ++++++++++++++++++++++++--------------------
 1 file changed, 30 insertions(+), 26 deletions(-)

diff --git a/ranger/container/fsobject.py b/ranger/container/fsobject.py
index 6320d3d7..3c4e7aaa 100644
--- a/ranger/container/fsobject.py
+++ b/ranger/container/fsobject.py
@@ -51,9 +51,6 @@ class FileSystemObject(  # pylint: disable=too-many-instance-attributes
         FileManagerAware, SettingsAware):
     basename = None
     relative_path = None
-    relative_path_lower = None
-    dirname = None
-    extension = None
     infostring = None
     path = None
     permissions = None
@@ -91,13 +88,42 @@ class FileSystemObject(  # pylint: disable=too-many-instance-attributes
     vcsstatus = None
     vcsremotestatus = None
 
-    linemode = DEFAULT_LINEMODE
     linemode_dict = dict(
         (linemode.name, linemode()) for linemode in
         [DefaultLinemode, TitleLinemode, PermissionsLinemode, FileInfoLinemode,
          MtimeLinemode, SizeMtimeLinemode]
     )
 
+    @lazy_property
+    def extension(self):
+        try:
+            lastdot = self.basename.rindex('.') + 1
+            return self.basename[lastdot:].lower()
+        except ValueError:
+            return None
+
+    @lazy_property
+    def relative_path_lower(self):
+        return self.relative_path.lower()
+
+    @lazy_property
+    def linemode(self):
+        # Set the line mode from fm.default_linemodes
+        for method, argument, linemode in self.fm.default_linemodes:
+            if linemode in self.linemode_dict:
+                if method == "always":
+                    return linemode
+                if method == "path" and argument.search(self.path):
+                    return linemode
+                if method == "tag" and self.realpath in self.fm.tags and \
+                        self.fm.tags.marker(self.realpath) in argument:
+                    return linemode
+        return DEFAULT_LINEMODE
+
+    @lazy_property
+    def dirname(self):
+        return dirname(self.path)
+
     def __init__(self, path, preload=None, path_is_abs=False, basename_is_rel_to=None):
         if not path_is_abs:
             path = abspath(path)
@@ -107,31 +133,9 @@ class FileSystemObject(  # pylint: disable=too-many-instance-attributes
             self.relative_path = self.basename
         else:
             self.relative_path = relpath(path, basename_is_rel_to)
-        self.relative_path_lower = self.relative_path.lower()
-        self.extension = splitext(self.basename)[1].lstrip(extsep) or None
-        self.dirname = dirname(path)
         self.preload = preload
         self.display_data = {}
 
-        try:
-            lastdot = self.basename.rindex('.') + 1
-            self.extension = self.basename[lastdot:].lower()
-        except ValueError:
-            self.extension = None
-
-        # Set the line mode from fm.default_linemodes
-        for method, argument, linemode in self.fm.default_linemodes:
-            if linemode in self.linemode_dict:
-                if method == "always":
-                    self.linemode = linemode
-                    break
-                if method == "path" and argument.search(path):
-                    self.linemode = linemode
-                    break
-                if method == "tag" and self.realpath in self.fm.tags and \
-                        self.fm.tags.marker(self.realpath) in argument:
-                    self.linemode = linemode
-                    break
 
     def __repr__(self):
         return "<{0} {1}>".format(self.__class__.__name__, self.path)
-- 
cgit 1.4.1-2-gfad0