bfd/
[deliverable/binutils-gdb.git] / ld / ldmisc.c
index 3e8b3717e5af412a48a172889d69aac42daff82b..8af043aa2b24bc70be099ca8ffe0d98386e9f6c8 100644 (file)
@@ -1,6 +1,6 @@
 /* ldmisc.c
-   Copyright 1991, 1992, 1993, 1994, 1995, 1996, 1997, 1998, 1999,
-   2000, 2002, 2003
+   Copyright 1991, 1992, 1993, 1994, 1995, 1996, 1997, 1998, 1999, 2000,
+   2001, 2002, 2003, 2004, 2005, 2006
    Free Software Foundation, Inc.
    Written by Steve Chamberlain of Cygnus Support.
 
@@ -18,8 +18,8 @@
 
    You should have received a copy of the GNU General Public License
    along with GLD; see the file COPYING.  If not, write to the Free
-   Software Foundation, 59 Temple Place - Suite 330, Boston, MA
-   02111-1307, USA.  */
+   Software Foundation, 51 Franklin Street - Fifth Floor, Boston, MA
+   02110-1301, USA.  */
 
 #include "bfd.h"
 #include "bfdlink.h"
 #include "ldlex.h"
 #include "ldmain.h"
 #include "ldfile.h"
+#include "elf-bfd.h"
 
 /*
  %% literal %
+ %A section name from a section
+ %B filename from a bfd
+ %C clever filename:linenumber with function
+ %D like %C, but no function name
+ %E current bfd error or errno
  %F error is fatal
+ %G like %D, but only function name
+ %I filename from a lang_input_statement_type
  %P print program name
+ %R info about a relent
  %S print script file and linenumber
- %E current bfd error or errno
- %I filename from a lang_input_statement_type
- %B filename from a bfd
  %T symbol name
- %X no object output, fail return
  %V hex bfd_vma
- %v hex bfd_vma, no leading zeros
  %W hex bfd_vma with 0x with no leading zeros taking up 8 spaces
- %C clever filename:linenumber with function
- %D like %C, but no function name
- %G like %D, but only function name
- %R info about a relent
- %s arbitrary string, like printf
+ %X no object output, fail return
  %d integer, like printf
+ %ld long, like printf
+ %lu unsigned long, like printf
+ %s arbitrary string, like printf
  %u integer, like printf
+ %v hex bfd_vma, no leading zeros
 */
 
 static void
