Changeset 12c35df

Timestamp:

Oct 28, 2003, 7:56:35 PM (21 years ago)

Author:

James M. Kretchmar <kretch@mit.edu>

Branches:

master, barnowl_perlaim, debian, owl, release-1.10, release-1.4, release-1.5, release-1.6, release-1.7, release-1.8, release-1.9

Children:

cee1f25

Parents:

faf71b5

Message:

Fixed replies to loopback messages
Fixed smartnarrow on classes/instances with spaces
Don't allow duplicates in command history
Added the 'loggingdirection' variable
All loopback messages log to 'loopback' now
Print an error message if trying an invalid color for a filter
Fixed bug causing > not to go to end of editwin every time

Files:

• ChangeLog (modified) (1 diff)
• commands.c (modified) (1 diff)
• editwin.c (modified) (2 diffs)
• filter.c (modified) (1 diff)
• fmtext.c (modified) (2 diffs)
• functions.c (modified) (4 diffs)
• global.c (modified) (1 diff)
• history.c (modified) (3 diffs)
• logging.c (modified) (4 diffs)
• owl.h (modified) (3 diffs)
• util.c (modified) (2 diffs)
• variable.c (modified) (1 diff)
• zephyr.c (modified) (1 diff)

ChangeLog

@@ -1 +1 @@
 $Id$
 
+2.0.12-pre-1
+        Fixed replies to loopback messages
+        Fixed smartnarrow on classes/instances with spaces
+        Don't allow duplicates in command history
+        Added the 'loggingdirection' variable
+        All loopback messages log to 'loopback' now
+        Print an error message if trying an invalid color for a filter
+        Fixed bug causing > not to go to end of editwin every time
+        
 2.0.11
         Updated basic help

commands.c

@@ -1909 +1909 @@
   return(NULL);
 }
-
 
 char *owl_command_show(int argc, char **argv, char *buff)

editwin.c

@@ -176 +176 @@
 }
 
-
 /* malloc more space for the buffer */
 void _owl_editwin_addspace(owl_editwin *e)
@@ -773 +772 @@
 
   /* do we need to scroll? */
+  /*
   if (e->buffy-e->topline > e->winlines) {
     e->topline+=e->winlines/2;
   }
+  */
+  owl_editwin_recenter(e);
 }
 

filter.c

@@ -30 +30 @@
   /* set the color */
   if (argc>=2 && !strcmp(argv[0], "-c")) {
-    f->color=owl_util_string_to_color(argv[1]);
+    if (owl_util_string_to_color(argv[1])==-1) {
+      owl_function_error("The color '%s' is not available, using default.", argv[1]);
+    } else {
+      f->color=owl_util_string_to_color(argv[1]);
+    }
     argc-=2;
     argv+=2;

fmtext.c

@@ -122 +122 @@
   }
 }
