[rfa] Use get_number_or_range for kill inferior etc.

Pedro Alves pedro@codesourcery.com
Fri Feb 25 21:45:00 GMT 2011


On Friday 25 February 2011 20:17:09, Michael Snyder wrote:
> -  tp = any_thread_of_process (pid);
> -  if (!tp)
> -    error (_("Inferior has no threads."));
> +      tp = any_thread_of_process (pid);
> +      if (!tp)
> +       error (_("Inferior has no threads."));

This is fine with me, but I think all the errors should
be updated to include the inferior ID that caused the
trouble.

>      error (_("Requires argument (inferior id to detach)"));

Probably want to make these plural as well.

On Friday 25 February 2011 20:17:09, Michael Snyder wrote:
> +gdb_test_no_output "remove-inferior 2-3" "remove-inferior 2-3"

Plural?

If you only have say inferiors 1, 3 and 5, these commands will
error out if you do 'CMD 1-5', claiming 2 or 4 are invalid.
Not sure that's user friendly.

And you'll need to update the docs as well, at least
because you renamed some commands.

-- 
Pedro Alves



More information about the Gdb-patches mailing list