1 files changed, 9 insertions, 3 deletions

diff --git a/libopie/pim/odatebookaccessbackend_xml.cpp b/libopie/pim/odatebookaccessbackend_xml.cpp
index 39c43c5..929d004 100644
--- a/libopie/pim/odatebookaccessbackend_xml.cpp
+++ b/libopie/pim/odatebookaccessbackend_xml.cpp
@@ -79,3 +79,3 @@ namespace {
         FNote,
-        FCreated,
+        FCreated,      // Should't this be called FRCreated ?
         FTimeZone,
@@ -85,2 +85,4 @@ namespace {
     };
+
+    // FIXME: Use OEvent::toMap() here !! (eilers)
     inline void save( const OEvent& ev, QString& buf ) {
@@ -277,3 +279,3 @@ bool ODateBookAccessBackend_XML::remove( int uid ) {
 bool ODateBookAccessBackend_XML::replace( const OEvent& ev ) {
-    replace( ev.uid() );
+    replace( ev.uid() ); // ??? Shouldn't this be "remove( ev.uid() ) ??? (eilers)
     return add( ev );
@@ -323,2 +325,4 @@ OEvent::ValueList ODateBookAccessBackend_XML::directRawRepeats() {
 }
+
+// FIXME: Use OEvent::fromMap() (eilers)
 bool ODateBookAccessBackend_XML::loadFile() {
@@ -361,3 +365,3 @@ bool ODateBookAccessBackend_XML::loadFile() {
     dict.insert( "note", new int(FNote) );
-    dict.insert( "created", new int(FCreated) );
+    dict.insert( "created", new int(FCreated) );  // Shouldn't this be FRCreated ??
     dict.insert( "recparent", new int(FRecParent) );
@@ -446,2 +450,4 @@ bool ODateBookAccessBackend_XML::loadFile() {
 }
+
+// FIXME: Use OEvent::fromMap() which makes this obsolete.. (eilers)
 void ODateBookAccessBackend_XML::finalizeRecord( OEvent& ev ) {