[RFA,v2,09/13] Pass the correct argument to the observer in reread_symbols

Message ID 20180721184720.3263-10-tom@tromey.com
State New
Headers show
  • Add -Wunused-variable
Related show

Commit Message

Tom Tromey July 21, 2018, 6:47 p.m.
This is actually a patch I found via another route.  Joel had asked me
to write a test, but I still have not found the time to do this.
Meanwhile, -Wunused-variable also found this error.

2018-07-21  Tom Tromey  <tom@tromey.com>

	* symfile.c (reread_symbols): Notify iter, not objfile.
 gdb/ChangeLog | 4 ++++
 gdb/symfile.c | 2 +-
 2 files changed, 5 insertions(+), 1 deletion(-)



diff --git a/gdb/symfile.c b/gdb/symfile.c
index 3614aa3f955..20697d9dca2 100644
--- a/gdb/symfile.c
+++ b/gdb/symfile.c
@@ -2608,7 +2608,7 @@  reread_symbols (void)
 	 gdb::observers::new_objfile.notify (NULL) has been called by
 	 clear_symtab_users above.  Notify the new files now.  */
       for (auto iter : new_objfiles)
-	gdb::observers::new_objfile.notify (objfile);
+	gdb::observers::new_objfile.notify (iter);
       /* At least one objfile has changed, so we can consider that
          the executable we're debugging has changed too.  */