- 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).
- Changed a test that adds a task that is due in one hour, then
verifies that the task is 'due'. This fails when run between
the times 2300 and 2400. Now the test makes the task due in
5 minutes, which narrows the window when the test can fail.
This no longer coincides with the time the tinderbox runs.
- 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.
- Now that the 'diagnostics' command can detect duplicate UUID
values, it makes sense to incorporate that into unit tests that
employ multiple recurring tasks.
- When bulk tasks are modified, the "project changed" messages are
now retained until processing is completed, so that only one message
per project is generated.
