From d7d6ff2fed328be93346c4599b4f10d7003db513 Mon Sep 17 00:00:00 2001
From: Dan Jackson <e28eta@yahoo.com>
Date: Wed, 26 Nov 2014 10:23:54 -0800
Subject: [PATCH] Use process-specific tmp file and clean up on exit

Fixes #279 using the suggestion from @hemanth, with some minor tweaks:

- I kept the leading . because I figured there was a reason it was put there originally.
- Changed the show_cursor method that's called when the process exits (prematurely or naturally) to be show_cursor_and_cleanup, and added a line to it that cleans up the tmp file.
---
 bin/git-effort | 11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)

diff --git a/bin/git-effort b/bin/git-effort
index a4128de..6258649 100755
--- a/bin/git-effort
+++ b/bin/git-effort
@@ -1,6 +1,6 @@
 #!/usr/bin/env bash
 
-tmp=/tmp/.git-effort
+tmp="/tmp/.$(basename $0).$$"
 above='0'
 color=
 
@@ -21,12 +21,13 @@ hide_cursor() {
 }
 
 #
-# show cursor
+# show cursor, and remove temporary file
 #
 
-show_cursor() {
+show_cursor_and_cleanup() {
   printf '\033[?25h'
   printf '\033[m\n'
+  rm "$tmp" > /dev/null 2>&1
   exit 1
 }
 
@@ -131,7 +132,7 @@ fi
 # hide cursor
 
 hide_cursor
-trap show_cursor INT
+trap show_cursor_and_cleanup INT
 
 # loop files
 
@@ -139,4 +140,4 @@ heading
 effort "${files[@]}" | tee $tmp && sort_effort
 echo
 
-show_cursor
+show_cursor_and_cleanup