commands: rename patch remove to patch drop

Rename the :patch remove command to :patch drop to better express the
this operation is the counter-part to :patch apply.

Signed-off-by: Koni Marti <koni.marti@gmail.com>
Acked-by: Robin Jarry <robin@jarry.cc>

5 files changed, 21 insertions, 21 deletions

diff --git a/lib/pama/remove.go b/lib/pama/drop.go
index d74874a1..595bc87d 100644
--- a/lib/pama/remove.go
+++ b/lib/pama/drop.go
@@ -7,7 +7,7 @@ import (
 	"git.sr.ht/~rjarry/aerc/log"
 )
 
-func (m PatchManager) RemovePatch(patch string) error {
+func (m PatchManager) DropPatch(patch string) error {
 	p, err := m.CurrentProject()
 	if err != nil {
 		return err
@@ -31,7 +31,7 @@ func (m PatchManager) RemovePatch(patch string) error {
 	for _, c := range p.Commits {
 		if !rc.Exists(c.ID) {
 			log.Errorf("failed to find commit. %v", c)
-			return fmt.Errorf("Cannot remove patch. " +
+			return fmt.Errorf("Cannot drop patch. " +
 				"Please rebase first with ':patch rebase'")
 		}
 		if c.Tag == patch {
@@ -44,18 +44,18 @@ func (m PatchManager) RemovePatch(patch string) error {
 		commitID := toRemove[i].ID
 		beforeIDs, err := rc.History(commitID)
 		if err != nil {
-			log.Errorf("failed to remove %v (commits before): %v", toRemove[i], err)
+			log.Errorf("failed to drop %v (commits before): %v", toRemove[i], err)
 			continue
 		}
-		err = rc.Remove(commitID)
+		err = rc.Drop(commitID)
 		if err != nil {
-			log.Errorf("failed to remove %v (remove): %v", toRemove[i], err)
+			log.Errorf("failed to drop %v: %v", toRemove[i], err)
 			continue
 		}
 		removed[commitID] = struct{}{}
 		afterIDs, err := rc.History(p.Base.ID)
 		if err != nil {
-			log.Errorf("failed to remove %v (commits after): %v", toRemove[i], err)
+			log.Errorf("failed to drop %v (commits after): %v", toRemove[i], err)
 			continue
 		}
 		afterIDs = afterIDs[len(afterIDs)-len(beforeIDs):]
@@ -77,7 +77,7 @@ func (m PatchManager) RemovePatch(patch string) error {
 	}
 
 	if len(removed) < len(toRemove) {
-		return fmt.Errorf("Failed to remove commits. Removed %d of %d.",
+		return fmt.Errorf("Failed to drop commits. Dropped %d of %d.",
 			len(removed), len(toRemove))
 	}
 
diff --git a/lib/pama/remove_test.go b/lib/pama/drop_test.go
index c9ce6c65..84bc2864 100644
--- a/lib/pama/remove_test.go
+++ b/lib/pama/drop_test.go
@@ -8,7 +8,7 @@ import (
 	"git.sr.ht/~rjarry/aerc/lib/pama/models"
 )
 
-func TestPatchmgmt_Remove(t *testing.T) {
+func TestPatchmgmt_Drop(t *testing.T) {
 	setup := func(p models.Project) (pama.PatchManager, models.RevisionController, models.PersistentStorer) {
 		return newTestManager(
 			[]string{"0", "1", "2", "3", "4", "5"},
@@ -19,21 +19,21 @@ func TestPatchmgmt_Remove(t *testing.T) {
 
 	tests := []struct {
 		name    string
-		remove  string
+		drop    string
 		commits []models.Commit
 		want    []models.Commit
 	}{
 		{
-			name:   "remove only patch",
-			remove: "patch1",
+			name: "drop only patch",
+			drop: "patch1",
 			commits: []models.Commit{
 				newCommit("1", "a", "patch1"),
 			},
 			want: []models.Commit{},
 		},
 		{
-			name:   "remove second one of two patch",
-			remove: "patch2",
+			name: "drop second one of two patch",
+			drop: "patch2",
 			commits: []models.Commit{
 				newCommit("1", "a", "patch1"),
 				newCommit("2", "b", "patch2"),
@@ -43,8 +43,8 @@ func TestPatchmgmt_Remove(t *testing.T) {
 			},
 		},
 		{
-			name:   "remove first one of two patch",
-			remove: "patch1",
+			name: "drop first one of two patch",
+			drop: "patch1",
 			commits: []models.Commit{
 				newCommit("1", "a", "patch1"),
 				newCommit("2", "b", "patch2"),
@@ -63,7 +63,7 @@ func TestPatchmgmt_Remove(t *testing.T) {
 		}
 		mgr, rc, _ := setup(p)
 
-		err := mgr.RemovePatch(test.remove)
+		err := mgr.DropPatch(test.drop)
 		if err != nil {
 			t.Errorf("test '%s' failed. %v", test.name, err)
 		}
diff --git a/lib/pama/models/models.go b/lib/pama/models/models.go
index 9b21f0f5..5d1a9c44 100644
--- a/lib/pama/models/models.go
+++ b/lib/pama/models/models.go
@@ -70,9 +70,9 @@ type RevisionController interface {
 	Author(string) string
 	// Date returns the date for the provided commit hash.
 	Date(string) string
-	// Remove removes the commit with the provided commit hash from the
+	// Drop removes the commit with the provided commit hash from the
 	// repository.
-	Remove(string) error
+	Drop(string) error
 	// ApplyCmd returns a string with an executable command that is used to
 	// apply patches with the :pipe command.
 	ApplyCmd() string
diff --git a/lib/pama/pama_test.go b/lib/pama/pama_test.go
index 98258249..e07cbab6 100644
--- a/lib/pama/pama_test.go
+++ b/lib/pama/pama_test.go
@@ -96,7 +96,7 @@ func (c *mockRevctrl) Date(commit string) string {
 	return ""
 }
 
-func (c *mockRevctrl) Remove(commit string) error {
+func (c *mockRevctrl) Drop(commit string) error {
 	for i, s := range c.commitIDs {
 		if s == commit {
 			c.commitIDs = append(c.commitIDs[:i], c.commitIDs[i+1:]...)
diff --git a/lib/pama/revctrl/git.go b/lib/pama/revctrl/git.go
index 7be9de58..2a35754e 100644
--- a/lib/pama/revctrl/git.go
+++ b/lib/pama/revctrl/git.go
@@ -68,10 +68,10 @@ func (g git) Date(commit string) string {
 	return s
 }
 
-func (g git) Remove(commit string) error {
+func (g git) Drop(commit string) error {
 	_, exitcode, err := g.do("rebase", "--onto", commit+"^", commit)
 	if exitcode > 0 {
-		return fmt.Errorf("failed to remove commit %s", commit)
+		return fmt.Errorf("failed to drop commit %s", commit)
 	}
 	return err
 }