"col_format_to_pref_str()" is used only in "prefs.c", and knows about
authorguy <guy@f5534014-38df-0310-8fa8-9805f1628bb7>
Sun, 22 Jul 2001 21:56:25 +0000 (21:56 +0000)
committerguy <guy@f5534014-38df-0310-8fa8-9805f1628bb7>
Sun, 22 Jul 2001 21:56:25 +0000 (21:56 +0000)
the format of string lists in a preferences file; rename it to
"put_string_list()", make it take the list as an argument rather than
working only on "prefs.col_list", and put it in "prefs.c".

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@3776 f5534014-38df-0310-8fa8-9805f1628bb7

column.c
column.h
prefs.c

index 90841602eab1070676770a4959774af24a9855ac..95fc361775b7167cd8de08f5ebf514a86cef3a3a 100644 (file)
--- a/column.c
+++ b/column.c
@@ -1,7 +1,7 @@
 /* column.c
  * Routines for handling column preferences
  *
 /* column.c
  * Routines for handling column preferences
  *
- * $Id: column.c,v 1.33 2001/07/22 21:50:46 guy Exp $
+ * $Id: column.c,v 1.34 2001/07/22 21:56:25 guy Exp $
  *
  * Ethereal - Network traffic analyzer
  * By Gerald Combs <gerald@zing.org>
  *
  * Ethereal - Network traffic analyzer
  * By Gerald Combs <gerald@zing.org>
@@ -388,55 +388,3 @@ get_column_title(gint col) {
 
   return(cfmt->title);  
 }
 
   return(cfmt->title);  
 }
-
-/* XXX - needs to handle quote marks inside the quoted string, by
-   backslash-escaping them.
-
-   XXX - does this really belong in "prefs.c", instead, as it has to know
-   about the syntax of the preferences file? */
-#define MAX_FMT_PREF_LEN      1024
-#define MAX_FMT_PREF_LINE_LEN   60
-gchar *
-col_format_to_pref_str(void) {
-  static gchar  pref_str[MAX_FMT_PREF_LEN] = "";
-  GList        *clp = g_list_first(prefs.col_list);
-  fmt_data     *cfmt;
-  int           cur_pos = 0, cur_len = 0, fmt_len;
-  
-  while (clp) {
-    cfmt = (fmt_data *) clp->data;
-    
-    fmt_len = strlen(cfmt->title) + 4;
-    if ((fmt_len + cur_len) < (MAX_FMT_PREF_LEN - 1)) {
-      if ((fmt_len + cur_pos) > MAX_FMT_PREF_LINE_LEN) {
-        cur_len--;
-        cur_pos = 0;
-               pref_str[cur_len] = '\n'; cur_len++;
-        pref_str[cur_len] = '\t'; cur_len++;
-      }
-      sprintf(&pref_str[cur_len], "\"%s\", ", cfmt->title);
-      cur_len += fmt_len;
-      cur_pos += fmt_len;
-    }
-
-    fmt_len = strlen(cfmt->fmt) + 4;
-    if ((fmt_len + cur_len) < (MAX_FMT_PREF_LEN - 1)) {
-      if ((fmt_len + cur_pos) > MAX_FMT_PREF_LINE_LEN) {
-        cur_len--;
-        cur_pos = 0;
-        pref_str[cur_len] = '\n'; cur_len++;
-        pref_str[cur_len] = '\t'; cur_len++;
-      }
-      sprintf(&pref_str[cur_len], "\"%s\", ", cfmt->fmt);
-      cur_len += fmt_len;
-      cur_pos += fmt_len;
-    }
-    
-    clp = clp->next;
-  }
-  
-  if (cur_len > 2)
-    pref_str[cur_len - 2] = '\0';
-
-  return(pref_str);
-}    
index 579ec2943423b4cd45d551669586c4b3a6a90a06..9edf0f1b73eb3c6a145c4a25b12b5bdb3b8c985d 100644 (file)
--- a/column.h
+++ b/column.h
@@ -1,7 +1,7 @@
 /* column.h
  * Definitions for column handling routines
  *
 /* column.h
  * Definitions for column handling routines
  *
- * $Id: column.h,v 1.7 2000/01/10 01:43:48 guy Exp $
+ * $Id: column.h,v 1.8 2001/07/22 21:56:25 guy Exp $
  *
  * Ethereal - Network traffic analyzer
  * By Gerald Combs <gerald@zing.org>
  *
  * Ethereal - Network traffic analyzer
  * By Gerald Combs <gerald@zing.org>
@@ -44,7 +44,6 @@ void                 get_column_format_matches(gboolean *, gint);
 gint                 get_column_format_from_str(gchar *);
 enum col_resize_type get_column_resize_type(gint);
 gchar               *get_column_title(gint);
 gint                 get_column_format_from_str(gchar *);
 enum col_resize_type get_column_resize_type(gint);
 gchar               *get_column_title(gint);
-gchar               *col_format_to_pref_str(void);
 char                *get_column_longest_string(gint);
 gint                 get_column_char_width(gint format);
 
 char                *get_column_longest_string(gint);
 gint                 get_column_char_width(gint format);
 
diff --git a/prefs.c b/prefs.c
index cd2bb52c540dd634ed48af28dfbc35d06e0d9ef6..e1631c82f2cf3aaff4f4611fc36d78bc25c5ecd9 100644 (file)
--- a/prefs.c
+++ b/prefs.c
@@ -1,7 +1,7 @@
 /* prefs.c
  * Routines for handling preferences
  *
 /* prefs.c
  * Routines for handling preferences
  *
- * $Id: prefs.c,v 1.55 2001/07/22 21:50:46 guy Exp $
+ * $Id: prefs.c,v 1.56 2001/07/22 21:56:25 guy Exp $
  *
  * Ethereal - Network traffic analyzer
  * By Gerald Combs <gerald@ethereal.com>
  *
  * Ethereal - Network traffic analyzer
  * By Gerald Combs <gerald@ethereal.com>
@@ -437,6 +437,56 @@ get_string_list(gchar *str)
   return(sl);
 }
 
   return(sl);
 }
 
+/* XXX - needs to handle quote marks inside the quoted string, by
+   backslash-escaping them.  */
+#define MAX_FMT_PREF_LEN      1024
+#define MAX_FMT_PREF_LINE_LEN   60
+static gchar *
+put_string_list(GList *sl)
+{
+  static gchar  pref_str[MAX_FMT_PREF_LEN] = "";
+  GList        *clp = g_list_first(sl);
+  fmt_data     *cfmt;
+  int           cur_pos = 0, cur_len = 0, fmt_len;
+  
+  while (clp) {
+    cfmt = (fmt_data *) clp->data;
+    
+    fmt_len = strlen(cfmt->title) + 4;
+    if ((fmt_len + cur_len) < (MAX_FMT_PREF_LEN - 1)) {
+      if ((fmt_len + cur_pos) > MAX_FMT_PREF_LINE_LEN) {
+        cur_len--;
+        cur_pos = 0;
+               pref_str[cur_len] = '\n'; cur_len++;
+        pref_str[cur_len] = '\t'; cur_len++;
+      }
+      sprintf(&pref_str[cur_len], "\"%s\", ", cfmt->title);
+      cur_len += fmt_len;
+      cur_pos += fmt_len;
+    }
+
+    fmt_len = strlen(cfmt->fmt) + 4;
+    if ((fmt_len + cur_len) < (MAX_FMT_PREF_LEN - 1)) {
+      if ((fmt_len + cur_pos) > MAX_FMT_PREF_LINE_LEN) {
+        cur_len--;
+        cur_pos = 0;
+        pref_str[cur_len] = '\n'; cur_len++;
+        pref_str[cur_len] = '\t'; cur_len++;
+      }
+      sprintf(&pref_str[cur_len], "\"%s\", ", cfmt->fmt);
+      cur_len += fmt_len;
+      cur_pos += fmt_len;
+    }
+    
+    clp = clp->next;
+  }
+  
+  if (cur_len > 2)
+    pref_str[cur_len - 2] = '\0';
+
+  return(pref_str);
+}    
+
 void
 clear_string_list(GList *sl) {
   GList *l = sl;
 void
 clear_string_list(GList *sl) {
   GList *l = sl;
@@ -1400,7 +1450,7 @@ write_prefs(char **pf_path_return)
 
   fprintf (pf, "# Packet list column format.  Each pair of strings consists "
     "of a column title \n# and its format.\n"
 
   fprintf (pf, "# Packet list column format.  Each pair of strings consists "
     "of a column title \n# and its format.\n"
-    "%s: %s\n\n", PRS_COL_FMT, col_format_to_pref_str());
+    "%s: %s\n\n", PRS_COL_FMT, put_string_list(prefs.col_list));
 
   fprintf (pf, "# TCP stream window color preferences.  Each value is a six "
     "digit hexadecimal value in the form rrggbb.\n");
 
   fprintf (pf, "# TCP stream window color preferences.  Each value is a six "
     "digit hexadecimal value in the form rrggbb.\n");