Code Cleanup
- Unnecessary test replaced by an assertion.
This commit is contained in:
committed by
Paul Beckingham
parent
573e0b2a65
commit
08b37c9943
23
src/Task.cpp
23
src/Task.cpp
@@ -29,6 +29,7 @@
|
|||||||
|
|
||||||
#include <sstream>
|
#include <sstream>
|
||||||
#include <stdlib.h>
|
#include <stdlib.h>
|
||||||
|
#include <assert.h>
|
||||||
#include <math.h>
|
#include <math.h>
|
||||||
#include <algorithm>
|
#include <algorithm>
|
||||||
#include <Context.h>
|
#include <Context.h>
|
||||||
@@ -1494,23 +1495,17 @@ float Task::urgency_due () const
|
|||||||
////////////////////////////////////////////////////////////////////////////////
|
////////////////////////////////////////////////////////////////////////////////
|
||||||
float Task::urgency_age () const
|
float Task::urgency_age () const
|
||||||
{
|
{
|
||||||
if (has ("entry"))
|
assert (has ("entry"));
|
||||||
{
|
|
||||||
Date now;
|
|
||||||
Date entry (get_date ("entry"));
|
|
||||||
int age = (now - entry) / 86400; // in days
|
|
||||||
float max = context.config.getReal ("urgency.age.max");
|
|
||||||
|
|
||||||
if (max == 0)
|
Date now;
|
||||||
return 1.0;
|
Date entry (get_date ("entry"));
|
||||||
|
int age = (now - entry) / 86400; // in days
|
||||||
|
float max = context.config.getReal ("urgency.age.max");
|
||||||
|
|
||||||
if (age > max)
|
if (max == 0 || age > max)
|
||||||
return 1.0;
|
return 1.0;
|
||||||
else
|
|
||||||
return (1.0 * age/max);
|
|
||||||
}
|
|
||||||
|
|
||||||
return 0.0;
|
return (1.0 * age/max);
|
||||||
}
|
}
|
||||||
|
|
||||||
////////////////////////////////////////////////////////////////////////////////
|
////////////////////////////////////////////////////////////////////////////////
|
||||||
|
|||||||
Reference in New Issue
Block a user