- 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.
- 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.
- Fixed problem where urgency was not properly calculated for waiting tasks.
- Tweaked urgency coefficients to make most of the values closer together, and
therefore more sensitive.
- Inverted 'waiting' coefficient.
- Boosted 'next', 'due' and 'blocking' coefficients.
- Modified unit tests accordingly.
- Lessened the impact of 'blocked', 'project', 'tags' and 'age'.
- Fixed bug #552, where 'rc.verbose=off' suppressed warnings (thanks to Peter
De Poorter).
- Fixed bug #863, which suppressed report labels with rc.verbose=off (thanks to
Michelle Crane).
Manually include unistd.h in a couple of places get things building with gcc
4.7, which removed it from the global namespace. This fixes the following build
errors:
src/A3.cpp:283:3: error: 'STDIN_FILENO' was not declared in this scope
src/commands/CmdEdit.cpp:598:30: error: 'getpid' was not declared in this scope
src/commands/CmdEdit.cpp:603:47: error: 'chdir' was not declared in this scope
Signed-off-by: Paul Beckingham <paul@beckingham.net>
- Corrected performance measurement. The TDB2::gc requires data to
be loaded first, but load time is measured separately. This resulted
in repeat counting. Now the gc time subtracts any load time that is
accumulated *during* the gc. This is now a fair accounting of the time.
- Fixed bug that caused only parent recurring tasks to have their attributes
properly removed.
- When duplicating a parent recurring task, a new recurring parent task is
created. When a child recurring task is duplicated, a plain task is created.
- Added unit tests.
- Thanks to Jennifer Cormier.
- Added feature #632, which allows environment variables TASKRC and TASKDATA
to override .taskrc and .task directory locations (thanks to Steve Rader).
- Added unit tests.
+
- Tag matching was being performed using the regex \b<tag>\b, which makes
taskwarrior dependent on regex lib bugs for basic functionality. This is
now modified to use pseudo-operators _hastag_ and _notag_.
- lua_open has been declared as deprecated in lua 5.1 and removed from
5.2. Usage of luaL_newstate is thus recommended in both series.
Signed-off-by: Paul Beckingham <paul@beckingham.net>
- Fixed bug #636, which causes the burndown reports to not honor the
_forcecolor=off setting (thanks to Steve Rader, Uli Martens).
Signed-off-by: Paul Beckingham <paul@beckingham.net>
- Fixed bug #818, which caused partial tag matching (thanks to Joe Holloway).
- Note that the regex word boundary anchors are different for Solaris
and Linux, and largely broken on OSX.
- Added unit tests.
