Changeset 2171

Timestamp:

06/06/07 11:39:52 (6 years ago)

Author:

njp

Message:

Fix call ordering in history; Clean up journal code for outgoing calls

Location:

trunk/src/target/OM-2007/applications/openmoko-dialer/src

Files:

• dialer-window-history.c (modified) (4 diffs)
• dialer-window-outgoing.c (modified) (1 diff)

trunk/src/target/OM-2007/applications/openmoko-dialer/src/dialer-window-history.c

@@ -727 +727 @@
 }
 
+/* Sort two entries by date */
+static gint
+sort_by_date (MokoJournalEntry *a, MokoJournalEntry *b)
+{
+  const MokoTime *at, *bt;
+  time_t ta, tb;
+  
+  at = moko_journal_entry_get_dtstart (a);
+  bt = moko_journal_entry_get_dtstart (b);
+  
+  ta = moko_time_as_timet (at);
+  tb = moko_time_as_timet (bt);
+  
+  return (gint)difftime (ta, tb);
+}
+
 
 /**
@@ -750 +766 @@
   int i = 0, j =0, n_entries;
   MokoJournalEntry *j_entry;
+  GList *entries = NULL, *e;
   //DBG_ENTER();
 
@@ -834 +851 @@
   
   i = j = 0;
-  for (i = 0; i < n_entries && j < HISTORY_MAX_ENTRIES; i++)
+  for (i = 0; i < n_entries; i++)
   {
     moko_journal_get_entry_at (p_dialer_data->journal, i, &j_entry);
@@ -840 +857 @@
     /* We're not interested in anything other than voice entrys */
     if (moko_journal_entry_get_type (j_entry) != VOICE_JOURNAL_ENTRY)
-    {
       continue;
-    }
+
+    entries = g_list_insert_sorted (entries, 
+                                    (gpointer)j_entry,
+                                    (GCompareFunc)sort_by_date);
+  }
+  for (e = entries; e != NULL && j < HISTORY_MAX_ENTRIES; e = e->next) 
+  {
     if (history_add_entry (list_store, j_entry))
       j++;

trunk/src/target/OM-2007/applications/openmoko-dialer/src/dialer-window-outgoing.c

@@ -154 +154 @@
   moko_journal_entry_get_voice_info (entry, &info);
   moko_journal_entry_set_dtstart (entry, moko_time_new_today ());
+  moko_journal_voice_info_set_distant_number (info, number);
   moko_journal_add_entry (data->journal, entry);
-  moko_journal_voice_info_set_distant_number (info, number);
-  /* FIXME: We should be able to associate a number with a number, and add that
-            info to the entry */
+  /* FIXME: We should be able to associate a number with a contact uid, and 
+            add that info to the entry */
 
   /* connect our handler to track call progress */