try to use less memory and only create objects like

ORecur, OPimState, OPimNotifyManager and OPimMaintainer if necessary

4 files changed, 164 insertions, 30 deletions

diff --git a/libopie/pim/otodo.cpp b/libopie/pim/otodo.cpp
index ea66d39..f3df119 100644
--- a/libopie/pim/otodo.cpp
+++ b/libopie/pim/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/libopie/pim/otodo.h b/libopie/pim/otodo.h
index 2f66f55..a58d9aa 100644
--- a/libopie/pim/otodo.h
+++ b/libopie/pim/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;
     /*
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;
     /*