-vfinfo (FILE *fp, const char *fmt, va_list arg)
+vfinfo (FILE *fp, const char *fmt, va_list arg, bfd_boolean is_warning)
 {
   bfd_boolean fatal = FALSE;
 
@@ -76,10 +80,6 @@ vfinfo (FILE *fp, const char *fmt, va_list arg)
          fmt++;
          switch (*fmt++)
            {
-           default:
-             fprintf (fp, "%%%c", fmt[-1]);
-             break;
-
            case '%':
              /* literal % */
              putc ('%', fp);
@@ -158,11 +158,38 @@ vfinfo (FILE *fp, const char *fmt, va_list arg)
              }
              break;
 
+           case 'A':
+             /* section name from a section */
+             {
+               asection *sec = va_arg (arg, asection *);
+               bfd *abfd = sec->owner;
+               const char *group = NULL;
+               struct coff_comdat_info *ci;
+
+               fprintf (fp, "%s", sec->name);
+               if (abfd != NULL
+                   && bfd_get_flavour (abfd) == bfd_target_elf_flavour
+                   && elf_next_in_group (sec) != NULL
+                   && (sec->flags & SEC_GROUP) == 0)
+                 group = elf_group_name (sec);
+               else if (abfd != NULL
+                        && bfd_get_flavour (abfd) == bfd_target_coff_flavour
+                        && (ci = bfd_coff_get_comdat_section (sec->owner,
+                                                              sec)) != NULL)
+                 group = ci->name;
+               if (group != NULL)
+                 fprintf (fp, "[%s]", group);
+             }
+             break;
+
            case 'B':
              /* filename from a bfd */
              {
                bfd *abfd = va_arg (arg, bfd *);
-               if (abfd->my_archive)
+
+               if (abfd == NULL)
+                 fprintf (fp, "%s generated", program_name);
+               else if (abfd->my_archive)
                  fprintf (fp, "%s(%s)", abfd->my_archive->filename,
                           abfd->filename);
                else
@@ -241,50 +268,67 @@ vfinfo (FILE *fp, const char *fmt, va_list arg)
                const char *functionname;
                unsigned int linenumber;
                bfd_boolean discard_last;
-               char *sec_name;
 
                abfd = va_arg (arg, bfd *);
                section = va_arg (arg, asection *);
                offset = va_arg (arg, bfd_vma);
 
-               entry = (lang_input_statement_type *) abfd->usrdata;
-               if (entry != (lang_input_statement_type *) NULL
-                   && entry->asymbols != (asymbol **) NULL)
-                 asymbols = entry->asymbols;
+               if (abfd == NULL)
+                 {
+                   entry = NULL;
+                   asymbols = NULL;
+                 }
                else
                  {
-                   long symsize;
-                   long symbol_count;
-
-                   symsize = bfd_get_symtab_upper_bound (abfd);
-                   if (symsize < 0)
-                     einfo (_("%B%F: could not read symbols\n"), abfd);
-                   asymbols = xmalloc (symsize);
-                   symbol_count = bfd_canonicalize_symtab (abfd, asymbols);
-                   if (symbol_count < 0)
-                     einfo (_("%B%F: could not read symbols\n"), abfd);
-                   if (entry != (lang_input_statement_type *) NULL)
+                   entry = (lang_input_statement_type *) abfd->usrdata;
+                   if (entry != (lang_input_statement_type *) NULL
+                       && entry->asymbols != (asymbol **) NULL)
+                     asymbols = entry->asymbols;
+                   else
                      {
-                       entry->asymbols = asymbols;
-                       entry->symbol_count = symbol_count;
+                       long symsize;
+                       long sym_count;
+
+                       symsize = bfd_get_symtab_upper_bound (abfd);
+                       if (symsize < 0)
+                         einfo (_("%B%F: could not read symbols\n"), abfd);
+                       asymbols = xmalloc (symsize);
+                       sym_count = bfd_canonicalize_symtab (abfd, asymbols);
+                       if (sym_count < 0)
+                         einfo (_("%B%F: could not read symbols\n"), abfd);
+                       if (entry != (lang_input_statement_type *) NULL)
+                         {
+                           entry->asymbols = asymbols;
+                           entry->symbol_count = sym_count;
+                         }
                      }
                  }
 
-               sec_name = bfd_get_section_ident (section);
-               lfinfo (fp, "%B(%s+0x%v)", abfd,
-                       sec_name ? sec_name : section->name, offset);
-               if (sec_name)
-                 free (sec_name);
+               /* The GNU Coding Standard requires that error messages
+                  be of the form:
+                  
+                    source-file-name:lineno: message
 
+                  We do not always have a line number available so if
+                  we cannot find them we print out the section name and
+                  offset instread.  */
                discard_last = TRUE;
-               if (bfd_find_nearest_line (abfd, section, asymbols, offset,
-                                          &filename, &functionname,
-                                          &linenumber))
+               if (abfd != NULL
+                   && bfd_find_nearest_line (abfd, section, asymbols, offset,
+                                             &filename, &functionname,
+                                             &linenumber))
                  {
-                   bfd_boolean need_colon = TRUE;
-
                    if (functionname != NULL && fmt[-1] == 'C')
                      {
+                       /* Detect the case where we are printing out a
+                          message for the same function as the last
+                          call to vinfo ("%C").  In this situation do
+                          not print out the ABFD filename or the
+                          function name again.  Note - we do still
+                          print out the source filename, as this will
+                          allow programs that parse the linker's output
+                          (eg emacs) to correctly locate multiple
+                          errors in the same source file.  */
                        if (last_bfd == NULL
                            || last_file == NULL
                            || last_function == NULL
@@ -293,9 +337,8 @@ vfinfo (FILE *fp, const char *fmt, va_list arg)
                                && strcmp (last_file, filename) != 0)
                            || strcmp (last_function, functionname) != 0)
                          {
-                           lfinfo (fp, _(": In function `%T':\n"),
-                                   functionname);
-                           need_colon = FALSE;
+                           lfinfo (fp, _("%B: In function `%T':\n"),
+                                   abfd, functionname);
 
                            last_bfd = abfd;
                            if (last_file != NULL)
@@ -309,19 +352,21 @@ vfinfo (FILE *fp, const char *fmt, va_list arg)
                          }
                        discard_last = FALSE;
                      }
+                   else
+                     lfinfo (fp, "%B:", abfd);
 
                    if (filename != NULL)
-                     {
-                       if (need_colon)
-                         putc (':', fp);
-                       fputs (filename, fp);
-                     }
+                     fprintf (fp, "%s:", filename);
 
                    if (functionname != NULL && fmt[-1] == 'G')
-                     lfinfo (fp, ":%T", functionname);
+                     lfinfo (fp, "%T", functionname);
                    else if (filename != NULL && linenumber != 0)
-                     fprintf (fp, ":%u", linenumber);
+                     fprintf (fp, "%u", linenumber);
+                   else
+                     lfinfo (fp, "(%A+0x%v)", section, offset);
                  }
+               else
+                 lfinfo (fp, "%B:(%A+0x%v)", abfd, section, offset);
 
                if (asymbols != NULL && entry == NULL)
                  free (asymbols);
@@ -357,11 +402,30 @@ vfinfo (FILE *fp, const char *fmt, va_list arg)
              /* unsigned integer, like printf */
              fprintf (fp, "%u", va_arg (arg, unsigned int));
              break;
+
+           case 'l':
+             if (*fmt == 'd')
+               {
+                 fprintf (fp, "%ld", va_arg (arg, long));
+                 ++fmt;
+                 break;
+               }
+             else if (*fmt == 'u')
+               {
+                 fprintf (fp, "%lu", va_arg (arg, unsigned long));
+                 ++fmt;
+                 break;
+               }
+             /* Fall thru */
+
+           default:
+             fprintf (fp, "%%%c", fmt[-1]);
+             break;
            }
        }
     }
 
