- Fixed bug #996, so that verbosity code and documentation are in agreement,
and that the 'verbose=off' works as intended (thanks to Peter De Poorter,
Louis-Claude Canon).
- Fixed bug #995, which mis-parsed UUIDs in filters as other elements (thanks
to Bryce Harrington).
- Nibbler no longer permits 8-character UUID abbreviations, because too often
they resemble other forms, such as '1111111d' which looks like a duration.
- Modified unit tests accordingly.
- Added std::string::reserve in high-use locations.
- Used Task::get_date over Task::get where possible.
- Added missing code (bug) for waking up any completed tasks that are freshly
waited.
- Greater use of STL methods for bulk copying.
- Fixed bug #991, which removes an obsolete consistency check.
- Removed 'annotations' example from man page, because it is obsolete.
Signed-off-by: Paul Beckingham <paul@beckingham.net>
- Fixed bug #987, so that total active time does not continue to accumulate
after an active task is completed.
Signed-off-by: Paul Beckingham <paul@beckingham.net>
- Introduced a new filter optimization that recognizes filters with no 'OR', or
'XOR' operators, includes IDs, but does not include UUIDs. This combination
means completed.data is not referenced.
- Fixed bug where '6 months' was interpreted as 180 days, but when rendered
was shown as '5 months' (thanks to Aikido Guy).
- Multiple unit test corrections.
- Broke the indentTree function into two pieces - one that processes
a list, the other that processes a single item. This makes it more
efficient for use places that have only one element.
- Addressed valgrind complaints by freeing allocations for commands and columns.
This leaves one complaint about getpwuid, which is either an optional free,
or not valid, depending on OS. (Thanks to Bryce Harrington).
- Removed the feature that allows commands to be piped in to stdin,
and appended to any existing command line. This feature is
conditionally compiled, controlled by the FEATURE_STDIN define in
main.h
- Many unit tests contained "echo '-- y'", and now use "echo 'y'"
because the '--' is no longer supported on stdin.
- Thanks to the IRC team for testing, including Bryce Harrington,
Sam Stuck, Owen Clarke, Greg Grossmeier.
- Added bounds check before call to string::substr to avoid passing a
negative start pos. Test rc.t was broken from commit 0ac9a4b because
it contains 'report.:b' and legacyCheckForDeprecatedVariables was
trying to substr starting from pos 'length() - 12' (-3).
- Fixed bug #837, which caused incorrect urgency calculations for tasks that
have completed dependencies, and problems when editing those tasks (thanks
to Matt Kraai).
- Added Task::addDependency (const std::string&) method for reconstructing
dependencies on tasks with no ID.
- Modfified Task::urgency_blocked so that it considers the blocking task's
status. This is an expensive test, and so it is wrapped by a cheaper test
to see if there are/were any dependencies at all. This means that urgency
calculations are not slowed for tasks without dependencies, and is slower
but more accurate for tasks that do have dependencies.
- Modified edit command so that dependencies are shown as IDs or UUIDs
depending on whether the task is pending or not.
- Modified edit command so that dependencies are parsed as IDs or UUIDs
depending on whether the task is pending or not.
- Modified wording in the template of the edit command to reflect the above.
- Added unit tests bug.837.t.