On Tue, Mar 14, 2017 at 01:01:23AM -0400, Scott Kostyshak wrote:
> On Mon, Mar 13, 2017 at 11:03:40AM +0100, Jean-Marc Lasgouttes wrote:
>
> > Question, why don't you += to append stuff to a string?
>
> I just forget. I will do this for the next patch.
I think I realize why it's not natural to m
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
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 cle
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
On Mon, Mar 13, 2017 at 11:03:40AM +0100, Jean-Marc Lasgouttes wrote:
> Le 07/03/2017 à 06:08, Scott Kostyshak a écrit :
> > For the first patch, I changed the write() routine to add quotes around
> > the branch name. Otherwise, read() gets confused. But should I instead
> > have changed the read r
Le 07/03/2017 à 06:08, Scott Kostyshak a écrit :
For the first patch, I changed the write() routine to add quotes around
the branch name. Otherwise, read() gets confused. But should I instead
have changed the read routine()?
Lexer::quoteString does what you need. And actually you could just use
On Tue, Nov 01, 2016 at 08:29:10PM +0100, Jean-Marc Lasgouttes wrote:
> I am surprised that addArg does not need to add a space to the current
> argument.
It appears I got away with this because splitArg() handles double-quotes:
// Extracts arguments from str into args. Arguments are delimted
Le 30/10/2016 à 21:57, Scott Kostyshak a écrit :
As for the annoying code to build the command line, I'd say it would be nice
to have some helpers like FuncRequuest::addArg(std::string) or something
like that.
Done in the attached patch.
I am surprised that addArg does not need to add a space
On Fri, Oct 21, 2016 at 11:20:03AM +0200, Jean-Marc Lasgouttes wrote:
> Le 21/10/2016 à 04:47, Scott Kostyshak a écrit :
> > > Makes sense. I'll work on a better patch.
> >
> > Is the attached what you meant? If so I would audit the other calls to
> > LFUN_SET_COLOR.
>
> Yes, I think so. Note tha
Le 21/10/2016 à 04:47, Scott Kostyshak a écrit :
Makes sense. I'll work on a better patch.
Is the attached what you meant? If so I would audit the other calls to
LFUN_SET_COLOR.
Yes, I think so. Note that the double quotes are not really part of the
syntax, they can be omitted as before. The
On Fri, Oct 14, 2016 at 04:38:31PM -0400, Scott Kostyshak wrote:
> > The second solution would be to use
> >
> > lyx_name = cmd.getArg(0);
> > x11_name = cmd.getArg(1);
> >
> > In this case, the syntax for set-color would be
> > set-color foo bar
> > set-color "foo bar" baz
> > set-color foo "ba
On Fri, Oct 14, 2016 at 07:16:36PM +0200, Jean-Marc Lasgouttes wrote:
> Le 14/10/2016 à 17:51, Scott Kostyshak a écrit :
> > Any thoughts?
>
> I would propose instead to use the helpers getArg and getLongArg in
> FuncRequest. Two possibilities:
>
> lyx_name = cmd.getArg(0);
> x11_name = cmd.getLo
Le 14/10/2016 à 17:51, Scott Kostyshak a écrit :
Any thoughts?
I would propose instead to use the helpers getArg and getLongArg in
FuncRequest. Two possibilities:
lyx_name = cmd.getArg(0);
x11_name = cmd.getLongArg(1);
This will do what you want, at the price of a badly defined syntax: it
13 matches
Mail list logo