-  if (config.fatal_warnings)
+  if (is_warning && config.fatal_warnings)
     config.make_executable = FALSE;
 
   if (fatal)
@@ -421,7 +485,7 @@ info_msg (const char *fmt, ...)
   va_list arg;
 
   va_start (arg, fmt);
-  vfinfo (stdout, fmt, arg);
+  vfinfo (stdout, fmt, arg, FALSE);
   va_end (arg);
 }
 
@@ -433,7 +497,7 @@ einfo (const char *fmt, ...)
   va_list arg;
 
   va_start (arg, fmt);
-  vfinfo (stderr, fmt, arg);
+  vfinfo (stderr, fmt, arg, TRUE);
   va_end (arg);
 }
 
@@ -451,7 +515,7 @@ minfo (const char *fmt, ...)
   va_list arg;
 
   va_start (arg, fmt);
-  vfinfo (config.map_file, fmt, arg);
+  vfinfo (config.map_file, fmt, arg, FALSE);
   va_end (arg);
 }
 
@@ -461,7 +525,7 @@ lfinfo (FILE *file, const char *fmt, ...)
   va_list arg;
 
   va_start (arg, fmt);
-  vfinfo (file, fmt, arg);
+  vfinfo (file, fmt, arg, FALSE);
   va_end (arg);
 }
 \f
@@ -494,64 +558,3 @@ ld_abort (const char *file, int line, const char *fn)
   einfo (_("%P%F: please report this bug\n"));
   xexit (1);
 }
-
-bfd_boolean
-error_handler (int id, const char *fmt, ...)
-{
-  va_list arg;
-
-  va_start (arg, fmt);
-
-  switch (id)
-    {
-    default:
-      break;
-
-    /* We can be called with
-
-       error_handler (-LD_DEFINITION_IN_DISCARDED_SECTION, "", 0);
-
-       to make this error non-fatal and
-
-       error_handler (-LD_DEFINITION_IN_DISCARDED_SECTION, "", 1);
-
-       to make this error fatal.  */
-    case -LD_DEFINITION_IN_DISCARDED_SECTION:
-    case LD_DEFINITION_IN_DISCARDED_SECTION:
-      {
-       static struct bfd_hash_table *hash;
-       static int fatal = 1;
-       const char *name;
-
-       if (id == -LD_DEFINITION_IN_DISCARDED_SECTION)
-         {
-           fatal = va_arg (arg, int);
-           goto out;
-         }
-
-       name = va_arg (arg, const char *);
-       /* Only warn once about a particular undefined symbol.  */
-       if (hash == NULL)
-         {
-           hash = xmalloc (sizeof (struct bfd_hash_table));
-           if (! bfd_hash_table_init (hash, bfd_hash_newfunc))
-             einfo (_("%F%P: bfd_hash_table_init failed: %E\n"));
-         }
-
-       if (bfd_hash_lookup (hash, name, FALSE, FALSE) != NULL)
-         goto out;
-
-       if (bfd_hash_lookup (hash, name, TRUE, TRUE) == NULL)
-         einfo (_("%F%P: bfd_hash_lookup failed: %E\n"));
-
-       if (fatal)
-         config.make_executable = FALSE;
-      }
-      break;
-    }
-  vfinfo (stderr, fmt, arg);
-
-out:
-  va_end (arg);
-  return TRUE;
-}
This page took 0.0489 seconds and 4 git commands to generate.