Re: [PATCH] Add remove-inferiors test

[Simon Marchi <simon dot marchi at ericsson dot com>]

On 16-08-17 02:49 PM, Pedro Alves wrote:
> On 08/17/2016 06:12 PM, Simon Marchi wrote:
> 
>> gdb/testsuite/ChangeLog:
>>
>> 	* gdb.multi/remove-inferiors.exp: New file.
>> 	* gdb.multi/remove-inferiors.c: New file.
> 
> OK.
> 
>> +proc test_remove_inferiors { } {
>> +    global binfile
>> +
>> +    clean_restart ${binfile}
>> +
>> +    # Add another inferior and switch to it.
>> +    add_inferior 2 "add second inferior"
>> +    switch_to_inferior 2 "switch to second inferior"
>> +
>> +    # Test that it is not possible to remove the current inferior.
>> +    gdb_test "remove-inferiors 2" \
>> +	     "warning: Can not remove current symbol inferior 2." \
>> +	     "can't remove current inferior"
> 
> Eh, this "symbol inferior" reference sounds like a bad
> global replace on my part, years ago.  The original multi-exec
> support called program spaces "symbol spaces" instead.
> 
> Follow up patch to remove that stray "symbol" preapproved.

Ahh that explains it, I thought it was odd but didn't think about
it too much.

Pushed the main patch, as well as this one that fixes the error message:


>From eb2332d78d4ef40a2696aa0f6c833ea26a739efc Mon Sep 17 00:00:00 2001
From: Simon Marchi <simon.marchi@ericsson.com>
Date: Wed, 17 Aug 2016 16:02:27 -0400
Subject: [PATCH] Fix remove-inferior error message

This error message should not contain the word symbol:

  (gdb) remove-inferiors 1
  Warning: Can not remove current symbol inferior 1.

gdb/ChangeLog:

	* inferior.c (remove_inferior_command): Fix error message.

gdb/testsuite/ChangeLog:

	* gdb.multi/remove-inferiors.exp (test_remove_inferiors): Fix
	expected error message.
---
 gdb/ChangeLog                                | 4 ++++
 gdb/inferior.c                               | 2 +-
 gdb/testsuite/ChangeLog                      | 5 +++++
 gdb/testsuite/gdb.multi/remove-inferiors.exp | 2 +-
 4 files changed, 11 insertions(+), 2 deletions(-)

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 51504a0..dbd90dd 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,5 +1,9 @@
 2016-08-17  Simon Marchi  <simon.marchi@ericsson.com>

+	* inferior.c (remove_inferior_command): Fix error message.
+
+2016-08-17  Simon Marchi  <simon.marchi@ericsson.com>
+
 	* inferior.c (exit_inferior_1): Remove comment.

 2016-08-15  Matthew Wahab  <matthew.wahab@arm.com>
diff --git a/gdb/inferior.c b/gdb/inferior.c
index 6507ae7..47d91c7 100644
--- a/gdb/inferior.c
+++ b/gdb/inferior.c
@@ -803,7 +803,7 @@ remove_inferior_command (char *args, int from_tty)

       if (inf == current_inferior ())
 	{
-	  warning (_("Can not remove current symbol inferior %d."), num);
+	  warning (_("Can not remove current inferior %d."), num);
 	  continue;
 	}

diff --git a/gdb/testsuite/ChangeLog b/gdb/testsuite/ChangeLog
index 6e01da2..d025133 100644
--- a/gdb/testsuite/ChangeLog
+++ b/gdb/testsuite/ChangeLog
@@ -1,5 +1,10 @@
 2016-08-17  Simon Marchi  <simon.marchi@ericsson.com>

+	* gdb.multi/remove-inferiors.exp (test_remove_inferiors): Fix
+	expected error message.
+
+2016-08-17  Simon Marchi  <simon.marchi@ericsson.com>
+
 	* gdb.multi/remove-inferiors.exp: New file.
 	* gdb.multi/remove-inferiors.c: New file.

diff --git a/gdb/testsuite/gdb.multi/remove-inferiors.exp b/gdb/testsuite/gdb.multi/remove-inferiors.exp
index c49b31d..d46beec 100644
--- a/gdb/testsuite/gdb.multi/remove-inferiors.exp
+++ b/gdb/testsuite/gdb.multi/remove-inferiors.exp
@@ -40,7 +40,7 @@ proc test_remove_inferiors { } {

     # Test that it is not possible to remove the current inferior.
     gdb_test "remove-inferiors 2" \
-	     "warning: Can not remove current symbol inferior 2." \
+	     "warning: Can not remove current inferior 2." \
 	     "can't remove current inferior"

     # Test that it is possible to remove a non-active non-current inferior.
-- 
2.9.3