+
+/* Internal function.  Append text from 'in' between index 'start' and
+ * 'stop' to the end of 'f'
+ */
+void _owl_fmtext_append_fmtext(owl_fmtext *f, owl_fmtext *in, int start, int stop)
+{
+  int newlen, i;
+
+  newlen=strlen(f->textbuff)+(stop-start+1);
+  f->textbuff=owl_realloc(f->textbuff, newlen+1);
+  f->fmbuff=owl_realloc(f->fmbuff, newlen+1);
+  f->colorbuff=owl_realloc(f->colorbuff, newlen+1);
+
+  strncat(f->textbuff, in->textbuff+start, stop-start+1);
+  f->textbuff[newlen]='\0';
+  for (i=start; i<=stop; i++) {
+    f->fmbuff[f->textlen+(i-start)]=in->fmbuff[i];
+    f->colorbuff[f->textlen+(i-start)]=in->colorbuff[i];
+  }
+  f->textlen=newlen;
+}
+
+/* append fmtext 'in' to 'f' */
+void owl_fmtext_append_fmtext(owl_fmtext *f, owl_fmtext *in)
+{
+  _owl_fmtext_append_fmtext(f, in, 0, in->textlen);
+
+}
+
+/* Append 'nspaces' number of spaces to the end of 'f' */
+void owl_fmtext_append_spaces(owl_fmtext *f, int nspaces)
+{
+  int i;
+  for (i=0; i<nspaces; i++) {
+    owl_fmtext_append_normal(f, " ");
+  }
+}
+
+/* requires that the list values are strings or NULL.
+ * joins the elements together with join_with. 
+ * If format_fn is specified, passes it the list element value
+ * and it will return a string which this needs to free. */
+void owl_fmtext_append_list(owl_fmtext *f, owl_list *l, char *join_with, char *(format_fn)(void*))
+{
+  int i, size;
+  void *elem;
+  char *text;
+
+  size = owl_list_get_size(l);
+  for (i=0; i<size; i++) {
+    elem = (char*)owl_list_get_element(l,i);
+    if (elem && format_fn) {
+      text = format_fn(elem);
+      if (text) {
+        owl_fmtext_append_normal(f, text);
+        owl_free(text);
+      }
+    } else if (elem) {
+      owl_fmtext_append_normal(f, elem);
+    }
+    if ((i < size-1) && join_with) {
+      owl_fmtext_append_normal(f, join_with);
+    }
+  }
+}
+
+/* Return a plain version of the fmtext.  Caller is responsible for
+ * freeing the return
+ */
+char *owl_fmtext_print_plain(owl_fmtext *f)
+{
+  return(owl_strdup(f->textbuff));
+}
+
+/* add the formatted text to the curses window 'w'.  The window 'w'
+ * must already be initiatlized with curses
+ */
+void owl_fmtext_curs_waddstr(owl_fmtext *f, WINDOW *w)
+{
+  char *tmpbuff;
+  int position, trans1, trans2, len, lastsame;
+
+  if (w==NULL) {
+    owl_function_debugmsg("Hit a null window in owl_fmtext_curs_waddstr.");
+    return;
+  }
+
+  tmpbuff=owl_malloc(f->textlen+10);
+
+  position=0;
+  len=f->textlen;
+  while (position<=len) {
+    /* find the last char with the current format and color */
+    trans1=owl_util_find_trans(f->fmbuff+position, len-position);
+    trans2=owl_util_find_trans(f->colorbuff+position, len-position);
+
+    if (trans1<trans2) {
+      lastsame=position+trans1;
+    } else {
+      lastsame=position+trans2;
+    }
+
+    /* set the format */
+    wattrset(w, A_NORMAL);
+    if (f->fmbuff[position] & OWL_FMTEXT_ATTR_BOLD) {
+      wattron(w, A_BOLD);
+    }
+    if (f->fmbuff[position] & OWL_FMTEXT_ATTR_REVERSE) {
+      wattron(w, A_REVERSE);
+    }
+    if (f->fmbuff[position] & OWL_FMTEXT_ATTR_UNDERLINE) {
+      wattron(w, A_UNDERLINE);
+    }
+
+    /* set the color */
+    /* warning, this is sort of a hack */
+    if (owl_global_get_hascolors(&g)) {
+      if (f->colorbuff[position]!=OWL_COLOR_DEFAULT) {
+        wattron(w, COLOR_PAIR(f->colorbuff[position]));
+      }
+    }
+
+    /* add the text */
+    strncpy(tmpbuff, f->textbuff + position, lastsame-position+1);
+    tmpbuff[lastsame-position+1]='\0';
+    waddstr(w, tmpbuff);
+
+    position=lastsame+1;
+  }
+  owl_free(tmpbuff);
+}
+
+
+/* start with line 'aline' (where the first line is 0) and print
+ * 'lines' number of lines into 'out'
+ */
+int owl_fmtext_truncate_lines(owl_fmtext *in, int aline, int lines, owl_fmtext *out)
+{
+  char *ptr1, *ptr2;
+  int i, offset;
+  
+  /* find the starting line */
+  ptr1=in->textbuff;
+  if (aline!=0) {
+    for (i=0; i<aline; i++) {
+      ptr1=strchr(ptr1, '\n');
+      if (!ptr1) return(-1);
+      ptr1++;
+    }
+  }
+  /* ptr1 now holds the starting point */
+
+  /* copy in the next 'lines' lines */
+  if (lines<1) return(-1);
+
+  for (i=0; i<lines; i++) {
+    offset=ptr1-in->textbuff;
+    ptr2=strchr(ptr1, '\n');
+    if (!ptr2) {
+      _owl_fmtext_append_fmtext(out, in, offset, (in->textlen)-1);
+      return(-1);
+    }
+    _owl_fmtext_append_fmtext(out, in, offset, (ptr2-ptr1)+offset);
+    ptr1=ptr2+1;
+  }
+  return(0);
+}
+
+/* Truncate the message so that each line begins at column 'acol' and
+ * ends at 'bcol' or sooner.  The first column is number 0.  The new
+ * message is placed in 'out'.  The message is * expected to end in a
+ * new line for now
+ */
+void owl_fmtext_truncate_cols(owl_fmtext *in, int acol, int bcol, owl_fmtext *out)
+{
+  char *ptr1, *ptr2, *last;
+  int len, offset;
+
+  last=in->textbuff+in->textlen-1;
+  ptr1=in->textbuff;
+  while (ptr1<=last) {
+    ptr2=strchr(ptr1, '\n');
+    if (!ptr2) {
+      /* but this shouldn't happen if we end in a \n */
+      break;
+    }
+    
+    if (ptr2==ptr1) {
+      owl_fmtext_append_normal(out, "\n");
+      ptr1++;
+      continue;
+    }
+
+    /* we need to check that we won't run over here */
+    len=bcol-acol;
+    if (len > (ptr2-(ptr1+acol))) {
+      /* the whole line fits with room to spare, don't take a full 'len' */
+      len=ptr2-(ptr1+acol);
+    }
+    if (len>last-ptr1) {
+      /* the whole rest of the text fits with room to spare, adjust for it */
+      len-=(last-ptr1);
+    }
+    if (len<=0) {
+      /* saftey check */
+      owl_fmtext_append_normal(out, "\n");
+      ptr1=ptr2+1;
+      continue;
+    }
+
+    offset=ptr1-in->textbuff;
+    _owl_fmtext_append_fmtext(out, in, offset+acol, offset+acol+len);
+
+    ptr1=ptr2+1;
+  }
+}
+
+/* Return the number of lines in 'f' */
+int owl_fmtext_num_lines(owl_fmtext *f)
+{
+  int lines, i;
+
+  if (f->textlen==0) return(0);
+  
+  lines=0;
+  for (i=0; i<f->textlen; i++) {
+    if (f->textbuff[i]=='\n') lines++;
+  }
+
+  /* if the last char wasn't a \n there's one more line */
+  if (f->textbuff[i-1]!='\n') lines++;
+
+  return(lines);
+}
+
+char *owl_fmtext_get_text(owl_fmtext *f)
+{
+  return(f->textbuff);
+}
+
+void owl_fmtext_set_char(owl_fmtext *f, int index, int ch)
+{
+  /* set the charater at 'index' to be 'char'.  If index is out of
+   * bounds don't do anything */
+  if ((index < 0) || (index > f->textlen-1)) return;
+  f->textbuff[index]=ch;
+}
+
+/* Free all memory allocated by the object */
+void owl_fmtext_free(owl_fmtext *f)
+{
+  if (f->textbuff) owl_free(f->textbuff);
+  if (f->fmbuff) owl_free(f->fmbuff);
+  if (f->colorbuff) owl_free(f->colorbuff);
+}
+
+/* Make a copy of the fmtext 'src' into 'dst' */
+void owl_fmtext_copy(owl_fmtext *dst, owl_fmtext *src)
+{
+  int mallocsize;
+
+  if (src->textlen==0) {
+    mallocsize=5;
+  } else {
+    mallocsize=src->textlen+2;
+  }
+  dst->textlen=src->textlen;
+  dst->textbuff=owl_malloc(mallocsize);
+  dst->fmbuff=owl_malloc(mallocsize);
+  dst->colorbuff=owl_malloc(mallocsize);
+  memcpy(dst->textbuff, src->textbuff, src->textlen+1);
+  memcpy(dst->fmbuff, src->fmbuff, src->textlen);
+  memcpy(dst->colorbuff, src->colorbuff, src->textlen);
+}
+
+/* highlight all instances of "string".  Return the number of
+ * instances found.  This is a case insensitive search.
+ */
+int owl_fmtext_search_and_highlight(owl_fmtext *f, char *string)
+{
+
+  int found, len;
+  char *ptr1, *ptr2;
+
+  len=strlen(string);
+  found=0;
+  ptr1=f->textbuff;
+  while (ptr1-f->textbuff <= f->textlen) {
+    ptr2=stristr(ptr1, string);
+    if (!ptr2) return(found);
+
+    found++;
+    _owl_fmtext_add_attr(f, OWL_FMTEXT_ATTR_REVERSE,
+                         ptr2 - f->textbuff,
+                         ptr2 - f->textbuff + len - 1);
+
+    ptr1=ptr2+len;
+  }
+  return(found);
+}
+
+/* return 1 if the string is found, 0 if not.  This is a case
+ *  insensitive search.
+ */
+int owl_fmtext_search(owl_fmtext *f, char *string)
+{
+
+  if (stristr(f->textbuff, string)) return(1);
+  return(0);
+}
+
 
 /* Append the text 'text' to 'f' and interpret the zephyr style
@@ -311 +622 @@
   }
 }
-
-/* Internal function.  Append text from 'in' between index 'start' and
- * 'stop' to the end of 'f'
- */
-void _owl_fmtext_append_fmtext(owl_fmtext *f, owl_fmtext *in, int start, int stop)
-{
-  int newlen, i;
-
-  newlen=strlen(f->textbuff)+(stop-start+1);
-  f->textbuff=owl_realloc(f->textbuff, newlen+1);
-  f->fmbuff=owl_realloc(f->fmbuff, newlen+1);
-  f->colorbuff=owl_realloc(f->colorbuff, newlen+1);
-
-  strncat(f->textbuff, in->textbuff+start, stop-start+1);
-  f->textbuff[newlen]='\0';
-  for (i=start; i<=stop; i++) {
-    f->fmbuff[f->textlen+(i-start)]=in->fmbuff[i];
-    f->colorbuff[f->textlen+(i-start)]=in->colorbuff[i];
-  }
-  f->textlen=newlen;
-}
-
-/* append fmtext 'in' to 'f' */
-void owl_fmtext_append_fmtext(owl_fmtext *f, owl_fmtext *in)
-{
-  _owl_fmtext_append_fmtext(f, in, 0, in->textlen);
-
-}
-
-/* Append 'nspaces' number of spaces to the end of 'f' */
-void owl_fmtext_append_spaces(owl_fmtext *f, int nspaces)
-{
-  int i;
-  for (i=0; i<nspaces; i++) {
-    owl_fmtext_append_normal(f, " ");
-  }
-}
-
-/* requires that the list values are strings or NULL.
- * joins the elements together with join_with. 
- * If format_fn is specified, passes it the list element value
- * and it will return a string which this needs to free. */
-void owl_fmtext_append_list(owl_fmtext *f, owl_list *l, char *join_with, char *(format_fn)(void*))
-{
-  int i, size;
-  void *elem;
-  char *text;
-
-  size = owl_list_get_size(l);
-  for (i=0; i<size; i++) {
-    elem = (char*)owl_list_get_element(l,i);
-    if (elem && format_fn) {
-      text = format_fn(elem);
-      if (text) {
-        owl_fmtext_append_normal(f, text);
-        owl_free(text);
-      }
-    } else if (elem) {
-      owl_fmtext_append_normal(f, elem);
-    }
-    if ((i < size-1) && join_with) {
-      owl_fmtext_append_normal(f, join_with);
-    }
-  }
-}
-
-/* Return a plain version of the fmtext.  Caller is responsible for
- * freeing the return
- */
-char *owl_fmtext_print_plain(owl_fmtext *f)
-{
-  return(owl_strdup(f->textbuff));
-}
-
-/* add the formatted text to the curses window 'w'.  The window 'w'
- * must already be initiatlized with curses
- */
-void owl_fmtext_curs_waddstr(owl_fmtext *f, WINDOW *w)
-{
-  char *tmpbuff;
-  int position, trans1, trans2, len, lastsame;
-
-  if (w==NULL) {
-    owl_function_debugmsg("Hit a null window in owl_fmtext_curs_waddstr.");
-    return;
-  }
-
-  tmpbuff=owl_malloc(f->textlen+10);
-
-  position=0;
-  len=f->textlen;
-  while (position<=len) {
-    /* find the last char with the current format and color */
-    trans1=owl_util_find_trans(f->fmbuff+position, len-position);
-    trans2=owl_util_find_trans(f->colorbuff+position, len-position);
-
-    if (trans1<trans2) {
-      lastsame=position+trans1;
-    } else {
-      lastsame=position+trans2;
-    }
-
-    /* set the format */
-    wattrset(w, A_NORMAL);
-    if (f->fmbuff[position] & OWL_FMTEXT_ATTR_BOLD) {
-      wattron(w, A_BOLD);
-    }
-    if (f->fmbuff[position] & OWL_FMTEXT_ATTR_REVERSE) {
-      wattron(w, A_REVERSE);
-    }
-    if (f->fmbuff[position] & OWL_FMTEXT_ATTR_UNDERLINE) {
-      wattron(w, A_UNDERLINE);
-    }
-
-    /* set the color */
-    /* warning, this is sort of a hack */
-    if (owl_global_get_hascolors(&g)) {
-      if (f->colorbuff[position]!=OWL_COLOR_DEFAULT) {
-        wattron(w, COLOR_PAIR(f->colorbuff[position]));
-      }
-    }
-
-    /* add the text */
-    strncpy(tmpbuff, f->textbuff + position, lastsame-position+1);
-    tmpbuff[lastsame-position+1]='\0';
-    waddstr(w, tmpbuff);
-
-    position=lastsame+1;
-  }
-  owl_free(tmpbuff);
-}
-
-
-/* start with line 'aline' (where the first line is 0) and print
- * 'lines' number of lines into 'out'
- */
-int owl_fmtext_truncate_lines(owl_fmtext *in, int aline, int lines, owl_fmtext *out)
-{
-  char *ptr1, *ptr2;
-  int i, offset;
-  
-  /* find the starting line */
-  ptr1=in->textbuff;
-  if (aline!=0) {
-    for (i=0; i<aline; i++) {
-      ptr1=strchr(ptr1, '\n');
-      if (!ptr1) return(-1);
-      ptr1++;
-    }
-  }
-  /* ptr1 now holds the starting point */
-
-  /* copy in the next 'lines' lines */
-  if (lines<1) return(-1);
-
-  for (i=0; i<lines; i++) {
-    offset=ptr1-in->textbuff;
-    ptr2=strchr(ptr1, '\n');
-    if (!ptr2) {
-      _owl_fmtext_append_fmtext(out, in, offset, (in->textlen)-1);
-      return(-1);
-    }
-    _owl_fmtext_append_fmtext(out, in, offset, (ptr2-ptr1)+offset);
-    ptr1=ptr2+1;
-  }
-  return(0);
-}
-
-/* Truncate the message so that each line begins at column 'acol' and
- * ends at 'bcol' or sooner.  The first column is number 0.  The new
- * message is placed in 'out'.  The message is * expected to end in a
- * new line for now
- */
-void owl_fmtext_truncate_cols(owl_fmtext *in, int acol, int bcol, owl_fmtext *out)
-{
-  char *ptr1, *ptr2, *last;
-  int len, offset;
-
-  last=in->textbuff+in->textlen-1;
-  ptr1=in->textbuff;
-  while (ptr1<=last) {
-    ptr2=strchr(ptr1, '\n');
-    if (!ptr2) {
-      /* but this shouldn't happen if we end in a \n */
-      break;
-    }
-    
-    if (ptr2==ptr1) {
-      owl_fmtext_append_normal(out, "\n");
-      ptr1++;
-      continue;
-    }
-
-    /* we need to check that we won't run over here */
-    len=bcol-acol;
-    if (len > (ptr2-(ptr1+acol))) {
-      /* the whole line fits with room to spare, don't take a full 'len' */
-      len=ptr2-(ptr1+acol);
-    }
-    if (len>last-ptr1) {
-      /* the whole rest of the text fits with room to spare, adjust for it */
-      len-=(last-ptr1);
-    }
-    if (len<=0) {
-      /* saftey check */
-      owl_fmtext_append_normal(out, "\n");
-      ptr1=ptr2+1;
-      continue;
-    }
-
-    offset=ptr1-in->textbuff;
-    _owl_fmtext_append_fmtext(out, in, offset+acol, offset+acol+len);
-
-    ptr1=ptr2+1;
-  }
-}
-
-/* Return the number of lines in 'f' */
-int owl_fmtext_num_lines(owl_fmtext *f)
-{
-  int lines, i;
-
-  if (f->textlen==0) return(0);
-  
-  lines=0;
-  for (i=0; i<f->textlen; i++) {
-    if (f->textbuff[i]=='\n') lines++;
-  }
-
-  /* if the last char wasn't a \n there's one more line */
-  if (f->textbuff[i-1]!='\n') lines++;
-
-  return(lines);
-}
-
-char *owl_fmtext_get_text(owl_fmtext *f)
-{
-  return(f->textbuff);
-}
-
-void owl_fmtext_set_char(owl_fmtext *f, int index, int ch)
-{
-  /* set the charater at 'index' to be 'char'.  If index is out of
-   * bounds don't do anything */
-  if ((index < 0) || (index > f->textlen-1)) return;
-  f->textbuff[index]=ch;
-}
-
-/* Free all memory allocated by the object */
-void owl_fmtext_free(owl_fmtext *f)
-{
-  if (f->textbuff) owl_free(f->textbuff);
-  if (f->fmbuff) owl_free(f->fmbuff);
-  if (f->colorbuff) owl_free(f->colorbuff);
-}
-
-/* Make a copy of the fmtext 'src' into 'dst' */
-void owl_fmtext_copy(owl_fmtext *dst, owl_fmtext *src)
-{
-  int mallocsize;
-
-  if (src->textlen==0) {
-    mallocsize=5;
-  } else {
-    mallocsize=src->textlen+2;
-  }
-  dst->textlen=src->textlen;
-  dst->textbuff=owl_malloc(mallocsize);
-  dst->fmbuff=owl_malloc(mallocsize);
-  dst->colorbuff=owl_malloc(mallocsize);
-  memcpy(dst->textbuff, src->textbuff, src->textlen+1);
-  memcpy(dst->fmbuff, src->fmbuff, src->textlen);
-  memcpy(dst->colorbuff, src->colorbuff, src->textlen);
-}
-
-/* highlight all instance of "string".  Return the number of
- * instances found.  This is a case insensitive search.
- */
-int owl_fmtext_search_and_highlight(owl_fmtext *f, char *string)
-{
-
-  int found, len;
-  char *ptr1, *ptr2;
-
-  len=strlen(string);
-  found=0;
-  ptr1=f->textbuff;
-  while (ptr1-f->textbuff <= f->textlen) {
-    ptr2=stristr(ptr1, string);
-    if (!ptr2) return(found);
-
-    found++;
-    _owl_fmtext_add_attr(f, OWL_FMTEXT_ATTR_REVERSE,
-                         ptr2 - f->textbuff,
-                         ptr2 - f->textbuff + len - 1);
-
-    ptr1=ptr2+len;
-  }
-  return(found);
-}
-
-/* return 1 if the string is found, 0 if not.  This is a case
- *  insensitive search.
- */
-int owl_fmtext_search(owl_fmtext *f, char *string)
-{
-
-  if (stristr(f->textbuff, string)) return(1);
-  return(0);
-}

