Make Branches.Remove() accept a Branch as its target

Author: nulltoken

LibGit2Sharp.Tests/BranchFixture.cs

@@ -454,11 +454,27 @@ private void AssertRemoval(string branchName, bool isRemote, bool shouldPrevious
         [Theory]
         [InlineData("i-do-numbers", false)]
         [InlineData("origin/br2", true)]
-        public void CanRemoveAnExistingBranch(string branchName, bool isRemote)
+        public void CanRemoveAnExistingNamedBranch(string branchName, bool isRemote)
         {
             AssertRemoval(branchName, isRemote, true);
         }
 
+        [Theory]
+        [InlineData("i-do-numbers")]
+        [InlineData("origin/br2")]
+        public void CanRemoveAnExistingBranch(string branchName)
+        {
+            TemporaryCloneOfTestRepo path = BuildTemporaryCloneOfTestRepo(StandardTestRepoPath);
+
+            using (var repo = new Repository(path.RepositoryPath))
+            {
+                Branch curBranch = repo.Branches[branchName];
+
+                repo.Branches.Remove(curBranch);
+                Branch branch = repo.Branches[branchName];
+                Assert.Null(branch);
+            }
+        }
 
         [Theory]
         [InlineData("I-donot-exist", false)]

LibGit2Sharp/BranchCollection.cs

@@ -192,6 +192,17 @@ public virtual void Remove(string name, bool isRemote = false)
             Proxy.git_branch_delete(repo.Handle, name, isRemote ? GitBranchType.GIT_BRANCH_REMOTE : GitBranchType.GIT_BRANCH_LOCAL);
         }
 
+        /// <summary>
+        ///   Deletes the specified branch.
+        /// </summary>
+        /// <param name = "branch">The branch to delete.</param>
+        public virtual void Remove(Branch branch)
+        {
+            Ensure.ArgumentNotNull(branch, "branch");
+
+            Remove(branch.Name, branch.IsRemote);
+        }
+
         /// <summary>
         ///   Deletes the branch with the specified name.
         /// </summary>