Changes in history.c [d4927a7:2bc6bb16]

File:

• history.c (modified) (4 diffs)

history.c

@@ -3 +3 @@
 void owl_history_init(owl_history *h)
 {
-  owl_list_create(&(h->hist));
-  h->cur=0;                     /* current position in history */
-  h->touched=0;                 /* whether we've gone into history */
-  h->partial=0;                 /* is the 0th element is partially composed? */
-  h->repeats=1;                 /* by default we'll allow repeat entries */
-}
-
-void owl_history_set_norepeats(owl_history *h)
-{
-  h->repeats=0;
+  g_queue_init(&h->hist);
+  h->cur = h->hist.tail;        /* current position in history */
+  h->partial = false;           /* is the 0th element is partially composed? */
 }
 
 const char *owl_history_get_prev(owl_history *h)
 {
+  if (!h) return NULL;
 
-  if (!h) return NULL;
-  h->touched=1;
+  if (h->cur == NULL || g_list_previous(h->cur) == NULL) return NULL;
 
-  if (owl_list_get_size(&(h->hist))==0) return(NULL);
-
-  if (h->cur == owl_list_get_size(&(h->hist))-1) {
-    return(NULL);
-  }
-
-  h->cur++;
-
-  return(owl_list_get_element(&(h->hist), h->cur));
+  h->cur = g_list_previous(h->cur);
+  return h->cur->data;
 }
 
@@ -35 +21 @@
 {
   if (!h) return NULL;
-  if (owl_list_get_size(&(h->hist))==0) return(NULL);
-  if (h->cur==0) {
-    return(NULL);
-  }
 
-  h->cur--;
-  return(owl_list_get_element(&(h->hist), h->cur));
+  if (h->cur == NULL || g_list_next(h->cur) == NULL) return NULL;
+
+  h->cur = g_list_next(h->cur);
+  return h->cur->data;
 }
 
-void owl_history_store(owl_history *h, const char *line)
+void owl_history_store(owl_history *h, const char *line, bool partial)
 {
-  int size;
+  if (!h) return;
 
-  if (!h) return;
-  size=owl_list_get_size(&(h->hist));
+  owl_history_reset(h);
 
-  /* if partial is set, remove the first entry first */
-  if (h->partial) {
-    g_free(owl_list_get_element(&(h->hist), 0));
-    owl_list_remove_element(&(h->hist), 0);
-  }
-
-  /* if repeats are disallowed, check if the line is the same as the last */
-  if (owl_list_get_size(&(h->hist))>0) {
-    if (!strcmp(line, owl_list_get_element(&(h->hist), 0))) return;
-  }
+  /* check if the line is the same as the last */
+  if (!partial && !g_queue_is_empty(&h->hist) &&
+      strcmp(line, g_queue_peek_tail(&h->hist)) == 0)
+    return;
 
   /* if we've reached the max history size, pop off the last element */
-  if (size>OWL_HISTORYSIZE) {
-    g_free(owl_list_get_element(&(h->hist), size-1));
-    owl_list_remove_element(&(h->hist), size-1);
-  }
+  if (g_queue_get_length(&h->hist) > OWL_HISTORYSIZE)
+    g_free(g_queue_pop_head(&h->hist));
 
   /* add the new line */
-  owl_list_prepend_element(&(h->hist), g_strdup(line));
-}
-
-void owl_history_set_partial(owl_history *h)
-{
-  if (!h) return;
-  h->partial=1;
+  g_queue_push_tail(&h->hist, g_strdup(line));
+  h->partial = partial;
+  h->cur = h->hist.tail;
 }
 
@@ -81 +52 @@
 {
   if (!h) return;
-  h->cur=0;
-  h->touched=0;
-  h->partial=0;
+
+  /* if partial is set, remove the first entry first */
+  if (h->partial) {
+    g_free(g_queue_pop_tail(&h->hist));
+    h->partial = false;
+  }
+
+  h->cur = h->hist.tail;
 }
 
@@ -89 +65 @@
 {
   if (!h) return(0);
-  if (h->touched) return(1);
-  return(0);
+  return h->cur != NULL && g_list_next(h->cur) != NULL;
 }