[PATCH 2/4] gdbserver: Don't add extra NULL to program args
Simon Marchi
simark@simark.ca
Thu May 7 19:54:29 GMT 2020
On 2020-04-29 7:16 a.m., Michael Weghorn via Gdb-patches wrote:
> The vector holding the program args is passed as a parameter
> to target_create_inferior, which then passes it to
> stringify_argv for all platforms, where any NULL entry in
> the vector is ignored, so there seems to be no reason
> to actually add one after all.
>
> (Since the intention is to replace uses of stringify_argv with
> construct_inferior_arguments in a follow-up commit and that
> function doesn't currently handle such NULL arguments, it
> would otherwise have to be extended.)
Thanks, this LGTM.
And a big thank you for writing informative commit messages up front, those
help a lot understand the need for your changes.
Simon
More information about the Gdb-patches
mailing list