Changed call style for Git.blame and Git::Blame.new to allow passing options hash (don't know how I missed that the first time around... options are pretty useless if you don't have a way of setting them :)

sandbender · sandbender · commit 4583bfae4dcc · 2010-09-05T14:55:12.000-07:00
diff --git a/lib/git/base.rb b/lib/git/base.rb
@@ -176,8 +176,8 @@ def status
     end
 
     # return a Git::Blame object
-    def blame(file = '', start = '', fin = '')
-      Git::Blame.new(self, file, start, fin)
+    def blame(file = '', opts = {})
+      Git::Blame.new(self, file, opts)
     end
         
     # returns a Git::Branches object of all the Git::Branch objects for this repo
diff --git a/lib/git/blame.rb b/lib/git/blame.rb
@@ -3,10 +3,10 @@ module Git
   class Blame
     include Enumerable
     
-    def initialize(base, file = '', start = '', fin = '')
+    def initialize(base, file = '', opts = {})
       @base = base
 
-      construct_blame(file, start, fin)
+      construct_blame(file, opts)
     end
 
     # Todo - factor this in to BlameLine instead, and have this loop?    
@@ -79,10 +79,12 @@ def initialize(line, hash)
 
 
       # This will run the blame (via our lib.rb), and parse the porcelain-formatted blame output into BlameLine objects
-      def construct_blame(file = '', start = '', fin = '')
+      def construct_blame(file = '', opts = {})
         @lines = {}
 
-        lines = @base.lib.blame({:file => file, :start => start, :fin => fin})
+        opts[:file] = file
+
+        lines = @base.lib.blame(opts)
 
         parsed_lines = {}
         commits = {}
