Duy Nguyen <pclouds@xxxxxxxxx> writes: >> if (recurse_submodules) >> argv_array_push(&submodule_options, "--recurse-submodules"); > > Side note. It would be awesome if you could make parse_options() (or a > new function) do the reverse process: given a 'struct option' with > valid data, spit out argv_array. Less worrying about git-grep having > new option but not passed to subgrep by accident. You can have a new > flag to tell it to ignore certain options if you don't want to pass > all. It indeed would be awesome. I do not offhand know if it would be feasible or the result is easy to use without mistakes, though.