functions.c

@@ -1883 +1883 @@
     if (owl_message_is_type_admin(m)) {
       owl_function_error("You cannot reply to an admin message");
+      return;
+    }
+
+    /* loopback */
+    if (owl_message_is_type_loopback(m)) {
+      owl_function_loopwrite_setup();
       return;
     }
@@ -2268 +2274 @@
       return;
     }
+    if (owl_util_string_to_color(argv[3])==-1) {
+      owl_function_error("The color '%s' is not available.", argv[3]);
+      return;
+    }
     owl_filter_set_color(f, owl_util_string_to_color(argv[3]));
     owl_global_set_needrefresh(&g);
@@ -2405 +2415 @@
   argbuff=owl_malloc(len+20);
   tmpclass=owl_text_quote(class, OWL_REGEX_QUOTECHARS, OWL_REGEX_QUOTEWITH);
+  owl_text_tr(tmpclass, ' ', '.');
   if (instance) {
     tmpinstance=owl_text_quote(instance, OWL_REGEX_QUOTECHARS, OWL_REGEX_QUOTEWITH);
+    owl_text_tr(tmpinstance, ' ', '.');
   }
   sprintf(argbuff, "( class ^%s$ )", tmpclass);
@@ -2707 +2719 @@
 
   /* deal with the case of trying change the filter color */
