From: Tom Tromey <tom@tromey.com>
Date: Sat, 27 Sep 2025 21:12:34 +0000 (-0600)
Subject: Remove an extraneous 'return' from void method
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=7d06ed71b84985102ed3a054884f1f6accec883f;p=thirdparty%2Fbinutils-gdb.git

Remove an extraneous 'return' from void method

cooked_index_functions::map_symbol_filenames returns 'void' but also
does "return ... expression".  While valid, this seems strange.  This
patch removes the unnecessary 'return'.

Reviewed-By: Guinevere Larsen <guinevere@redhat.com>
---

diff --git a/gdb/dwarf2/cooked-index.h b/gdb/dwarf2/cooked-index.h
index 42d7ec3313f..90a18af971e 100644
--- a/gdb/dwarf2/cooked-index.h
+++ b/gdb/dwarf2/cooked-index.h
@@ -260,8 +260,8 @@ struct cooked_index_functions : public dwarf2_base_index_functions
 			     bool need_fullname) override
   {
     wait (objfile, true);
-    return (dwarf2_base_index_functions::map_symbol_filenames
-	    (objfile, fun, need_fullname));
+    dwarf2_base_index_functions::map_symbol_filenames (objfile, fun,
+						       need_fullname);
   }
 
   void compute_main_name (struct objfile *objfile) override