Two brown paper bags later OTodo is ok again... pointers can be so fscking awesome

I had problems with QShared because I forgot to copy it in &operator=
And the other one was d = 0

6 files changed, 42 insertions, 36 deletions

diff --git a/libopie/pim/orecordlist.h b/libopie/pim/orecordlist.h
index 75bb33c..36728b8 100644
--- a/libopie/pim/orecordlist.h
+++ b/libopie/pim/orecordlist.h
@@ -39,9 +39,9 @@ public:
     /**
      * a * operator ;)
      * use it like this T = (*it);
      */
-    T &operator*();
+    T operator*();
     ORecordListIterator &operator++();
     ORecordListIterator &operator--();
 
     bool operator==( const ORecordListIterator& it );
@@ -98,17 +98,18 @@ template <class T>
 ORecordListIterator<T>::ORecordListIterator() {
     m_current = 0;
     m_temp = 0l;
     m_end = true;
+    m_record = T();
 }
 template <class T>
 ORecordListIterator<T>::~ORecordListIterator() {
 /* nothing to delete */
 }
 
 template <class T>
 ORecordListIterator<T>::ORecordListIterator( const ORecordListIterator<T>& it) {
-    qWarning("ORecordListIterator");
+//    qWarning("ORecordListIterator copy c'tor");
     m_uids = it.m_uids;
     m_current = it.m_current;
     m_temp = it.m_temp;
     m_end = it.m_end;
@@ -120,16 +121,16 @@ ORecordListIterator<T> &ORecordListIterator<T>::operator=( const ORecordListIter
     m_uids = it.m_uids;
     m_current = it.m_current;
     m_temp = it.m_temp;
     m_end = it.m_end;
-    m_record = it.m_record;
+//    m_record = it.m_record;
 
     return *this;
 }
 
 template <class T>
-T &ORecordListIterator<T>::operator*() {
-    qWarning("operator* %d %d", m_current,  m_uids[m_current] );
+T ORecordListIterator<T>::operator*() {
+//    qWarning("operator* %d %d", m_current,  m_uids[m_current] );
     if (!m_end )
         m_record = m_temp->find( m_uids[m_current] );
     else
         m_record = T();
@@ -192,9 +193,8 @@ ORecordList<T>::~ORecordList() {
 /* nothing to do here */
 }
 template <class T>
 ORecordList<T>::Iterator ORecordList<T>::begin() {
-    qWarning("ORecordList::begin");
     Iterator it( m_ids, m_acc );
     return it;
 }
 template <class T>
diff --git a/libopie/pim/otodo.cpp b/libopie/pim/otodo.cpp
index b3c14e6..cc46b21 100644
--- a/libopie/pim/otodo.cpp
+++ b/libopie/pim/otodo.cpp
@@ -35,14 +35,15 @@ struct OTodo::OTodoData : public QShared {
 OTodo::OTodo(const OTodo &event )
     : OPimRecord( event ),  data( event.data )
 {
     data->ref();
-    qWarning("ref up");
+//    qWarning("ref up");
 }
 OTodo::~OTodo() {
-    qWarning("~OTodo " + data->sum );
+
+//    qWarning("~OTodo "  );
     if ( data->deref() ) {
-        qWarning("OTodo::dereffing");
+//        qWarning("OTodo::dereffing");
         delete data;
         data = 0l;
     }
 }
@@ -53,9 +54,9 @@ OTodo::OTodo(bool completed, int priority,
                      ushort progress,
                      bool hasDate, QDate date, int uid )
     : OPimRecord(  uid )
 {
-    qWarning("OTodoData " + summary);
+//    qWarning("OTodoData " + summary);
     setCategories( category );
 
     data = new OTodoData;
 
@@ -76,14 +77,14 @@ OTodo::OTodo(bool completed, int priority,
                      ushort progress,
                      bool hasDate, QDate date, int uid )
     : OPimRecord(  uid )
 {
-    qWarning("OTodoData" + summary);
+//    qWarning("OTodoData" + summary);
     setCategories( idsFromString( category.join(";") ) );
 
     data = new OTodoData;
-    data->date = date;
 
+    data->date = date;
     data->isCompleted = completed;
     data->hasDate = hasDate;
     data->priority = priority;
     data->sum = summary;
@@ -159,8 +160,9 @@ void OTodo::setHasAlarmDateTime( bool hasAlarmDateTime )
     data->hasAlarmDateTime = hasAlarmDateTime;
 }
 void OTodo::setDescription(const QString &desc )
 {
+    qWarning( "desc " + desc );
     changeOrModify();
     data->desc = Qtopia::simplifyMultiLineSpace(desc );
 }
 void OTodo::setSummary( const QString& sum )
@@ -306,21 +308,22 @@ bool OTodo::operator==(const OTodo &toDoEvent )const
     return OPimRecord::operator==( toDoEvent );
 }
 void OTodo::deref() {
 
-    //qWarning("deref in ToDoEvent");
+//    qWarning("deref in ToDoEvent");
     if ( data->deref() ) {
-        //qWarning("deleting");
+//        qWarning("deleting");
         delete data;
-        d= 0;
+        data= 0;
     }
 }
 OTodo &OTodo::operator=(const OTodo &item )
 {
-    OPimRecord::operator=( item );
+//    OPimRecord::operator=( item );
     //qWarning("operator= ref ");
     item.data->ref();
     deref();
+    data = item.data;
 
     return *this;
 }
 
@@ -354,9 +357,9 @@ QMap<QString, QString> OTodo::toExtraMap()const {
  * right in place
  */
 void OTodo::changeOrModify() {
     if ( data->count != 1 ) {
-        qWarning("changeOrModify");
+//        qWarning("changeOrModify");
         data->deref();
         OTodoData* d2 = new OTodoData();
         copy(data, d2 );
         data = d2;
diff --git a/libopie/pim/otodoaccessxml.cpp b/libopie/pim/otodoaccessxml.cpp
index 21756c9..3a72881 100644
--- a/libopie/pim/otodoaccessxml.cpp
+++ b/libopie/pim/otodoaccessxml.cpp
@@ -67,16 +67,16 @@ bool OTodoAccessXML::load() {
             m_events.insert( ev.uid(), ev );
 
             element = element->nextChild();
         }
-        return true;
     }else {
         qWarning("could not parse");
         return false;;
     }
     delete root;
 
     m_opened = true;
+    qWarning("Access %d" + m_events.count() );
     return true;
 }
 bool OTodoAccessXML::reload() {
     return load();
diff --git a/libopie2/opiepim/backend/otodoaccessxml.cpp b/libopie2/opiepim/backend/otodoaccessxml.cpp
index 21756c9..3a72881 100644
--- a/libopie2/opiepim/backend/otodoaccessxml.cpp
+++ b/libopie2/opiepim/backend/otodoaccessxml.cpp
@@ -67,16 +67,16 @@ bool OTodoAccessXML::load() {
             m_events.insert( ev.uid(), ev );
 
             element = element->nextChild();
         }
-        return true;
     }else {
         qWarning("could not parse");
         return false;;
     }
     delete root;
 
     m_opened = true;
+    qWarning("Access %d" + m_events.count() );
     return true;
 }
 bool OTodoAccessXML::reload() {
     return load();
diff --git a/libopie2/opiepim/orecordlist.h b/libopie2/opiepim/orecordlist.h
index 75bb33c..36728b8 100644
--- a/libopie2/opiepim/orecordlist.h
+++ b/libopie2/opiepim/orecordlist.h
@@ -39,9 +39,9 @@ public:
     /**
      * a * operator ;)
      * use it like this T = (*it);
      */
-    T &operator*();
+    T operator*();
     ORecordListIterator &operator++();
     ORecordListIterator &operator--();
 
     bool operator==( const ORecordListIterator& it );
@@ -98,17 +98,18 @@ template <class T>
 ORecordListIterator<T>::ORecordListIterator() {
     m_current = 0;
     m_temp = 0l;
     m_end = true;
+    m_record = T();
 }
 template <class T>
 ORecordListIterator<T>::~ORecordListIterator() {
 /* nothing to delete */
 }
 
 template <class T>
 ORecordListIterator<T>::ORecordListIterator( const ORecordListIterator<T>& it) {
-    qWarning("ORecordListIterator");
+//    qWarning("ORecordListIterator copy c'tor");
     m_uids = it.m_uids;
     m_current = it.m_current;
     m_temp = it.m_temp;
     m_end = it.m_end;
@@ -120,16 +121,16 @@ ORecordListIterator<T> &ORecordListIterator<T>::operator=( const ORecordListIter
     m_uids = it.m_uids;
     m_current = it.m_current;
     m_temp = it.m_temp;
     m_end = it.m_end;
-    m_record = it.m_record;
+//    m_record = it.m_record;
 
     return *this;
 }
 
 template <class T>
-T &ORecordListIterator<T>::operator*() {
-    qWarning("operator* %d %d", m_current,  m_uids[m_current] );
+T ORecordListIterator<T>::operator*() {
+//    qWarning("operator* %d %d", m_current,  m_uids[m_current] );
     if (!m_end )
         m_record = m_temp->find( m_uids[m_current] );
     else
         m_record = T();
@@ -192,9 +193,8 @@ ORecordList<T>::~ORecordList() {
 /* nothing to do here */
 }
 template <class T>
 ORecordList<T>::Iterator ORecordList<T>::begin() {
-    qWarning("ORecordList::begin");
     Iterator it( m_ids, m_acc );
     return it;
 }
 template <class T>
diff --git a/libopie2/opiepim/otodo.cpp b/libopie2/opiepim/otodo.cpp
index b3c14e6..cc46b21 100644
--- a/libopie2/opiepim/otodo.cpp
+++ b/libopie2/opiepim/otodo.cpp
@@ -35,14 +35,15 @@ struct OTodo::OTodoData : public QShared {
 OTodo::OTodo(const OTodo &event )
     : OPimRecord( event ),  data( event.data )
 {
     data->ref();
-    qWarning("ref up");
+//    qWarning("ref up");
 }
 OTodo::~OTodo() {
-    qWarning("~OTodo " + data->sum );
+
+//    qWarning("~OTodo "  );
     if ( data->deref() ) {
-        qWarning("OTodo::dereffing");
+//        qWarning("OTodo::dereffing");
         delete data;
         data = 0l;
     }
 }
@@ -53,9 +54,9 @@ OTodo::OTodo(bool completed, int priority,
                      ushort progress,
                      bool hasDate, QDate date, int uid )
     : OPimRecord(  uid )
 {
-    qWarning("OTodoData " + summary);
+//    qWarning("OTodoData " + summary);
     setCategories( category );
 
     data = new OTodoData;
 
@@ -76,14 +77,14 @@ OTodo::OTodo(bool completed, int priority,
                      ushort progress,
                      bool hasDate, QDate date, int uid )
     : OPimRecord(  uid )
 {
-    qWarning("OTodoData" + summary);
+//    qWarning("OTodoData" + summary);
     setCategories( idsFromString( category.join(";") ) );
 
     data = new OTodoData;
-    data->date = date;
 
+    data->date = date;
     data->isCompleted = completed;
     data->hasDate = hasDate;
     data->priority = priority;
     data->sum = summary;
@@ -159,8 +160,9 @@ void OTodo::setHasAlarmDateTime( bool hasAlarmDateTime )
     data->hasAlarmDateTime = hasAlarmDateTime;
 }
 void OTodo::setDescription(const QString &desc )
 {
+    qWarning( "desc " + desc );
     changeOrModify();
     data->desc = Qtopia::simplifyMultiLineSpace(desc );
 }
 void OTodo::setSummary( const QString& sum )
@@ -306,21 +308,22 @@ bool OTodo::operator==(const OTodo &toDoEvent )const
     return OPimRecord::operator==( toDoEvent );
 }
 void OTodo::deref() {
 
-    //qWarning("deref in ToDoEvent");
+//    qWarning("deref in ToDoEvent");
     if ( data->deref() ) {
-        //qWarning("deleting");
+//        qWarning("deleting");
         delete data;
-        d= 0;
+        data= 0;
     }
 }
 OTodo &OTodo::operator=(const OTodo &item )
 {
-    OPimRecord::operator=( item );
+//    OPimRecord::operator=( item );
     //qWarning("operator= ref ");
     item.data->ref();
     deref();
+    data = item.data;
 
     return *this;
 }
 
@@ -354,9 +357,9 @@ QMap<QString, QString> OTodo::toExtraMap()const {
  * right in place
  */
 void OTodo::changeOrModify() {
     if ( data->count != 1 ) {
-        qWarning("changeOrModify");
+//        qWarning("changeOrModify");
         data->deref();
         OTodoData* d2 = new OTodoData();
         copy(data, d2 );
         data = d2;