+  if (owl_util_string_to_color(color)==-1) {
+    owl_function_error("No color named '%s' avilable.");
+    return;
+  }
   owl_filter_set_color(f, owl_util_string_to_color(color));
   owl_global_set_needrefresh(&g);

global.c

@@ -75 +75 @@
   owl_history_init(&(g->msghist));
   owl_history_init(&(g->cmdhist));
+  owl_history_set_norepeats(&(g->cmdhist));
   g->nextmsgid=0;
 

history.c

@@ -9 +9 @@
   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;
 }
 
@@ -42 +48 @@
 void owl_history_store(owl_history *h, char *line)
 {
-  int size;
+  int i, size;
 
   if (!h) return;
+  size=owl_list_get_size(&(h->hist));
+
+  /* if repeats are disallowed, check if the line is present already */
+  for (i=0; i<size; i++) {
+    if (!strcmp(line, owl_list_get_element(&(h->hist), i))) return;
+  }
 
   /* if partial is set, remove the first entry first */
@@ -52 +64 @@
 
   /* if we've reached the max history size, pop off the last element */
-  size=owl_list_get_size(&(h->hist));
   if (size>OWL_HISTORYSIZE) {
     owl_free(owl_list_get_element(&(h->hist), size-1));

logging.c

@@ -13 +13 @@
   char *tobuff, *ptr="";
 
+  if (owl_global_get_loggingdirection(&g)==OWL_LOGGING_DIRECTION_IN) return;
+
   tobuff=owl_malloc(strlen(to)+20);
   strcpy(tobuff, to);
@@ -61 +63 @@
   char *tobuff;
 
+  if (owl_global_get_loggingdirection(&g)==OWL_LOGGING_DIRECTION_IN) return;
+
   tobuff=owl_sprintf("aim:%s", to);
 
@@ -102 +106 @@
   char *tobuff;
 
-  tobuff=owl_sprintf("loopback:%s", "loppback");
+  if (owl_global_get_loggingdirection(&g)==OWL_LOGGING_DIRECTION_IN) return;
+
+  tobuff=owl_sprintf("loopback");
 
   /* expand ~ in path names */
@@ -143 +149 @@
   char *frombuff=NULL, *from=NULL, *buff=NULL, *ptr;
   int len, ch, i, personal;
+
+  if (owl_global_get_loggingdirection(&g)==OWL_LOGGING_DIRECTION_OUT) return;
       
   /* check for nolog */

owl.h

@@ -19 +19 @@
 static const char owl_h_fileIdent[] = "$Id$";
 
-#define OWL_VERSION         2.0.11
-#define OWL_VERSION_STRING "2.0.11"
+#define OWL_VERSION         2.0.12-pre-1
+#define OWL_VERSION_STRING "2.0.12-pre-1"
 
 /* Feature that is being tested to redirect stderr through a pipe. 
@@ -74 +74 @@
 #define OWL_DIRECTION_DOWNWARDS 1
 #define OWL_DIRECTION_UPWARDS   2
+
+#define OWL_LOGGING_DIRECTION_BOTH 0
+#define OWL_LOGGING_DIRECTION_IN   1
+#define OWL_LOGGING_DIRECTION_OUT  2
 
 #define OWL_SCROLLMODE_NORMAL      0
@@ -355 +359 @@
   int touched;
   int partial;
+  int repeats;
 } owl_history;
 

util.c

@@ -408 +408 @@
 }
 
-/* Return the owl color associated with the named color */
+/* Return the owl color associated with the named color.  Return -1
+ * if the named color is not available
+ */
 int owl_util_string_to_color(char *color)
 {
@@ -430 +432 @@
     return(OWL_COLOR_DEFAULT);
   }
-  return(OWL_COLOR_DEFAULT);
+  return(-1);
 }
 

variable.c

@@ -111 +111 @@
                "The filename in that directory is derived from\n"
                "the name of the class to which the message was sent.\n" ),
+
+  OWLVAR_ENUM( "loggingdirection" /* %OwlVarStub */, OWL_LOGGING_DIRECTION_BOTH,
+               "specifices which kind of messages should be logged",
+               "Can be one of 'both', 'in', or 'out'.  If 'in' is\n"
+               "selected, only incoming messages are logged, if 'out'\n"
+               "is selected only outgoing messages are logged.  If 'both'\n"
+               "is selected both incoming and outgoing messages are\n"
+               "logged.",
+               "both,in,out"),
 
   OWLVAR_BOOL( "colorztext" /* %OwlVarStub */, 1,

zephyr.c

@@ -774 +774 @@
   return(out);
 #else
-  return("");
+  return(owl_strdup("Zephyr not available"));
 #endif
 }