summaryrefslogtreecommitdiff
path: root/libopie2
authorzecke <zecke>2003-05-07 16:02:41 (UTC)
committer zecke <zecke>2003-05-07 16:02:41 (UTC)
commitaccd04a63230ac46978f77deae1b0d1419618730 (patch) (side-by-side diff)
tree6a54ce4cdda8d86a781838b7b990939e2962d461 /libopie2
parentef9b40f99443fabed972d29ce47c2ccb29e77210 (diff)
downloadopie-accd04a63230ac46978f77deae1b0d1419618730.zip
opie-accd04a63230ac46978f77deae1b0d1419618730.tar.gz
opie-accd04a63230ac46978f77deae1b0d1419618730.tar.bz2
try to use less memory and only create objects like
ORecur, OPimState, OPimNotifyManager and OPimMaintainer if necessary
Diffstat (limited to 'libopie2') (more/less context) (show whitespace changes)
-rw-r--r--libopie2/opiepim/otodo.cpp81
-rw-r--r--libopie2/opiepim/otodo.h16
2 files changed, 82 insertions, 15 deletions
diff --git a/libopie2/opiepim/otodo.cpp b/libopie2/opiepim/otodo.cpp
index ea66d39..f3df119 100644
--- a/libopie2/opiepim/otodo.cpp
+++ b/libopie2/opiepim/otodo.cpp
@@ -25,2 +25,6 @@ struct OTodo::OTodoData : public QShared {
OTodoData() : QShared() {
+ recur = 0;
+ state = 0;
+ maintainer = 0;
+ notifiers = 0;
};
@@ -35,8 +39,8 @@ struct OTodo::OTodoData : public QShared {
ushort prog;
- OPimState state;
- ORecur recur;
- OPimMaintainer maintainer;
+ OPimState *state;
+ ORecur *recur;
+ OPimMaintainer *maintainer;
QDate start;
QDate completed;
- OPimNotifyManager notifiers;
+ OPimNotifyManager *notifiers;
};
@@ -153,10 +157,32 @@ QString OTodo::description()const
}
+bool OTodo::hasState() const{
+ if (!data->state ) return false;
+ return ( data->state->state() != OPimState::Undefined );
+}
OPimState OTodo::state()const {
- return data->state;
+ if (!data->state ) {
+ OPimState state;
+ return state;
+ }
+
+ return (*data->state);
+}
+bool OTodo::hasRecurrence()const {
+ if (!data->recur) return false;
+ return data->recur->doesRecur();
}
ORecur OTodo::recurrence()const {
- return data->recur;
+ if (!data->recur) return ORecur();
+
+ return (*data->recur);
+}
+bool OTodo::hasMaintainer()const {
+ if (!data->maintainer) return false;
+
+ return (data->maintainer->mode() != OPimMaintainer::Undefined );
}
OPimMaintainer OTodo::maintainer()const {
- return data->maintainer;
+ if (!data->maintainer) return OPimMaintainer();
+
+ return (*data->maintainer);
}
@@ -203,3 +229,6 @@ void OTodo::setState( const OPimState& state ) {
changeOrModify();
- data->state = state;
+ if (data->state )
+ (*data->state) = state;
+ else
+ data->state = new OPimState( state );
}
@@ -207,3 +236,6 @@ void OTodo::setRecurrence( const ORecur& rec) {
changeOrModify();
- data->recur = rec;
+ if (data->recur )
+ (*data->recur) = rec;
+ else
+ data->recur = new ORecur( rec );
}
@@ -211,3 +243,7 @@ void OTodo::setMaintainer( const OPimMaintainer& pim ) {
changeOrModify();
- data->maintainer = pim;
+
+ if (data->maintainer )
+ (*data->maintainer) = pim;
+ else
+ data->maintainer = new OPimMaintainer( pim );
}
@@ -262,4 +298,10 @@ QString OTodo::toRichText() const
}
+bool OTodo::hasNotifiers()const {
+ if (!data->notifiers) return false;
+ return data->notifiers->isEmpty();
+}
OPimNotifyManager& OTodo::notifiers() {
- return data->notifiers;
+ if (!data->notifiers )
+ data->notifiers = new OPimNotifyManager;
+ return (*data->notifiers);
}
@@ -407,8 +449,17 @@ void OTodo::copy( OTodoData* src, OTodoData* dest ) {
dest->prog = src->prog;
- dest->state = src->state;
- dest->recur = src->recur;
- dest->maintainer = src->maintainer;
+
+ if (src->state )
+ dest->state = new OPimState( *src->state );
+
+ if (src->recur )
+ dest->recur = new ORecur( *src->recur );
+
+ if (src->maintainer )
+ dest->maintainer = new OPimMaintainer( *src->maintainer )
+ ;
dest->start = src->start;
dest->completed = src->completed;
- dest->notifiers = src->notifiers;
+
+ if (src->notifiers )
+ dest->notifiers = new OPimNotifyManager( *src->notifiers );
}
diff --git a/libopie2/opiepim/otodo.h b/libopie2/opiepim/otodo.h
index 2f66f55..a58d9aa 100644
--- a/libopie2/opiepim/otodo.h
+++ b/libopie2/opiepim/otodo.h
@@ -130,2 +130,7 @@ public:
/**
+ * does it have a state?
+ */
+ bool hasState()const;
+
+ /**
* What is the state of this OTodo?
@@ -135,2 +140,7 @@ public:
/**
+ * has recurrence?
+ */
+ bool hasRecurrence()const;
+
+ /**
* the recurrance of this
@@ -140,2 +150,7 @@ public:
/**
+ * does this OTodo have a maintainer?
+ */
+ bool hasMaintainer()const;
+
+ /**
* the Maintainer of this OTodo
@@ -160,2 +175,3 @@ public:
+ bool hasNotifiers()const;
/*