From 1d8a60a9ba57a18940f73157858fde1b8d887a98 Mon Sep 17 00:00:00 2001
From: Paul Beckingham <paul@beckingham.net>
Date: Sun, 9 Jan 2011 21:25:09 -0500
Subject: [PATCH] Bug

- Fixed bug that caused the 'done' command to always exit with a non-zero
  status (thanks to Steve Rader).
---
 ChangeLog       | 2 ++
 NEWS            | 2 +-
 src/command.cpp | 2 ++
 3 files changed, 5 insertions(+), 1 deletion(-)

diff --git a/ChangeLog b/ChangeLog
index 2618829bc..c429b0b24 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -29,6 +29,8 @@
  + Added _query helper command for script writers, which accepts a filter like
    any other report, but returns only full JSON.
  + The dependency columns are now right-justified (thanks to Eric Fluger).
+ + Fixed bug that caused the 'done' command to always exit with a non-zero
+   status (thanks to Steve Rader).
  + Fixed bug #515, which displayed an incorrect message after duplicating a
    non-existent task (thanks to Peter De Poorter).
  + Fixed bug #529, where the 'depends' attribute was not mentioned in the
diff --git a/NEWS b/NEWS
index a9be79d0f..742b32e16 100644
--- a/NEWS
+++ b/NEWS
@@ -11,7 +11,7 @@ New Features in taskwarrior 1.9.4
   - The first month in the calendar report can now be changed with an offset
     value.
   - No more dependency on ncurses.
-  - New '_query helper command for script writers, which accepts a filter like
+  - New '_query' helper command for script writers, which accepts a filter like
     any other report, but returns unformatted JSON.
   - Assorted bug fixes.
 
diff --git a/src/command.cpp b/src/command.cpp
index cca6155df..94dd1cbb7 100644
--- a/src/command.cpp
+++ b/src/command.cpp
@@ -1864,12 +1864,14 @@ int handleDone (std::string& outs)
           nagged = nag (*task);
       }
       else
+      {
         out << "Task "
             << task->id
             << " '"
             << task->get ("description")
             << "' is neither pending nor waiting.\n";
         rc = 1;
+      }
     }
 
     if (count)