Fixed a stupid bug that ignored only one modification

Getty · Jun 5, 2012 · 1f60d29 · 1f60d29
1 parent 6c9acef
commit 1f60d29
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git a/lib/Git/Wrapper.pm b/lib/Git/Wrapper.pm
@@ -218,6 +218,7 @@ sub log {
 
     if ($raw) {
       my @modifications;
+
       while(@out and $out[0] =~ m/^\:(\d{6}) (\d{6}) (\w{7})\.\.\. (\w{7})\.\.\. (\w{1})\t(.*)$/) {
         push @modifications, Git::Wrapper::File::RawModification->new($6,$5,$1,$2,$3,$4);
         shift @out;

diff --git a/lib/Git/Wrapper/Log.pm b/lib/Git/Wrapper/Log.pm
@@ -20,7 +20,15 @@ sub id { shift->{id} }
 
 sub attr { shift->{attr} }
 
-sub modifications { my $self = shift; @_ > 1 ? ($self->{modifications} = [@_]) : @{$self->{modifications}} }
+sub modifications {
+	my $self = shift;
+	if (@_ > 0) {
+		$self->{modifications} = [@_];
+		return scalar @{$self->{modifications}};
+	} else {
+		return @{$self->{modifications}};
+	}
+}
 
 sub message { @_ > 1 ? ($_[0]->{message} = $_[1]) : $_[0]->{message} }