On Sat, Mar 18, 2017 at 05:49:30PM +0100, Jean-Marc Lasgouttes wrote:
> Le 18/03/2017 à 17:02, Scott Kostyshak a écrit :
> > On Tue, Mar 14, 2017 at 01:01:23AM -0400, Scott Kostyshak wrote:
> > 
> > > I have a patch and it seems to work well. I want to clean it up before
> > > posting. I will post it again for review because even though it seems to
> > > work well, I wonder if there is a cleaner way of doing it. I will post
> > > a cleaned up version in a couple of days.
> > 
> > Attached are two patches. Are they what you had in mind?
> 
> Or maybe something much simpler like (not sure about the exact syntax, look
> up other examples)
> 
> Lexer lex(is)
> while (lex.next(true))
>       args.push_back(lex.getString());

OK I will take a look at this.

> It is not necessary to rewrite what is already done by Lexer IMO.

+1

> > Because I edit splitArg(), this could cause problems with other LFUNs.
> 
> You mean LFUNS that expect that \ is not interpreted? I would think that
> those would use the full argument directly, or use getLongArg().

OK good to know.

Thanks,

Scott

Attachment: signature.asc
Description: PGP signature

Reply via email to