Changeset 5343:ba0d96408425

Timestamp:

2008-01-15 01:49:41 (8 months ago)

Author:

Rocco Rutte <pdmef@…>

Branch:

HEAD

Message:

Make formatting of query menu configurable via $query_format (closes #170).

Files:

• UPDATING (modified) (1 diff)
• globals.h (modified) (1 diff)
• init.h (modified) (1 diff)
• query.c (modified) (2 diffs)

UPDATING

@@ -5 +5 @@
   !: modified feature, -: deleted feature, +: new feature
 
+  + $query_format (customize external query menu)
   ! inode sorting is always enabled
   + $time_inc suppresses progress updates less than $time_inc

globals.h

@@ -112 +112 @@
 WHERE char *PrintCmd;
 WHERE char *QueryCmd;
+WHERE char *QueryFormat;
 WHERE char *Realname;
 WHERE char *SendCharset;

init.h

@@ -2270 +2270 @@
   ** with the query string the user types.  See ``$query'' for more
   ** information.
+  */
+  { "query_format",     DT_STR, R_NONE, UL &QueryFormat, UL "%4c %t %-25.25a %-25.25n %?e?(%e)?" },
+  /*
+  ** .pp
+  ** This variable describes the format of the `query' menu. The
+  ** following printf-style sequences are understood:
+  ** .pp
+  ** .dl
+  ** .dt %a  .dd destination address
+  ** .dt %c  .dd current entry number
+  ** .dt %e  .dd extra information *
+  ** .dt %n  .dd destination name
+  ** .dt %t  .dd ``*'' if current entry is tagged, a space otherwise
+  ** .dt %>X .dd right justify the rest of the string and pad with "X"
+  ** .dt %|X .dd pad to the end of the line with "X"
+  ** .dt %*X .dd soft-fill with character "X" as pad
+  ** .de
+  ** .pp
+  ** For an explanation of `soft-fill', see the ``$$index_format'' documentation.
+  ** .pp
+  ** * = can be optionally printed if nonzero, see the ``$$status_format'' documentation.
   */
   { "quit",             DT_QUAD, R_NONE, OPT_QUIT, M_YES },

query.c

@@ -33 +33 @@
 typedef struct query
 {
+  int num;
   ADDRESS *addr;
   char *name;
@@ -181 +182 @@
 }
 
-/* This is the callback routine from mutt_menuLoop() which is used to generate
- * a menu entry for the requested item number.
- */
-#define QUERY_MIN_COLUMN_LENGHT 20 /* Must be < 70/2 */
+static const char * query_format_str (char *dest, size_t destlen, size_t col,
+                                      char op, const char *src,
+                                      const char *fmt, const char *ifstring,
+                                      const char *elsestring,
+                                      unsigned long data, format_flag flags)
+{
+  ENTRY *entry = (ENTRY *)data;
+  QUERY *query = entry->data;
+  char tmp[SHORT_STRING];
+  char buf2[STRING] = "";
+  int optional = (flags & M_FORMAT_OPTIONAL);
+
+  switch (op)
+  {
+  case 'a':
+    rfc822_write_address (buf2, sizeof (buf2), query->addr, 1);
+    snprintf (tmp, sizeof (tmp), "%%%ss", fmt);
+    snprintf (dest, destlen, tmp, buf2);
+    break;
+  case 'c':
+    snprintf (tmp, sizeof (tmp), "%%%sd", fmt);
+    snprintf (dest, destlen, tmp, query->num);
+    break;
+  case 'e':
+    if (!optional)
+    {
+      snprintf (tmp, sizeof (tmp), "%%%ss", fmt);
+      snprintf (dest, destlen, tmp, NONULL (query->other));
+    }
+    else if (!query->other || !*query->other)
+      optional = 0;
+    break;
+  case 'n':
+    snprintf (tmp, sizeof (tmp), "%%%ss", fmt);
+    snprintf (dest, destlen, tmp, NONULL (query->name));
+    break;
+  case 't':
+    snprintf (tmp, sizeof (tmp), "%%%sc", fmt);
+    snprintf (dest, destlen, tmp, entry->tagged ? '*' : ' ');
+    break;
+  default:
+    snprintf (tmp, sizeof (tmp), "%%%sc", fmt);
+    snprintf (dest, destlen, tmp, op);
+    break;
+  }
+
+  if (optional)
+    mutt_FormatString (dest, destlen, col, ifstring, query_format_str, data, 0);
+  else if (flags & M_FORMAT_OPTIONAL)
+    mutt_FormatString (dest, destlen, col, elsestring, query_format_str, data, 0);
+
+  return src;
+}
+
 static void query_entry (char *s, size_t slen, MUTTMENU *m, int num)
 {
-  ENTRY *table = (ENTRY *) m->data;
-  char buf2[SHORT_STRING], buf[SHORT_STRING] = "";
-
-  /* need a query format ... hard coded constants are not good */
-  while (FirstColumn + SecondColumn > 70)
-  {
-    FirstColumn = FirstColumn * 3 / 4;
-    SecondColumn = SecondColumn * 3 / 4;
-    if (FirstColumn < QUERY_MIN_COLUMN_LENGHT)
-      FirstColumn = QUERY_MIN_COLUMN_LENGHT;
-    if (SecondColumn < QUERY_MIN_COLUMN_LENGHT)
-      SecondColumn = QUERY_MIN_COLUMN_LENGHT;
-  }
-
-  rfc822_write_address (buf, sizeof (buf), table[num].data->addr, 1);
-
-  mutt_format_string (buf2, sizeof (buf2),
-                      FirstColumn + 2, FirstColumn + 2,
-                      FMT_LEFT, ' ', table[num].data->name,
-                      mutt_strlen (table[num].data->name), 0);
-
-  snprintf (s, slen, " %c %3d %s %-*.*s %s", 
-            table[num].tagged ? '*':' ',
-            num+1,
-            buf2,
-            SecondColumn+2,
-            SecondColumn+2,
-            buf,
-            NONULL (table[num].data->other));
+  ENTRY *entry = &((ENTRY *) m->data)[num];
+
+  entry->data->num = num;
+  mutt_FormatString (s, slen, 0, NONULL (QueryFormat), query_format_str,
+                     (unsigned long) entry, M_FORMAT_ARROWCURSOR);
 }