[ccache] [PATCH] add support for '@' parameters

Boie, Andrew P andrew.p.boie at intel.com
Mon Jul 30 15:09:24 MDT 2012


> -----Original Message-----
> From: joel.rosdahl at gmail.com [mailto:joel.rosdahl at gmail.com] On Behalf Of
> Joel Rosdahl
> > +                       file_args = args_init_from_string(argdata);
> 
> Isn't args_init_from_string too simplistic for parsing the contents?

Oof, yeah you're right. It seems that args_init_from_string needs to be significantly beefed-up. I don't suppose you know of some library code somewhere that does this? I should dig into the bash sources, IIRC it's the shell that turns the string into argv array...

> It would be nice if this code could be extracted to a new
> args_insert(struct args *args, struct args *to_insert, int pos)
> function or so. (There's a test suite for args_* in test/test_args.c.)

Done. I'll add a test case as well.

-Andrew


More information about the ccache mailing list