help-gengetopt
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [help-gengetopt] config parser patch for more graceful parsing


From: Lorenzo Bettini
Subject: Re: [help-gengetopt] config parser patch for more graceful parsing
Date: Fri, 28 Sep 2007 18:39:07 +0200
User-agent: Mozilla-Thunderbird 2.0.0.4 (X11/20070828)

Papp Gyozo (VBuster) wrote:
Hi,

some time ago I'd sent this patch but somehow it was silently forgotten without 
any notice. As the comment states this patch allows more freedom in the config 
file syntax; values will be read to the end of line or the comment sign (#).

This way one neither has to enclose nor escape string like these:

name = James Mc'Gregor
key  = MC JMS

I hope this patch could be in the next package.

--- gengetopt-2.18.orig/src/skels/c_source.h_skel       Fri Jul 14 20:07:31 2006
+++ gengetopt-2.18/src/skels/c_source.h_skel    Tue Nov 21 19:36:40 2006
@@ -653,8 +653,11 @@
             }
         }
       else
-        { /* read up the remaining part up to a delimiter */
-          next_token = strcspn (farg, " \t\r\n#\'\"");
+        { /* read up the remaining part up to end of line or comment sign */
+          char *ch = (str_index + (next_token = strcspn (farg, "\r\n#")));
+          /* remove trailing whitespaces */
+          for ( --ch; (*ch >= '\t' && *ch <= '\r') || *ch == ' '; --ch )
+            --next_token;
           str_index += next_token;
         }


Hi

I'm really sorry about that, it must have escaped me.

I noticed that the strcspn does not include ' nor " is that correct?

thanks and again my apologies :-)
cheers
        Lorenzo

--
Lorenzo Bettini, PhD in Computer Science, DSI, Univ. di Firenze
ICQ# lbetto, 16080134     (GNU/Linux User # 158233)
HOME: http://www.lorenzobettini.it MUSIC: http://www.purplesucker.com
http://www.myspace.com/supertrouperabba
BLOGS: http://tronprog.blogspot.com  http://longlivemusic.blogspot.com
http://www.gnu.org/software/src-highlite
http://www.gnu.org/software/gengetopt
http://www.gnu.org/software/gengen http://doublecpp.sourceforge.net




reply via email to

[Prev in Thread] Current Thread [Next in Thread]