Corrected a problem with commits_between returning None.

This was caused by calling reverse on the list, which reverses the list in
place and returns None. We really need tests for this.

2 files changed, 9 insertions, 2 deletions

diff --git a/CHANGES b/CHANGES
index 0d236f3d..2826c884 100644
--- a/CHANGES
+++ b/CHANGES
@@ -40,6 +40,13 @@ Head
 * Corrected problem where branches was only returning the last path component
   instead of the entire path component following refs/heads/.
 
+Repo
+----
+* Modified the gzip archive creation to use the python gzip module.
+
+* Corrected ``commits_between`` always returning None instead of the reversed
+  list.
+
 
 0.1.5
 =====
diff --git a/lib/git/repo.py b/lib/git/repo.py
index da18208c..a714eea1 100644
--- a/lib/git/repo.py
+++ b/lib/git/repo.py
@@ -138,12 +138,12 @@ class Repo(object):
             is the branch/commit name of the older item
 
         ``path``
-            is an optinal path
+            is an optional path
 
         Returns
             ``git.Commit[]``
         """
-        return Commit.find_all(self, "%s..%s" % (frm, to), path).reverse()
+        return reversed(Commit.find_all(self, "%s..%s" % (frm, to)))
 
     def commits_since(self, start='master', path='', since='1970-01-01'):
         """