Bind git_commit_tree

Author: alexcrichton

libgit2-sys/lib.rs

@@ -1240,7 +1240,7 @@ extern {
     pub fn git_commit_time(commit: *const git_commit) -> git_time_t;
     pub fn git_commit_time_offset(commit: *const git_commit) -> c_int;
     pub fn git_commit_tree(tree_out: *mut *mut git_tree,
-                           commit: *const git_commit) -> c_uint;
+                           commit: *const git_commit) -> c_int;
     pub fn git_commit_tree_id(commit: *const git_commit) -> *const git_oid;
     pub fn git_commit_amend(id: *mut git_oid,
                             commit_to_amend: *const git_commit,

src/commit.rs

@@ -49,11 +49,20 @@ impl<'a> Commit<'a> {
 
     /// Get the id of the tree pointed to by this commit.
     ///
-    /// No attempts are made to fetch an object from the
+    /// No attempts are made to fetch an object from the ODB.
     pub fn tree_id(&self) -> Oid {
         unsafe { Oid::from_raw(raw::git_commit_tree_id(&*self.raw)) }
     }
 
+    /// Get the tree pointed to by a commit.
+    pub fn tree(&self) -> Result<Tree, Error> {
+        let mut ret = 0 as *mut raw::git_tree;
+        unsafe {
+            try_call!(raw::git_commit_tree(&mut ret, &*self.raw));
+            Ok(Tree::from_raw(ret))
+        }
+    }
+
     /// Get access to the underlying raw pointer.
     pub fn raw(&self) -> *mut raw::git_commit { self.raw }
 
@@ -298,6 +307,7 @@ mod tests {
         commit.message_encoding();
         commit.summary().unwrap();
         commit.tree_id();
+        commit.tree().unwrap();
         assert_eq!(commit.parents().count(), 0);
 
         assert_eq!(commit.author().name(), Some("name"));

src/repo.rs

@@ -783,7 +783,7 @@ impl Repository {
         let mut raw = 0 as *mut raw::git_tree;
         unsafe {
             try_call!(raw::git_tree_lookup(&mut raw, self.raw(), oid.raw()));
-            Ok(Tree::from_raw(self, raw))
+            Ok(Tree::from_raw(raw))
         }
     }
 

src/tree.rs

@@ -30,8 +30,7 @@ impl<'repo> Tree<'repo> {
     ///
     /// This method is unsafe as there is no guarantee that `raw` is a valid
     /// pointer.
-    pub unsafe fn from_raw(_repo: &Repository,
-                           raw: *mut raw::git_tree) -> Tree {
+    pub unsafe fn from_raw(raw: *mut raw::git_tree) -> Tree<'repo> {
         Tree {
             raw: raw,
             marker1: marker::ContravariantLifetime,