Bug

- Fixed bug that caused the 'done' command to always exit with a non-zero
  status (thanks to Steve Rader).

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

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.
 

src/command.cpp

@@ -1864,6 +1864,7 @@ int handleDone (std::string& outs)
           nagged = nag (*task);
       }
       else
+      {
         out << "Task "
             << task->id
             << " '"
@@ -1871,6 +1872,7 @@ int handleDone (std::string& outs)
             << "' is neither pending nor waiting.\n";
         rc = 1;
       }
+    }
 
     if (count)
       context.tdb.commit ();