* gas/hppa/reloc/reloc.exp: Minor fixes so that SOM & ELF can
[deliverable/binutils-gdb.git] / bfd / cache.c
index 1d4ee567ed135beb46de05c38b759dbdc80dc38c..cefae728f631d4a4fe93d9b42e9b459588224b1c 100644 (file)
@@ -1,6 +1,5 @@
-
 /* BFD library -- caching of file descriptors.
-   Copyright (C) 1990-1991 Free Software Foundation, Inc.
+   Copyright 1990, 1991, 1992 Free Software Foundation, Inc.
    Hacked by Steve Chamberlain of Cygnus Support (steve@cygnus.com).
 
 This file is part of BFD, the Binary File Descriptor library.
@@ -21,7 +20,7 @@ Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.  */
 
 /*
 SECTION
-       File Caching
+       File caching
 
        The file caching mechanism is embedded within BFD and allows
        the application to open as many BFDs as it wants without
@@ -29,15 +28,13 @@ SECTION
        limit (often as low as 20 open files).  The module in
        <<cache.c>> maintains a least recently used list of
        <<BFD_CACHE_MAX_OPEN>> files, and exports the name
-       <<bfd_cache_lookup>> which runs around and makes sure that
+       <<bfd_cache_lookup>>, which runs around and makes sure that
        the required BFD is open. If not, then it chooses a file to
        close, closes it and opens the one wanted, returning its file
        handle. 
 
 */
 
-/* $Id$ */
-
 #include "bfd.h"
 #include "sysdep.h"
 #include "libbfd.h"
@@ -47,7 +44,7 @@ INTERNAL_FUNCTION
        BFD_CACHE_MAX_OPEN macro
 
 DESCRIPTION
-       The maxiumum number of files which the cache will keep open at
+       The maximum number of files which the cache will keep open at
        one time.
 
 .#define BFD_CACHE_MAX_OPEN 10
@@ -55,6 +52,12 @@ DESCRIPTION
 */
 
 
+static boolean
+bfd_cache_delete PARAMS ((bfd *));
+
+/* Number of bfds on the chain.  All such bfds have their file open;
+   if it closed, they get snipd()d from the chain.  */
+
 static int open_files;
 
 static bfd *cache_sentinel;    /* Chain of BFDs with active fds we've
@@ -76,26 +79,23 @@ DESCRIPTION
 bfd *bfd_last_cache;
 
 /*
* INTERNAL_FUNCTION
*     bfd_cache_lookup
- *
* DESCRIPTION
*     Checks to see if the required BFD is the same as the last one
*     looked up. If so then it can use the iostream in the BFD with
- *     impunity, since it can't have changed since the last lookup,
- *     otherwise it has to perform the complicated lookup function 
- *
* .#define bfd_cache_lookup(x) \
* .    ((x)==bfd_last_cache? \
* .      (FILE*)(bfd_last_cache->iostream): \
* .       bfd_cache_lookup_worker(x))
- *
- *
+  INTERNAL_FUNCTION
      bfd_cache_lookup
+  DESCRIPTION
      Check to see if the required BFD is the same as the last one
      looked up. If so, then it can use the stream in the BFD with
+       impunity, since it can't have changed since the last lookup;
+       otherwise, it has to perform the complicated lookup function.
+  .#define bfd_cache_lookup(x) \
+  .    ((x)==bfd_last_cache? \
+  .      (FILE*)(bfd_last_cache->iostream): \
+  .       bfd_cache_lookup_worker(x))
  */
 
-static void bfd_cache_delete();
-
-
 static void
 DEFUN_VOID(close_one)
 {
@@ -111,7 +111,7 @@ DEFUN_VOID(close_one)
     }
 
     kill->where = ftell((FILE *)(kill->iostream));
-    bfd_cache_delete(kill);
+    (void) bfd_cache_delete(kill);
 }
 
 /* Cuts the BFD abfd out of the chain in the cache */
@@ -124,15 +124,24 @@ DEFUN(snip,(abfd),
   if (cache_sentinel == abfd) cache_sentinel = (bfd *)NULL;
 }
 
-static void
+static boolean
 DEFUN(bfd_cache_delete,(abfd),
       bfd *abfd)
 {
-  fclose ((FILE *)(abfd->iostream));
+  boolean ret;
+
+  if (fclose ((FILE *)(abfd->iostream)) == 0)
+    ret = true;
+  else
+    {
+      ret = false;
+      bfd_error = system_call_error;
+    }
   snip (abfd);
   abfd->iostream = NULL;
   open_files--;
   bfd_last_cache = 0;
+  return ret;
 }
   
 static bfd *
@@ -155,22 +164,16 @@ DEFUN(insert,(x,y),
 }
 \f
 
-/*
-INTERNAL_FUNCTION
-       bfd_cache_init
-
-SYNOPSIS
-       void  bfd_cache_init (bfd *);
-
-DESCRIPTION
-       Initialize a BFD by putting it on the cache LRU.
-*/
+/* Initialize a BFD by putting it on the cache LRU.  */
 
 void
 DEFUN(bfd_cache_init,(abfd),
       bfd *abfd)
 {
+  if (open_files >= BFD_CACHE_MAX_OPEN)
+    close_one ();
   cache_sentinel = insert(abfd, cache_sentinel);
+  ++open_files;
 }
 
 
@@ -178,21 +181,29 @@ DEFUN(bfd_cache_init,(abfd),
 INTERNAL_FUNCTION
        bfd_cache_close
 
+SYNOPSIS
+       boolean bfd_cache_close (bfd *abfd);
+
 DESCRIPTION
-       Remove the BFD from the cache. If the attached file is open,
+       Remove the BFD @var{abfd} from the cache. If the attached file is open,
        then close it too.
 
-SYNOPSIS
-       void bfd_cache_close (bfd *);
+RETURNS
+       <<false>> is returned if closing the file fails, <<true>> is
+       returned if all is well.
 */
-void
+boolean
 DEFUN(bfd_cache_close,(abfd),
       bfd *abfd)
 {
   /* If this file is open then remove from the chain */
   if (abfd->iostream) 
     {
-      bfd_cache_delete(abfd);
+      return bfd_cache_delete(abfd);
+    }
+  else
+    {
+      return true;
     }
 }
 
@@ -200,15 +211,15 @@ DEFUN(bfd_cache_close,(abfd),
 INTERNAL_FUNCTION
        bfd_open_file
 
+SYNOPSIS
+       FILE* bfd_open_file(bfd *abfd);
+
 DESCRIPTION
-       Call the OS to open a file for this BFD.  Returns the FILE *
-       (possibly null) that results from this operation.  Sets up the
-       BFD so that future accesses know the file is open. If the FILE
-       * returned is null, then there is won't have been put in the
+       Call the OS to open a file for @var{abfd}.  Return the <<FILE *>>
+       (possibly NULL) that results from this operation.  Set up the
+       BFD so that future accesses know the file is open. If the <<FILE *>>
+       returned is NULL, then it won't have been put in the
        cache, so it won't have to be removed from it.
-
-SYNOPSIS
-       FILE* bfd_open_file(bfd *);
 */
 
 FILE *
@@ -242,7 +253,6 @@ DEFUN(bfd_open_file, (abfd),
   }
 
   if (abfd->iostream) {
-    open_files++;
     bfd_cache_init (abfd);
   }
 
@@ -253,16 +263,16 @@ DEFUN(bfd_open_file, (abfd),
 INTERNAL_FUNCTION
        bfd_cache_lookup_worker
 
+SYNOPSIS
+       FILE *bfd_cache_lookup_worker(bfd *abfd);
+
 DESCRIPTION
        Called when the macro <<bfd_cache_lookup>> fails to find a
-       quick answer. Finds a file descriptor for this BFD.  If
-       necessary, it open it. If there are already more than
-       BFD_CACHE_MAX_OPEN files open, it trys to close one first, to
+       quick answer.  Find a file descriptor for @var{abfd}.  If
+       necessary, it open it.  If there are already more than
+       <<BFD_CACHE_MAX_OPEN>> files open, it tries to close one first, to
        avoid running out of file descriptors.  
 
-SYNOPSIS
-       FILE *bfd_cache_lookup_worker(bfd *);
-
 */
 
 FILE *
This page took 0.026728 seconds and 4 git commands to generate.