Fix override of common symbols for a.out

The default should be bfd_link_common_skip_none, as in the original
patch at https://sourceware.org/ml/binutils/2002-07/msg00717.html

	* aoutx.h (aout_link_check_ar_symbols): Remove default and handle
	bfd_link_common_skip_none in switch.

diff --git a/bfd/ChangeLog b/bfd/ChangeLog
index 85ea86a004b748b69dd4e58fa94be04f995871d1..19364c0c779ed07c3722044dfff7600310b502ac 100644 (file)
--- a/bfd/ChangeLog
+++ b/bfd/ChangeLog
@@ -1,3 +1,8 @@
+2018-01-12  Gunther Nikl  <gnikl@users.sourceforge.net>
+
+       * aoutx.h (aout_link_check_ar_symbols): Remove default and handle
+       bfd_link_common_skip_none in switch.
+
 2018-01-12  Alan Modra  <amodra@gmail.com>
 
        PR ld/22649

diff --git a/bfd/aoutx.h b/bfd/aoutx.h
index 6dc4c68678e4ccb77f16aba7c790116bb4f217e3..eec9c4ad2a194272a2164b4d632757a5e485a9f1 100644 (file)
--- a/bfd/aoutx.h
+++ b/bfd/aoutx.h
@@ -3366,13 +3366,14 @@ aout_link_check_ar_symbols (bfd *abfd,
 
              switch (info->common_skip_ar_symbols)
                {
+               case bfd_link_common_skip_none:
+                 break;
                case bfd_link_common_skip_text:
                  skip = (type == (N_TEXT | N_EXT));
                  break;
                case bfd_link_common_skip_data:
                  skip = (type == (N_DATA | N_EXT));
                  break;
-               default:
                case bfd_link_common_skip_all:
                  skip = 1;
                  break;