Comment

Author: frou

src/blame.py

@@ -12,7 +12,7 @@
 
 class Blame(BaseBlame, sublime_plugin.TextCommand):
 
-    # Overrides begin ------------------------------------------------------------------
+    # Overrides (TextCommand) ----------------------------------------------------------
 
     def __init__(self, view):
         super().__init__(view)
@@ -114,6 +114,8 @@ def run(self, edit, prevving=False, fixed_row_num=None, sha_skip_list=[]):
 
         self.phantom_set.update(phantoms)
 
+    # Overrides (BaseBlame) ------------------------------------------------------------
+
     def _view(self):
         return self.view
 

src/blame_all.py

@@ -16,7 +16,7 @@ class BlameShowAll(BaseBlame, sublime_plugin.TextCommand):
 
     HORIZONTAL_SCROLL_DELAY_MS = 100
 
-    # Overrides begin ------------------------------------------------------------------
+    # Overrides (TextCommand) ----------------------------------------------------------
 
     def __init__(self, view):
         super().__init__(view)
@@ -82,6 +82,8 @@ def run(self, edit):
         # Bring the phantoms into view without the user needing to manually scroll left.
         self.horizontal_scroll_to_limit(left=True)
 
+    # Overrides (BaseBlame) ------------------------------------------------------------
+
     def _view(self):
         return self.view
 
@@ -92,6 +94,7 @@ def close_by_user_request(self):
         self.view.run_command("blame_erase_all")
 
     def recurse(self, *args, **kwargs):
+        # BlameAll doesn't need to rerun itself.
         raise NotImplementedError()
 
     # Overrides end --------------------------------------------------------------------

src/blame_inline.py

@@ -18,7 +18,7 @@ class BlameInlineListener(BaseBlame, sublime_plugin.ViewEventListener):
 
     pkg_setting_callback_added = False
 
-    # Overrides begin ------------------------------------------------------------------
+    # Overrides (ViewEventListener) ----------------------------------------------------
 
     def __init__(self, view):
         super().__init__(view)
@@ -49,6 +49,8 @@ def on_post_save_async(self):
         # no call to on_selection_modified_async after save.
         self.show_inline_blame_handler()
 
+    # Overrides (BaseBlame) ------------------------------------------------------------
+
     def extra_cli_args(self, line_num):
         args = ["-L", "{0},{0}".format(line_num), "--date=relative"]
         return args
@@ -57,10 +59,11 @@ def _view(self):
         return self.view
 
     def close_by_user_request(self):
-        # Inline Blame phantoms do not have a user-accessible close UI.
+        # Inline Blame phantoms doesn't have a user-accessible close UI.
         raise NotImplementedError()
 
     def recurse(self, *args, **kwargs):
+        # Inline Blame doesn't need to rerun itself.
         raise NotImplementedError()
 
     # Overrides end --------------------------------------------------------------------