gdb: remove empty clear_solib functions
authorSimon Marchi <simon.marchi@efficios.com>
Mon, 2 Oct 2023 18:52:09 +0000 (14:52 -0400)
committerSimon Marchi <simon.marchi@efficios.com>
Thu, 19 Oct 2023 14:57:51 +0000 (10:57 -0400)
Make the target_so_ops::clear_solib method optional, remove two empty
implementations.

Change-Id: Ifda297d50c74327d337091c58cdb5b3b60382591
Approved-By: Pedro Alves <pedro@palves.net>
Reviewed-By: Reviewed-By: Lancelot Six <lancelot.six@amd.com>
gdb/solib-aix.c
gdb/solib-target.c
gdb/solib.c

index fd6b1e7a4543234fec4fd058330b02caeef2bd1c..0b6ad83eed9207855b41f6856d1c83920bd0aa3a 100644 (file)
@@ -373,14 +373,6 @@ solib_aix_free_so (struct so_list *so)
   delete li;
 }
 
-/* Implement the "clear_solib" target_so_ops method.  */
-
-static void
-solib_aix_clear_solib (void)
-{
-  /* Nothing needed.  */
-}
-
 /* Compute and return the OBJFILE's section_offset array, using
    the associated loader info (INFO).  */
 
@@ -715,7 +707,7 @@ const struct target_so_ops solib_aix_so_ops =
   solib_aix_relocate_section_addresses,
   solib_aix_free_so,
   nullptr,
-  solib_aix_clear_solib,
+  nullptr,
   solib_aix_solib_create_inferior_hook,
   solib_aix_current_sos,
   solib_aix_open_symbol_file_object,
index ca9478f2ec306d765fb61b9ba7be410cfa0cea9a..865235de2995a13c9170f1bc0daad82be07dcbc5 100644 (file)
@@ -281,12 +281,6 @@ solib_target_solib_create_inferior_hook (int from_tty)
   /* Nothing needed.  */
 }
 
-static void
-solib_target_clear_solib (void)
-{
-  /* Nothing needed.  */
-}
-
 static void
 solib_target_free_so (struct so_list *so)
 {
@@ -440,7 +434,7 @@ const struct target_so_ops solib_target_so_ops =
   solib_target_relocate_section_addresses,
   solib_target_free_so,
   nullptr,
-  solib_target_clear_solib,
+  nullptr,
   solib_target_solib_create_inferior_hook,
   solib_target_current_sos,
   solib_target_open_symbol_file_object,
index a2a8a031f343b98b24a0fa4c95c526dbd1e96124..e8211814283a12da4c632278cf70ef31c7b88390 100644 (file)
@@ -1240,7 +1240,8 @@ clear_solib (void)
       free_so (so);
     }
 
-  ops->clear_solib ();
+  if (ops->clear_solib != nullptr)
+    ops->clear_solib ();
 }
 
 /* Shared library startup support.  When GDB starts up the inferior,
@@ -1409,7 +1410,8 @@ reload_shared_libraries (const char *ignored, int from_tty,
     {
       /* Reset or free private data structures not associated with
         so_list entries.  */
-      ops->clear_solib ();
+      if (ops->clear_solib != nullptr)
+       ops->clear_solib ();
 
       /* Remove any previous solib event breakpoint.  This is usually
         done in common code, at breakpoint_init_inferior time, but