Mirror of the gdb mailing list
 help / color / mirror / Atom feed
From: Eli Zaretskii <eliz@gnu.org>
To: gdb@sourceware.org
Subject: Re: Quoting, backslashes, CLI and MI
Date: Wed, 22 Feb 2006 19:25:00 -0000	[thread overview]
Message-ID: <ufymbkxlh.fsf@gnu.org> (raw)
In-Reply-To: <20060221213324.GA30729@nevyn.them.org> (message from Daniel 	Jacobowitz on Tue, 21 Feb 2006 16:33:24 -0500)

> Date: Tue, 21 Feb 2006 16:33:24 -0500
> From: Daniel Jacobowitz <drow@false.org>
> 
> I think that we want to continue using buildargv-style quoting for CLI
> commands, and that it would be desirable to use only MI-style
> quoting for MI commands.

I agree.  But that means MI commands that delegate to CLI will have to
process the arguments to modify the quoting, right?

> So I think that we should take this opportunity to fix up all MI
> commands to quote like the documentation says they do.

It would be good to fix that now, yes.

However, I Think we need 2 different styles of quoting: one for file
names, the other for strings.  Otherwise, supporting the Windows
backslashes will be hard.  Also, there are messy cases such as this:

    (gdb) break "foo bar.c":'MyClass::MyMethod'

(I'm not even sure I quoted it correctly ;-).  Can we really use the
same quoting rules for both the file-name and class/method name in
such situations?

> I'd like to fix up all the CLI and "set" commands to use buildargv style
> quoting, too.

If you agree with me on the two quoting styles, you will also agree
that we need two different flavors of buildargv.

> Andrew Cagney pointed out in a PR that we might need to update the
> readline filename completion, too.  I'm not sure if that's still relevant,
> it needs a third look.

It's relevant, at least wrt backslashes in Windows file names: if you
type TAB with backslash being the last character, completion won't
work (it does TRT when there are backslashes further to the left of
the cursor, though).  Also, quoted file names, with or without
whitespace, aren't completed correctly, no matter if they use forward
or backslashes.


  parent reply	other threads:[~2006-02-22 19:23 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-02-22  4:30 Daniel Jacobowitz
2006-02-22  4:35 ` Paul Koning
2006-02-22 19:57   ` Daniel Jacobowitz
2006-02-22 21:57     ` Paul Koning
2006-02-23  4:25       ` Mark Kettenis
2006-02-25  1:30       ` Eli Zaretskii
2006-02-22  4:40 ` Eli Zaretskii
2006-02-22  5:24   ` Daniel Jacobowitz
2006-02-22 19:30     ` Eli Zaretskii
2006-02-22 20:59       ` Daniel Jacobowitz
2006-02-22 17:39 ` Jim Blandy
2006-02-22 18:01   ` Daniel Jacobowitz
2006-02-22 18:05     ` Jim Blandy
2006-02-22 18:11       ` Daniel Jacobowitz
2006-02-22 19:24         ` Andrew STUBBS
2006-02-22 19:28           ` Daniel Jacobowitz
2006-02-22 19:50         ` Eli Zaretskii
2006-02-22 19:34     ` Eli Zaretskii
2006-02-22 19:53       ` Daniel Jacobowitz
2006-02-23 11:13         ` Eli Zaretskii
2006-02-22 19:25 ` Eli Zaretskii [this message]
2006-02-22 19:51   ` Daniel Jacobowitz
2006-02-23  4:32     ` Eli Zaretskii

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=ufymbkxlh.fsf@gnu.org \
    --to=eliz@gnu.org \
    --cc=gdb@sourceware.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox