Use bfd_cache_close, instead of accessing bfd->iostream directly.
[deliverable/binutils-gdb.git] / binutils / ar.c
index ad9e59d347e0317e8331d2498e5fdd39e19ab258..b85d62ecf0d7ba26fd543c04a1be5e5f038230a0 100644 (file)
@@ -1,30 +1,31 @@
 /* ar.c - Archive modify and extract.
-   Copyright 1991, 92, 93, 94, 95, 96, 97, 98, 1999
+   Copyright 1991, 1992, 1993, 1994, 1995, 1996, 1997, 1998, 1999, 2000,
+   2001, 2002, 2003
    Free Software Foundation, Inc.
 
-This file is part of GNU Binutils.
+   This file is part of GNU Binutils.
 
-This program is free software; you can redistribute it and/or modify
-it under the terms of the GNU General Public License as published by
-the Free Software Foundation; either version 2 of the License, or
-(at your option) any later version.
+   This program is free software; you can redistribute it and/or modify
+   it under the terms of the GNU General Public License as published by
+   the Free Software Foundation; either version 2 of the License, or
+   (at your option) any later version.
 
-This program is distributed in the hope that it will be useful,
-but WITHOUT ANY WARRANTY; without even the implied warranty of
-MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-GNU General Public License for more details.
+   This program is distributed in the hope that it will be useful,
+   but WITHOUT ANY WARRANTY; without even the implied warranty of
+   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+   GNU General Public License for more details.
 
-You should have received a copy of the GNU General Public License
-along with this program; if not, write to the Free Software
-Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.  */
+   You should have received a copy of the GNU General Public License
+   along with this program; if not, write to the Free Software
+   Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.  */
 \f
 /*
    Bugs: should use getopt the way tar does (complete w/optional -) and
    should have long options too. GNU ar used to check file against filesystem
    in quick_update and replace operations (would check mtime). Doesn't warn
    when name truncated. No way to specify pos_end. Error messages should be
-   more consistant.
-*/
+   more consistent.  */
+
 #include "bfd.h"
 #include "libiberty.h"
 #include "progress.h"
@@ -32,6 +33,8 @@ Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.  */
 #include "aout/ar.h"
 #include "libbfd.h"
 #include "arsup.h"
+#include "filenames.h"
+#include "binemul.h"
 #include <sys/stat.h>
 
 #ifdef __GO32___
@@ -51,54 +54,30 @@ Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.  */
 /* Kludge declaration from BFD!  This is ugly!  FIXME!  XXX */
 
 struct ar_hdr *
-  bfd_special_undocumented_glue PARAMS ((bfd * abfd, const char *filename));
+  bfd_special_undocumented_glue (bfd * abfd, const char *filename);
 
 /* Static declarations */
 
-static void
-mri_emul PARAMS ((void));
-
-static const char *
-normalize PARAMS ((const char *, bfd *));
-
-static void
-remove_output PARAMS ((void));
-
-static void
-map_over_members PARAMS ((bfd *, void (*)(bfd *), char **, int));
-
-static void
-print_contents PARAMS ((bfd * member));
-
-static void
-delete_members PARAMS ((bfd *, char **files_to_delete));
+static void mri_emul (void);
+static const char *normalize (const char *, bfd *);
+static void remove_output (void);
+static void map_over_members (bfd *, void (*)(bfd *), char **, int);
+static void print_contents (bfd * member);
+static void delete_members (bfd *, char **files_to_delete);
 
 #if 0
-static void
-do_quick_append PARAMS ((const char *archive_filename,
-                        char **files_to_append));
+static void do_quick_append
+  (const char *archive_filename, char **files_to_append);
 #endif
 
-static void
-move_members PARAMS ((bfd *, char **files_to_move));
-
-static void
-replace_members PARAMS ((bfd *, char **files_to_replace, boolean quick));
-
-static void
-print_descr PARAMS ((bfd * abfd));
-
-static void
-write_archive PARAMS ((bfd *));
-
-static void
-ranlib_only PARAMS ((const char *archname));
-
-static void
-ranlib_touch PARAMS ((const char *archname));
-
-static void
-usage PARAMS ((int));
+static void move_members (bfd *, char **files_to_move);
+static void replace_members
+  (bfd *, char **files_to_replace, bfd_boolean quick);
+static void print_descr (bfd * abfd);
+static void write_archive (bfd *);
+static void ranlib_only (const char *archname);
+static void ranlib_touch (const char *archname);
+static void usage (int);
 \f
 /** Globals and flags */
 
@@ -124,7 +103,7 @@ int newer_only = 0;
 
 /* Controls the writing of an archive symbol table (in BSD: a __.SYMDEF
    member).  -1 means we've been explicitly asked to not write a symbol table;
-   +1 means we've been explictly asked to write it;
+   +1 means we've been explicitly asked to write it;
    0 is the default.
    Traditionally, the default in BSD has been to not write the table.
    However, for POSIX.2 compliance the default is now to write a symbol table
@@ -145,25 +124,25 @@ enum pos
   } postype = pos_default;
 
 static bfd **
-get_pos_bfd PARAMS ((bfd **, enum pos, const char *));
+get_pos_bfd (bfd **, enum pos, const char *);
 
-/* For extract/delete only.  If COUNTED_NAME_MODE is true, we only
+/* For extract/delete only.  If COUNTED_NAME_MODE is TRUE, we only
    extract the COUNTED_NAME_COUNTER instance of that name.  */
-static boolean counted_name_mode = 0;
+static bfd_boolean counted_name_mode = 0;
 static int counted_name_counter = 0;
 
 /* Whether to truncate names of files stored in the archive.  */
-static boolean ar_truncate = false;
+static bfd_boolean ar_truncate = FALSE;
 
 /* Whether to use a full file name match when searching an archive.
    This is convenient for archives created by the Microsoft lib
    program.  */
-static boolean full_pathname = false;
+static bfd_boolean full_pathname = FALSE;
 
 int interactive = 0;
 
 static void
-mri_emul ()
+mri_emul (void)
 {
   interactive = isatty (fileno (stdin));
   yyparse ();
@@ -174,11 +153,7 @@ mri_emul ()
    whose name matches one in FILES.  */
 
 static void
-map_over_members (arch, function, files, count)
-     bfd *arch;
-     void (*function) PARAMS ((bfd *));
-     char **files;
-     int count;
+map_over_members (bfd *arch, void (*function)(bfd *), char **files, int count)
 {
   bfd *head;
   int match_count;
@@ -201,7 +176,7 @@ map_over_members (arch, function, files, count)
 
   for (; count > 0; files++, count--)
     {
-      boolean found = false;
+      bfd_boolean found = FALSE;
 
       match_count = 0;
       for (head = arch->next; head; head = head->next)
@@ -215,18 +190,18 @@ map_over_members (arch, function, files, count)
              bfd_stat_arch_elt (head, &buf);
            }
          if ((head->filename != NULL) &&
-             (!strcmp (normalize (*files, arch), head->filename)))
+             (!FILENAME_CMP (normalize (*files, arch), head->filename)))
            {
              ++match_count;
              if (counted_name_mode
-                 && match_count != counted_name_counter) 
+                 && match_count != counted_name_counter)
                {
                  /* Counting, and didn't match on count; go on to the
                      next one.  */
                  continue;
                }
 
-             found = true;
+             found = TRUE;
              function (head);
            }
        }
@@ -236,20 +211,19 @@ map_over_members (arch, function, files, count)
     }
 }
 \f
-boolean operation_alters_arch = false;
+bfd_boolean operation_alters_arch = FALSE;
 
 static void
