Put in a comment noting that the format of a preferences string is

determined by the code in "prefs.c", so arguably
"col_format_to_pref_str()" belongs there.

Put in a comment noting that said format includes support for escaping
quotation marks in quoted strings, and that we should handle that.


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

1 files changed, 7 insertions, 2 deletions

diff --git a/column.c b/column.c
index 1fdc3e9348..6f1588e107 100644
--- a/column.c
+++ b/column.c
@@ -1,7 +1,7 @@
 /* column.c
  * Routines for handling column preferences
  *
- * $Id: column.c,v 1.31 2001/07/11 04:44:58 guy Exp $
+ * $Id: column.c,v 1.32 2001/07/22 21:28:46 guy Exp $
  *
  * Ethereal - Network traffic analyzer
  * By Gerald Combs <gerald@zing.org>
@@ -389,10 +389,15 @@ get_column_title(gint col) {
   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() {
+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;