Re: [RFC PATCH 14/14] revert: Change insn sheet format

Hi again,

Jonathan Nieder writes:

Ramkumar Ramachandra wrote:

quoted

 I've intentionally left parse_cmdline_args unimplemented

[...]

quoted

                 Existing implementations in libraries like Glib are
 much too heavyweight.

Wait, how did glib enter the picture? :)

Completely unrelated -- I just incidentally saw "g_shell_parse_argv"
in glib which does what we want, but it's an overkill.

The implementation of
shell-style quoting in [1] is not very complicated; perhaps it could
complement git's existing parsers for shell-style single-quoted
expressions and C-style double-quoted expressions in quote.c.

Okay.

Of course, a more basic question is whether we want to allow passing
arbitrary command-line arguments through the insn sheet at all (a
part of me wishes "no", at least at first).

I have an overwhelming desire to say "no", but I can't think of an alternative.

Could you give an example to illustrate what this functionality would
be used for?  I can understand wanting to pass "-s" and "-X" flags to
a merge insn and "-X" to pick, but that's as far as my imagination
goes.

I wasn't imagining anything else.  That's just it -- I've just been
breaking my head trying to figure out how to do it :|

-- Ram