-usage (help)
-     int help;
+usage (int help)
 {
   FILE *s;
 
   s = help ? stdout : stderr;
-  
+
   if (! is_ranlib)
     {
       /* xgettext:c-format */
-      fprintf (s, _("Usage: %s [-]{dmpqrstx}[abcfilNoPsSuvV] [member-name] [count] archive-file file...\n"),
+      fprintf (s, _("Usage: %s [emulation options] [-]{dmpqrstx}[abcfilNoPsSuvV] [member-name] [count] archive-file file...\n"),
               program_name);
       /* xgettext:c-format */
       fprintf (s, _("       %s -M [<mri-script]\n"), program_name);
@@ -275,15 +249,23 @@ usage (help)
       fprintf (s, _("  [S]          - do not build a symbol table\n"));
       fprintf (s, _("  [v]          - be verbose\n"));
       fprintf (s, _("  [V]          - display the version number\n"));
+
+      ar_emul_usage (s);
     }
   else
-    /* xgettext:c-format */
-    fprintf (s, _("Usage: %s [-vV] archive\n"), program_name);
+    {
+      /* xgettext:c-format */
+      fprintf (s, _("Usage: %s [options] archive\n"), program_name);
+      fprintf (s, _(" Generate an index to speed access to archives\n"));
+      fprintf (s, _(" The options are:\n\
+  -h --help                    Print this help message\n\
+  -V --version                 Print version information\n"));
+    }
 
   list_supported_targets (program_name, stderr);
 
   if (help)
-    fprintf (s, _("Report bugs to bug-gnu-utils@gnu.org\n"));
+    fprintf (s, _("Report bugs to %s\n"), REPORT_BUGS_TO);
 
   xexit (help ? 0 : 1);
 }
@@ -292,9 +274,7 @@ usage (help)
    name which we will use in an archive.  */
 
 static const char *
-normalize (file, abfd)
-     const char *file;
-     bfd *abfd;
+normalize (const char *file, bfd *abfd)
 {
   const char *filename;
 
@@ -302,6 +282,16 @@ normalize (file, abfd)
     return file;
 
   filename = strrchr (file, '/');
+#ifdef HAVE_DOS_BASED_FILE_SYSTEM
+  {
+    /* We could have foo/bar\\baz, or foo\\bar, or d:bar.  */
+    char *bslash = strrchr (file, '\\');
+    if (filename == NULL || (bslash != NULL && bslash > filename))
+      filename = bslash;
+    if (filename == NULL && file[0] != '\0' && file[1] == ':')
+      filename = file + 1;
+  }
+#endif
   if (filename != (char *) NULL)
     filename++;
   else
@@ -330,12 +320,12 @@ static FILE *output_file = NULL;
 static bfd *output_bfd = NULL;
 
 static void
-remove_output ()
+remove_output (void)
 {
   if (output_filename != NULL)
     {
-      if (output_bfd != NULL && output_bfd->iostream != NULL)
-       fclose ((FILE *) (output_bfd->iostream));
+      if (output_bfd != NULL)
+       bfd_cache_close (output_bfd);
       if (output_file != NULL)
        fclose (output_file);
       unlink (output_filename);
@@ -345,10 +335,10 @@ remove_output ()
 /* The option parsing should be in its own function.
    It will be when I have getopt working.  */
 
+int main (int, char **);
+
 int
-main (argc, argv)
-     int argc;
-     char **argv;
+main (int argc, char **argv)
 {
   char *arg_ptr;
   char c;
@@ -362,9 +352,13 @@ main (argc, argv)
   int file_count;
   char *inarch_filename;
   int show_version;
+  int i;
 
 #if defined (HAVE_SETLOCALE) && defined (HAVE_LC_MESSAGES)
   setlocale (LC_MESSAGES, "");
+#endif
+#if defined (HAVE_SETLOCALE)
+  setlocale (LC_CTYPE, "");
 #endif
   bindtextdomain (PACKAGE, LOCALEDIR);
   textdomain (PACKAGE);
@@ -377,12 +371,22 @@ main (argc, argv)
       char *temp;
 
       temp = strrchr (program_name, '/');
+#ifdef HAVE_DOS_BASED_FILE_SYSTEM
+      {
+       /* We could have foo/bar\\baz, or foo\\bar, or d:bar.  */
+       char *bslash = strrchr (program_name, '\\');
+       if (temp == NULL || (bslash != NULL && bslash > temp))
+         temp = bslash;
+       if (temp == NULL && program_name[0] != '\0' && program_name[1] == ':')
+         temp = program_name + 1;
+      }
+#endif
       if (temp == NULL)
        temp = program_name;
       else
        ++temp;
       if (strlen (temp) >= 6
-         && strcmp (temp + strlen (temp) - 6, "ranlib") == 0)
+         && FILENAME_CMP (temp + strlen (temp) - 6, "ranlib") == 0)
        is_ranlib = 1;
       else
        is_ranlib = 0;
@@ -410,11 +414,20 @@ main (argc, argv)
 
   xatexit (remove_output);
 
+  for (i = 1; i < argc; i++)
+    if (! ar_emul_parse_arg (argv[i]))
+      break;
+  argv += (i - 1);
+  argc -= (i - 1);
+
   if (is_ranlib)
     {
-      boolean touch = false;
+      bfd_boolean touch = FALSE;
 
-      if (argc < 2 || strcmp (argv[1], "--help") == 0)
+      if (argc < 2
+         || strcmp (argv[1], "--help") == 0
+         || strcmp (argv[1], "-h") == 0
+         || strcmp (argv[1], "-H") == 0)
        usage (0);
       if (strcmp (argv[1], "-V") == 0
          || strcmp (argv[1], "-v") == 0
@@ -424,7 +437,7 @@ main (argc, argv)
       if (strcmp (argv[1], "-t") == 0)
        {
          ++arg_index;
-         touch = true;
+         touch = TRUE;
        }
       while (arg_index < argc)
        {
@@ -468,22 +481,22 @@ main (argc, argv)
            {
            case 'd':
              operation = delete;
-             operation_alters_arch = true;
+             operation_alters_arch = TRUE;
              break;
            case 'm':
              operation = move;
-             operation_alters_arch = true;
+             operation_alters_arch = TRUE;
              break;
            case 'p':
              operation = print_files;
              break;
            case 'q':
              operation = quick_append;
-             operation_alters_arch = true;
+             operation_alters_arch = TRUE;
              break;
            case 'r':
              operation = replace;
-             operation_alters_arch = true;
+             operation_alters_arch = TRUE;
              break;
            case 't':
              operation = print_table;
@@ -501,7 +514,7 @@ main (argc, argv)
          preserve_dates = 1;
          break;
        case 'V':
-         show_version = true;
+         show_version = TRUE;
          break;
        case 's':
          write_armap = 1;
@@ -528,17 +541,17 @@ main (argc, argv)
          mri_mode = 1;
          break;
        case 'N':
-         counted_name_mode = true;
+         counted_name_mode = TRUE;
          break;
        case 'f':
-         ar_truncate = true;
+         ar_truncate = TRUE;
          break;
        case 'P':
-         full_pathname = true;
+         full_pathname = TRUE;
          break;
        default:
          /* xgettext:c-format */
-         fprintf (stderr, _("%s: illegal option -- %c\n"), program_name, c);
+         non_fatal (_("illegal option -- %c"), c);
          usage (0);
        }
     }
@@ -580,12 +593,12 @@ main (argc, argv)
       if (postype != pos_default)
        posname = argv[arg_index++];
 
-      if (counted_name_mode) 
+      if (counted_name_mode)
        {
-          if (operation != extract && operation != delete) 
-            fatal (_("`N' is only meaningful with the `x' and 'd' options."));
+         if (operation != extract && operation != delete)
+            fatal (_("`N' is only meaningful with the `x' and `d' options."));
          counted_name_counter = atoi (argv[arg_index++]);
-          if (counted_name_counter <= 0)
+         if (counted_name_counter <= 0)
            fatal (_("Value for `N' must be positive."));
        }
 
@@ -621,7 +634,7 @@ main (argc, argv)
       if (operation == quick_append)
        {
          /* Note that quick appending to a non-existent archive creates it,
-            even if there are no files to append. */
+            even if there are no files to append.  */
          do_quick_append (inarch_filename, files);
          xexit (0);
        }
@@ -669,9 +682,7 @@ main (argc, argv)
          /* Shouldn't happen! */
        default:
          /* xgettext:c-format */
-         fprintf (stderr, _("%s: internal error -- this option not implemented\n"),
-                  program_name);
-         xexit (1);
+         fatal (_("internal error -- this option not implemented"));
        }
     }
 
@@ -682,9 +693,7 @@ main (argc, argv)
 }
 
 bfd *
-open_inarch (archive_filename, file)
-     const char *archive_filename;
-     const char *file;
+open_inarch (const char *archive_filename, const char *file)
 {
   const char *target;
   bfd **last_one;
@@ -699,12 +708,15 @@ open_inarch (archive_filename, file)
 
   if (stat (archive_filename, &sbuf) != 0)
     {
-#ifndef __GO32__
+#if !defined(__GO32__) || defined(__DJGPP__)
+
+      /* FIXME: I don't understand why this fragment was ifndef'ed
+        away for __GO32__; perhaps it was in the days of DJGPP v1.x.
+        stat() works just fine in v2.x, so I think this should be
+        removed.  For now, I enable it for DJGPP v2. -- EZ.  */
 
 /* KLUDGE ALERT! Temporary fix until I figger why
- * stat() is wrong ... think it's buried in GO32's IDT
- * - Jax
- */
+   stat() is wrong ... think it's buried in GO32's IDT - Jax */
       if (errno != ENOENT)
        bfd_fatal (archive_filename);
 #endif
@@ -738,6 +750,8 @@ open_inarch (archive_filename, file)
          || ! bfd_set_format (arch, bfd_archive)
          || ! bfd_close (arch))
        bfd_fatal (archive_filename);
+      else if (!silent_create)
+        non_fatal (_("creating %s"), archive_filename);
 
       /* If we die creating a new archive, don't leave it around.  */
       output_filename = archive_filename;
@@ -778,8 +792,7 @@ open_inarch (archive_filename, file)
 }
 
 static void
-print_contents (abfd)
-     bfd *abfd;
+print_contents (bfd *abfd)
 {
   int ncopied = 0;
   char *cbuf = xmalloc (BUFSIZE);
@@ -790,9 +803,10 @@ print_contents (abfd)
     fatal (_("internal stat error on %s"), bfd_get_filename (abfd));
 
   if (verbose)
-    printf ("\n<%s>\n\n", bfd_get_filename (abfd));
+    /* xgettext:c-format */
+    printf (_("\n<%s>\n\n"), bfd_get_filename (abfd));
 
-  bfd_seek (abfd, 0, SEEK_SET);
+  bfd_seek (abfd, (file_ptr) 0, SEEK_SET);
 
   size = buf.st_size;
   while (ncopied < size)
@@ -803,8 +817,7 @@ print_contents (abfd)
       if (tocopy > BUFSIZE)
        tocopy = BUFSIZE;
 
-      nread = bfd_read (cbuf, 1, tocopy, abfd);        /* oops -- broke
-                                                          abstraction!  */
+      nread = bfd_bread (cbuf, (bfd_size_type) tocopy, abfd);
       if (nread != tocopy)
        /* xgettext:c-format */
        fatal (_("%s is not a valid archive"),
@@ -826,8 +839,7 @@ print_contents (abfd)
    Gilmore  */
 
 void
-extract_file (abfd)
-     bfd *abfd;
+extract_file (bfd *abfd)
 {
   FILE *ostream;
   char *cbuf = xmalloc (BUFSIZE);
@@ -835,7 +847,7 @@ extract_file (abfd)
   long ncopied = 0;
   long size;
   struct stat buf;
-  
+
   if (bfd_stat_arch_elt (abfd, &buf) != 0)
     /* xgettext:c-format */
     fatal (_("internal stat error on %s"), bfd_get_filename (abfd));
@@ -844,11 +856,11 @@ extract_file (abfd)
   if (size < 0)
     /* xgettext:c-format */
     fatal (_("stat returns negative size for %s"), bfd_get_filename (abfd));
-  
+
   if (verbose)
     printf ("x - %s\n", bfd_get_filename (abfd));
 
-  bfd_seek (abfd, 0, SEEK_SET);
+  bfd_seek (abfd, (file_ptr) 0, SEEK_SET);
 
   ostream = NULL;
   if (size == 0)
@@ -872,7 +884,7 @@ extract_file (abfd)
        if (tocopy > BUFSIZE)
          tocopy = BUFSIZE;
 
-       nread = bfd_read (cbuf, 1, tocopy, abfd);
+       nread = bfd_bread (cbuf, (bfd_size_type) tocopy, abfd);
        if (nread != tocopy)
          /* xgettext:c-format */
          fatal (_("%s is not a valid archive"),
@@ -919,33 +931,37 @@ extract_file (abfd)
 /* Just do it quickly; don't worry about dups, armap, or anything like that */
 
 static void
-do_quick_append (archive_filename, files_to_append)
-     const char *archive_filename;
-     char **files_to_append;
+do_quick_append (const char *archive_filename, char **files_to_append)
 {
   FILE *ofile, *ifile;
   char *buf = xmalloc (BUFSIZE);
   long tocopy, thistime;
   bfd *temp;
   struct stat sbuf;
-  boolean newfile = false;
+  bfd_boolean newfile = FALSE;
   bfd_set_error (bfd_error_no_error);
 
   if (stat (archive_filename, &sbuf) != 0)
     {
 
-#ifndef __GO32__
+#if !defined(__GO32__) || defined(__DJGPP__)
+
+      /* FIXME: I don't understand why this fragment was ifndef'ed
+        away for __GO32__; perhaps it was in the days of DJGPP v1.x.
+        stat() works just fine in v2.x, so I think this should be
+        removed.  For now, I enable it for DJGPP v2.
+
+        (And yes, I know this is all unused, but somebody, someday,
+        might wish to resurrect this again... -- EZ.  */
 
 /* KLUDGE ALERT! Temporary fix until I figger why
- * stat() is wrong ... think it's buried in GO32's IDT
- * - Jax
- */
+   stat() is wrong ... think it's buried in GO32's IDT - Jax  */
 
       if (errno != ENOENT)
        bfd_fatal (archive_filename);
 #endif
 
-      newfile = true;
+      newfile = TRUE;
     }
 
   ofile = fopen (archive_filename, FOPEN_AUB);
@@ -960,9 +976,9 @@ do_quick_append (archive_filename, files_to_append)
     {
       bfd_fatal (archive_filename);
     }
-  if (newfile == false)
+  if (!newfile)
     {
-      if (bfd_check_format (temp, bfd_archive) != true)
+      if (!bfd_check_format (temp, bfd_archive))
        /* xgettext:c-format */
        fatal (_("%s is not an archive"), archive_filename);
     }
@@ -971,14 +987,13 @@ do_quick_append (archive_filename, files_to_append)
       fwrite (ARMAG, 1, SARMAG, ofile);
       if (!silent_create)
        /* xgettext:c-format */
-       fprintf (stderr, _("%s: creating %s\n"),
-                program_name, archive_filename);
+       non_fatal (_("creating %s"), archive_filename);
     }
 
   if (ar_truncate)
     temp->flags |= BFD_TRADITIONAL_FORMAT;
 
-  /* assume it's an achive, go straight to the end, sans $200 */
+  /* assume it's an archive, go straight to the end, sans $200 */
   fseek (ofile, 0, 2);
 
   for (; files_to_append && *files_to_append; ++files_to_append)
@@ -1028,8 +1043,7 @@ do_quick_append (archive_filename, files_to_append)
 #endif /* 0 */
 
 static void
-write_archive (iarch)
-     bfd *iarch;
+write_archive (bfd *iarch)
 {
   bfd *obfd;
   char *old_name, *new_name;
@@ -1061,7 +1075,7 @@ write_archive (iarch)
       obfd->flags |= BFD_TRADITIONAL_FORMAT;
     }
 
-  if (bfd_set_archive_head (obfd, contents_head) != true)
+  if (!bfd_set_archive_head (obfd, contents_head))
     bfd_fatal (old_name);
 
   if (!bfd_close (obfd))
@@ -1082,10 +1096,7 @@ write_archive (iarch)
    and should be a pos value.  */
 
 static bfd **
-get_pos_bfd (contents, default_pos, default_posname)
-     bfd **contents;
-     enum pos default_pos;
-     const char *default_posname;
+get_pos_bfd (bfd **contents, enum pos default_pos, const char *default_posname)
 {
   bfd **after_bfd = contents;
   enum pos realpos;
@@ -1110,7 +1121,7 @@ get_pos_bfd (contents, default_pos, default_posname)
   else
     {
       for (; *after_bfd; after_bfd = &(*after_bfd)->next)
-       if (strcmp ((*after_bfd)->filename, realposname) == 0)
+       if (FILENAME_CMP ((*after_bfd)->filename, realposname) == 0)
          {
            if (realpos == pos_after)
              after_bfd = &(*after_bfd)->next;
@@ -1121,13 +1132,11 @@ get_pos_bfd (contents, default_pos, default_posname)
 }
 
 static void
-delete_members (arch, files_to_delete)
-     bfd *arch;
-     char **files_to_delete;
+delete_members (bfd *arch, char **files_to_delete)
 {
   bfd **current_ptr_ptr;
-  boolean found;
-  boolean something_changed = false;
+  bfd_boolean found;
+  bfd_boolean something_changed = FALSE;
   int match_count;
 
   for (; *files_to_delete != NULL; ++files_to_delete)
@@ -1140,30 +1149,30 @@ delete_members (arch, files_to_delete)
 
       if (!strcmp (*files_to_delete, "__.SYMDEF"))
        {
-         arch->has_armap = false;
+         arch->has_armap = FALSE;
          write_armap = -1;
          continue;
        }
 
-      found = false;
+      found = FALSE;
       match_count = 0;
       current_ptr_ptr = &(arch->next);
       while (*current_ptr_ptr)
        {
-         if (strcmp (normalize (*files_to_delete, arch),
-                     (*current_ptr_ptr)->filename) == 0)
+         if (FILENAME_CMP (normalize (*files_to_delete, arch),
+                           (*current_ptr_ptr)->filename) == 0)
            {
              ++match_count;
              if (counted_name_mode
-                 && match_count != counted_name_counter) 
+                 && match_count != counted_name_counter)
                {
                  /* Counting, and didn't match on count; go on to the
                      next one.  */
                }
              else
                {
-                 found = true;
-                 something_changed = true;
+                 found = TRUE;
+                 something_changed = TRUE;
                  if (verbose)
                    printf ("d - %s\n",
                            *files_to_delete);
@@ -1175,7 +1184,7 @@ delete_members (arch, files_to_delete)
          current_ptr_ptr = &((*current_ptr_ptr)->next);
        }
 
-      if (verbose && found == false)
+      if (verbose && !found)
        {
          /* xgettext:c-format */
          printf (_("No member named `%s'\n"), *files_to_delete);
@@ -1184,7 +1193,7 @@ delete_members (arch, files_to_delete)
       ;
     }
 
-  if (something_changed == true)
+  if (something_changed)
     write_archive (arch);
   else
     output_filename = NULL;
@@ -1194,9 +1203,7 @@ delete_members (arch, files_to_delete)
 /* Reposition existing members within an archive */
 
 static void
-move_members (arch, files_to_move)
-     bfd *arch;
-     char **files_to_move;
+move_members (bfd *arch, char **files_to_move)
 {
   bfd **after_bfd;             /* New entries go after this one */
   bfd **current_ptr_ptr;       /* cdr pointer into contents */
@@ -1207,8 +1214,8 @@ move_members (arch, files_to_move)
       while (*current_ptr_ptr)
        {
          bfd *current_ptr = *current_ptr_ptr;
-         if (strcmp (normalize (*files_to_move, arch),
-                     current_ptr->filename) == 0)
+         if (FILENAME_CMP (normalize (*files_to_move, arch),
+                           current_ptr->filename) == 0)
            {
              /* Move this file to the end of the list - first cut from
                 where it is.  */
@@ -1230,9 +1237,8 @@ move_members (arch, files_to_move)
          current_ptr_ptr = &((*current_ptr_ptr)->next);
        }
       /* xgettext:c-format */
-      fprintf (stderr, _("%s: no entry %s in archive %s!\n"),
-              program_name, *files_to_move, arch->filename);
-      xexit (1);
+      fatal (_("no entry %s in archive %s!"), *files_to_move, arch->filename);
+
     next_file:;
     }
 
@@ -1242,16 +1248,12 @@ move_members (arch, files_to_move)
 /* Ought to default to replacing in place, but this is existing practice!  */
 
 static void
-replace_members (arch, files_to_move, quick)
-     bfd *arch;
-     char **files_to_move;
-     boolean quick;
+replace_members (bfd *arch, char **files_to_move, bfd_boolean quick)
 {
-  boolean changed = false;
+  bfd_boolean changed = FALSE;
   bfd **after_bfd;             /* New entries go after this one */
   bfd *current;
   bfd **current_ptr;
-  bfd *temp;
 
   while (files_to_move && *files_to_move)
     {
@@ -1264,8 +1266,8 @@ replace_members (arch, files_to_move, quick)
 
              /* For compatibility with existing ar programs, we
                 permit the same file to be added multiple times.  */
-             if (strcmp (normalize (*files_to_move, arch),
-                         normalize (current->filename, arch)) == 0
+             if (FILENAME_CMP (normalize (*files_to_move, arch),
+                               normalize (current->filename, arch)) == 0
                  && current->arelt_data != NULL)
                {
                  if (newer_only)
@@ -1280,7 +1282,8 @@ replace_members (arch, files_to_move, quick)
                        }
                      if (bfd_stat_arch_elt (current, &asbuf) != 0)
                        /* xgettext:c-format */
-                       fatal (_("internal stat error on %s"), current->filename);
+                       fatal (_("internal stat error on %s"),
+                              current->filename);
 
                      if (fsbuf.st_mtime <= asbuf.st_mtime)
                        goto next_file;
@@ -1288,25 +1291,14 @@ replace_members (arch, files_to_move, quick)
 
                  after_bfd = get_pos_bfd (&arch->next, pos_after,
                                           current->filename);
-                 temp = *after_bfd;
-
-                 *after_bfd = bfd_openr (*files_to_move, NULL);
-                 if (*after_bfd == (bfd *) NULL)
-                   {
-                     bfd_fatal (*files_to_move);
-                   }
-                 (*after_bfd)->next = temp;
-
-                 /* snip out this entry from the chain */
-                 *current_ptr = (*current_ptr)->next;
-
-                 if (verbose)
+                 if (ar_emul_replace (after_bfd, *files_to_move,
+                                      verbose))
                    {
-                     printf ("r - %s\n", *files_to_move);
+                     /* Snip out this entry from the chain.  */
+                     *current_ptr = (*current_ptr)->next;
+                     changed = TRUE;
                    }
 
-                 changed = true;
-
                  goto next_file;
                }
              current_ptr = &(current->next);
@@ -1314,22 +1306,11 @@ replace_members (arch, files_to_move, quick)
        }
 
       /* Add to the end of the archive.  */
-
       after_bfd = get_pos_bfd (&arch->next, pos_end, NULL);
-      temp = *after_bfd;
-      *after_bfd = bfd_openr (*files_to_move, NULL);
-      if (*after_bfd == (bfd *) NULL)
-       {
-         bfd_fatal (*files_to_move);
-       }
-      if (verbose)
-       {
-         printf ("a - %s\n", *files_to_move);
-       }
-
-      (*after_bfd)->next = temp;
 
-      changed = true;
+      if (get_file_size (* files_to_move) > 0
+         && ar_emul_append (after_bfd, *files_to_move, verbose))
+       changed = TRUE;
 
     next_file:;
 
@@ -1343,11 +1324,12 @@ replace_members (arch, files_to_move, quick)
 }
 
 static void
-ranlib_only (archname)
-     const char *archname;
+ranlib_only (const char *archname)
 {
   bfd *arch;
 
+  if (get_file_size (archname) < 1)
+    return;
   write_armap = 1;
   arch = open_inarch (archname, (char *) NULL);
   if (arch == NULL)
@@ -1358,8 +1340,7 @@ ranlib_only (archname)
 /* Update the timestamp of the symbol map of an archive.  */
 
 static void
-ranlib_touch (archname)
-     const char *archname;
+ranlib_touch (const char *archname)
 {
 #ifdef __GO32__
   /* I don't think updating works on go32.  */
@@ -1369,6 +1350,8 @@ ranlib_touch (archname)
   bfd *arch;
   char **matching;
 
+  if (get_file_size (archname) < 1)
+    return;
   f = open (archname, O_RDWR | O_BINARY, 0);
   if (f < 0)
     {
@@ -1404,8 +1387,7 @@ ranlib_touch (archname)
 /* Things which are interesting to map over all or some of the files: */
 
 static void
-print_descr (abfd)
-     bfd *abfd;
+print_descr (bfd *abfd)
 {
   print_arelt_descr (stdout, abfd, verbose);
 }
This page took 0.040033 seconds and 4 git commands to generate.