Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
The blame_whole_file setting is pointless and broken
  • Loading branch information
kemayo committed Feb 11, 2014
1 parent 5a9858e commit 7782758
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 7 deletions.
5 changes: 0 additions & 5 deletions Git.sublime-settings
Expand Up @@ -12,11 +12,6 @@
// use the panel for diff output, rather than a new scratch window (new tab)
,"diff_panel": false

// affects blame command when no selection is made
// true: blame whole file
// false: blame only current line
,"blame_whole_file": true

// If you'd rather have your status command open files instead of show you a
// diff, set this to true. You can still do `Git: Status` followed by
// 'Git: Diff Current File' to get a file diff
Expand Down
3 changes: 1 addition & 2 deletions history.py
Expand Up @@ -13,9 +13,8 @@ def run(self, edit):
# -C: retain blame when copying lines between files
command = ['git', 'blame', '-w', '-M', '-C']

s = sublime.load_settings("Git.sublime-settings")
selection = self.view.sel()[0] # todo: multi-select support?
if not selection.empty() or not s.get('blame_whole_file'):
if not selection.empty():
# just the lines we have a selection on
begin_line, begin_column = self.view.rowcol(selection.begin())
end_line, end_column = self.view.rowcol(selection.end())
Expand Down

0 comments on commit 7782758

Please sign in to comment.