# [PATCH] Remove unused sh functions from git-mergetool.sh

2 messages from 2008-03-03 to 2008-03-03. Participants: Charles Bailey, Junio C Hamano.
Thread: https://gitlist.dev/t/12466

## Charles Bailey, 2008-03-03 22:18

Subject: [PATCH] Remove unused sh functions from git-mergetool.sh
Message-ID: <20080303221817.GA7410@hashpling.org>
URL: https://gitlist.dev/e/20080303221817.GA7410%40hashpling.org

```
The tidy up of git mergetool's backup files handling made the two shell
functions save_backup and remove_backup redundant. This commit removes
these now unused functions.

Signed-off-by: Charles Bailey <charles@hashpling.org>
---

This patch belongs on top of the mergetool updates in pu. It removes a
couple of shell functions which were made redundant in commit
6fef6e2...  In fact it should logically be squashed into that commit.


 git-mergetool.sh |   12 ------------
 1 files changed, 0 insertions(+), 12 deletions(-)

diff --git a/git-mergetool.sh b/git-mergetool.sh
index d12bedf..5c86f69 100755
--- a/git-mergetool.sh
+++ b/git-mergetool.sh
@@ -127,18 +127,6 @@ check_unchanged () {
     fi
 }
 
-save_backup () {
-    if test "$status" -eq 0; then
-	mv -- "$BACKUP" "$MERGED.orig"
-    fi
-}
-
-remove_backup () {
-    if test "$status" -eq 0; then
-	rm "$BACKUP"
-    fi
-}

```

## Junio C Hamano, 2008-03-03 23:33

Subject: Re: [PATCH] Remove unused sh functions from git-mergetool.sh
Message-ID: <7vd4qbfknd.fsf@gitster.siamese.dyndns.org>
URL: https://gitlist.dev/e/7vd4qbfknd.fsf%40gitster.siamese.dyndns.org
In-Reply-To: <20080303221817.GA7410@hashpling.org>

```
Charles Bailey <charles@hashpling.org> writes:

> This patch belongs on top of the mergetool updates in pu. It removes a
> couple of shell functions which were made redundant in commit
> 6fef6e2...  In fact it should logically be squashed into that commit.

Will do.  Thanks.

```