small coffgen.c tidy

[Alan Modra]

_bfd_coff_free_cached_info should always call
_bfd_generic_bfd_free_cached_info, even if _bfd_coff_free_symbols
returns an error.  (It won't return an error here, but let's not leave
anyone wondering about _bfd_coff_free_cached_info.)

	* coffgen.c (_bfd_coff_free_cached_info): Ignore return status
	of _bfd_coff_free_symbols.

diff --git a/bfd/coffgen.c b/bfd/coffgen.c
index dc5b0f40020..27bd54dcdd0 100644
--- a/bfd/coffgen.c
+++ b/bfd/coffgen.c
@@ -3295,8 +3295,7 @@ _bfd_coff_free_cached_info (bfd *abfd)
 	 Do not clear the keep_syms and keep_strings flags.
 	 These may have been set by pe_ILF_build_a_bfd() indicating
 	 that the syms and strings pointers are not to be freed.  */
-      if (!_bfd_coff_free_symbols (abfd))
-	return false;
+      _bfd_coff_free_symbols (abfd);
     }
 
   return _bfd_generic_bfd_free_cached_info (abfd);

-- 
Alan Modra