diff mbox

Remove stale comment

Message ID 20160816144426.4366-1-simon.marchi@ericsson.com
State New, archived
Headers

Commit Message

Simon Marchi Aug. 16, 2016, 2:44 p.m. UTC 
  This comment seems outdated, since exiting an inferior does not remove
it.

gdb/ChangeLog:

	* inferior.c (exit_inferior_1): Remove comment.
---
 gdb/inferior.c | 2 --
 1 file changed, 2 deletions(-)

Comments

Pedro Alves Aug. 17, 2016, 1:57 p.m. UTC | #1 
On 08/16/2016 03:44 PM, Simon Marchi wrote:
> This comment seems outdated, since exiting an inferior does not remove
> it.

Right.  And that observer used to take an int pid argument instead of
the inferior pointer directly.

> 
> gdb/ChangeLog:
> 
> 	* inferior.c (exit_inferior_1): Remove comment.

OK.

Thanks,
Pedro Alves
Simon Marchi Aug. 17, 2016, 2:27 p.m. UTC | #2 
On 16-08-17 09:57 AM, Pedro Alves wrote:
> On 08/16/2016 03:44 PM, Simon Marchi wrote:
>> This comment seems outdated, since exiting an inferior does not remove
>> it.
> 
> Right.  And that observer used to take an int pid argument instead of
> the inferior pointer directly.
> 
>>
>> gdb/ChangeLog:
>>
>> 	* inferior.c (exit_inferior_1): Remove comment.
> 
> OK.

Thanks, pushed!
diff mbox

Patch

diff --git a/gdb/inferior.c b/gdb/inferior.c
index df53f74..78b9c60 100644
--- a/gdb/inferior.c
+++ b/gdb/inferior.c
@@ -250,8 +250,6 @@  exit_inferior_1 (struct inferior *inftoex, int silent)
 
   iterate_over_threads (delete_thread_of_inferior, &arg);
 
-  /* Notify the observers before removing the inferior from the list,
-     so that the observers have a chance to look it up.  */
   observer_notify_inferior_exit (inf);
 
   inf->pid = 0;