gergely/taskwarrior-2.x
1
0
Fork 0
Code Issues Actions Packages Projects Releases 1 Wiki Activity

Performance

Browse Source 
- Moved an expensive operation inside an if statement.
This commit is contained in:
Paul Beckingham
2012-07-25 23:14:15 -04:00
parent d89d51e7e1
commit 39456a3243
1 changed files with 12 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

21
src/rules.cpp
View File

@@ -213,45 +213,48 @@ static void colorizeKeyword (Task& task, const std::string& rule, const Color& b
//////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////
static void colorizeDue (Task& task, const Color& base, Color& c) static void colorizeDue (Task& task, const Color& base, Color& c)
{
if (task.has ("due"))
{ {
Task::status status = task.getStatus (); Task::status status = task.getStatus ();
if (status != Task::completed &&
if (task.has ("due") &&
status != Task::completed &&
status != Task::deleted) status != Task::deleted)
{ {
if (getDueState (task.get ("due")) == 1) if (getDueState (task.get ("due")) == 1)
c.blend (base); c.blend (base);
} }
} }
}
//////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////
static void colorizeDueToday (Task& task, const Color& base, Color& c) static void colorizeDueToday (Task& task, const Color& base, Color& c)
{
if (task.has ("due"))
{ {
Task::status status = task.getStatus (); Task::status status = task.getStatus ();
if (status != Task::completed &&
if (task.has ("due") &&
status != Task::completed &&
status != Task::deleted) status != Task::deleted)
{ {
if (getDueState (task.get ("due")) == 2) if (getDueState (task.get ("due")) == 2)
c.blend (base); c.blend (base);
} }
} }
}
//////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////
static void colorizeOverdue (Task& task, const Color& base, Color& c) static void colorizeOverdue (Task& task, const Color& base, Color& c)
{
if (task.has ("due"))
{ {
Task::status status = task.getStatus (); Task::status status = task.getStatus ();
if (status != Task::completed &&
if (task.has ("due") &&
status != Task::completed &&
status != Task::deleted) status != Task::deleted)
{ {
if (getDueState (task.get ("due")) == 3) if (getDueState (task.get ("due")) == 3)
c.blend (base); c.blend (base);
} }
} }
}
//////////////////////////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////////////////////////
static void colorizeRecurring (Task& task, const Color& base, Color& c) static void colorizeRecurring (Task& task, const Color& base, Color& c)