Complete.Org: Mailing Lists: Archives: freeciv-dev: September 2001:
[Freeciv-Dev] Re: Split patch (was Re: [RFC PATCH] init_techs)
Home

[Freeciv-Dev] Re: Split patch (was Re: [RFC PATCH] init_techs)

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: Justin Moore <justin@xxxxxxxxxxx>
Cc: Freeciv Developers <freeciv-dev@xxxxxxxxxxx>
Subject: [Freeciv-Dev] Re: Split patch (was Re: [RFC PATCH] init_techs)
From: Raimar Falke <hawk@xxxxxxxxxxxxxxxxxxxxxxx>
Date: Wed, 26 Sep 2001 10:47:07 +0200
Reply-to: rf13@xxxxxxxxxxxxxxxxxxxxxx

On Wed, Sep 26, 2001 at 01:35:38AM -0400, Justin Moore wrote:
> 
> > > > What about a strdup() in split?
> > >
> > >    I could, but I really think all memory allocation and de-allocation
> > > should be done at the same level, so-to-speak.
> >
> > What is the problem? split() will strdup() the string it gets and
> > frees it copy later.
> 
> char *buf = "   foo bar ";
> char *args[2];
> int found = split("\S", buf, args, 2);
> 
> If, within split, I do
>    char *copybuf = strdup(buf);
> and mess around with copybuf, I'm going to parse away the first few chars
> of it.  args[0] != copybuf, since I've cut away the first few characters.
> How will I know what to free?

I was thinking of that split() will free the copybuf.

> > >    On a side note, I'm going to remove the "automatically removes
> > > whitespace" implementation, and require the caller to pass a "\S" as a
> > > token if they want to split on whitespace.  If they want to remove
> > > whitespace surrounding other split tokens, they would pass in a "\s".  A
> > > "\S" implies "\s".
> > >
> > > split("\s,", buf, args, 5);
> > >
> > > will correctly parse
> > >
> > > Alphabet,Iron Working,Pottery, The Wheel
> > >
> > > Comments?
> >
> > I would rather like to see this explicitly as an argument of split.
> 
>    I think it's cleaner to keep fewer arguments.  I don't want this
> turning into MFCs, where you have to pass 17 NULLs or 0s into a function
> to get it right. ;p Just know some basic regex patters and it works.

Then make a

enum split_flags {
 REMOVE_SPLIT_ON_WS=1,
 REMOVE_REMOVE_WS=2
};

        Raimar

-- 
 email: rf13@xxxxxxxxxxxxxxxxx
 "On the eigth day, God started debugging"


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