]> git.wh0rd.org Git - fontconfig.git/commitdiff
Eliminate global cache. Eliminate multi-arch cache code.
authorKeith Packard <keithp@neko.keithp.com>
Mon, 28 Aug 2006 04:53:48 +0000 (21:53 -0700)
committerKeith Packard <keithp@neko.keithp.com>
Mon, 28 Aug 2006 04:53:48 +0000 (21:53 -0700)
With the removal of the in-directory cache files, and the addition of
per-user cache directories, there is no longer any reason to preserve the
giant global cache file. Eliminating of this unifies the cache structure
and simplifies the overall caching strategies greatly.

fc-cache/fc-cache.c
src/fccache.c
src/fccfg.c
src/fcdir.c
src/fcint.h

index 29974b2ce2b6161032642a7bd574ec13482e35fb..281f8359029bbbfc7810f055bba4b466b1f06e07 100644 (file)
@@ -227,8 +227,7 @@ scanDirs (FcStrList *list, FcConfig *config, char *program, FcBool force, FcBool
            ret++;
            continue;
        }
-       if (!force && FcDirCacheValid (dir, config) && 
-           FcDirCacheHasCurrentArch (dir, config))
+       if (!force && FcDirCacheValid (dir, config))
        {
            if (verbose)
                printf ("skipping, %d fonts, %d dirs\n",
index 020230e3e6530f287cde4c8bb9dd274de340ce6c..295ee7750abc21ac215a395b9839f33287dbee36 100644 (file)
@@ -25,6 +25,7 @@
 
 #include "fcint.h"
 #include "../fc-arch/fcarch.h"
+#include <stdio.h>
 #include <fcntl.h>
 #include <dirent.h>
 #include <string.h>
 #define O_BINARY 0
 #endif
 
-static int
+static FILE *
 FcDirCacheOpen (FcConfig *config, const FcChar8 *dir, FcChar8 **cache_path);
 
-static off_t
-FcCacheSkipToArch (int fd, const char * arch);
-
-static FcBool 
-FcCacheCopyOld (int fd, int fd_orig, off_t start);
-
 static void *
 FcDirCacheProduce (FcFontSet *set, FcCache * metadata);
 
-static FcBool
-FcDirCacheConsume (int fd, const char * dir, FcFontSet *set, FcConfig *config);
-
-static int
+static off_t
 FcCacheNextOffset(off_t w);
 
-static char *
-FcCacheMachineSignature (void);
-
 static FcBool
 FcCacheHaveBank (int bank);
 
@@ -86,47 +75,23 @@ static void MD5Transform(FcChar32 buf[4], FcChar32 in[16]);
 #define FC_DBG_CACHE_REF    1024
 
 static char *
-FcCacheReadString (int fd, char *dest, int len)
+FcCacheReadString (FILE *file, char *dest, int len)
 {
-    int    size;
-    int    slen;
+    int            c;
+    char    *d = dest;
 
     if (len == 0)
        return 0;
 
-    size = read (fd, dest, len-1);
-
-    if (size > 0)
-    {
-       dest[size] = '\0';
-       slen = strlen (dest);
-
-       lseek (fd, slen - size + 1, SEEK_CUR);
-       return slen < len ? dest : 0;
+    while ((c = getc (file)) != EOF && len > 0) {
+       *d++ = c;
+       if (c == '\0')
+           return dest;
+       len--;
     }
-
     return 0;
 }
 
-static void
-FcCacheSkipString (int fd)
-{
-    char buf[256];
-    int  size;
-    int  slen;
-
-    while ( (size = read (fd, buf, sizeof (buf)-1)) > 0) 
-    {
-        buf [size] = '\0';
-        slen = strlen (buf);
-        if (slen < size) 
-        {
-            lseek (fd, slen - size + 1, SEEK_CUR);
-            return;
-        }
-    }
-}
-
 static FcBool
 FcCacheWriteString (int fd, const char *chars)
 {
@@ -135,473 +100,11 @@ FcCacheWriteString (int fd, const char *chars)
     return FcTrue;
 }
 
-static void
-FcGlobalCacheDirDestroy (FcGlobalCacheDir *d)
-{
-    FcStrSetDestroy (d->subdirs);
-    FcMemFree (FC_MEM_STRING, strlen (d->name)+1);
-    free (d->name);
-    FcMemFree (FC_MEM_CACHE, sizeof (FcGlobalCacheDir));
-    free (d);
-}
-
-FcGlobalCache *
-FcGlobalCacheCreate (void)
-{
-    FcGlobalCache   *cache;
-
-    cache = malloc (sizeof (FcGlobalCache));
-    if (!cache)
-       return 0;
-    FcMemAlloc (FC_MEM_CACHE, sizeof (FcGlobalCache));
-    cache->dirs = 0;
-    cache->updated = FcFalse;
-    cache->fd = -1;
-    return cache;
-}
-
-void
-FcGlobalCacheDestroy (FcGlobalCache *cache)
-{
-    FcGlobalCacheDir   *d, *next;
-
-    for (d = cache->dirs; d; d = next)
-    {
-       next = d->next;
-       FcGlobalCacheDirDestroy (d);
-    }
-    FcMemFree (FC_MEM_CACHE, sizeof (FcGlobalCache));
-    if (cache->fd != -1)
-       close (cache->fd);
-    free (cache);
-}
-
-void
-FcGlobalCacheLoad (FcGlobalCache    *cache,
-                   FcStrSet        *staleDirs,
-                  const FcChar8    *cache_file,
-                  FcConfig         *config)
-{
-    char               name_buf[FC_MAX_FILE_LEN];
-    FcGlobalCacheDir   *d, *next;
-    FcFileTime         config_time = FcConfigModifiedTime (config);
-    char               * current_arch_machine_name;
-    char               candidate_arch_machine_name[MACHINE_SIGNATURE_SIZE + 9];
-    off_t              current_arch_start;
-
-    struct stat        cache_stat, dir_stat;
-    char               subdirName[FC_MAX_FILE_LEN + 1 + 12 + 1];
-
-    if (stat ((char *) cache_file, &cache_stat) < 0)
-        return;
-
-    cache->fd = open ((char *) cache_file, O_RDONLY | O_BINARY);
-    if (cache->fd == -1)
-       return;
-
-    cache->updated = FcFalse;
-
-    if (!FcCacheReadString (cache->fd, name_buf, sizeof (name_buf)))
-        goto bail_and_destroy;
-    if (strcmp (name_buf, FC_GLOBAL_MAGIC_COOKIE) != 0)
-        goto bail_and_destroy;
-
-    current_arch_machine_name = FcCacheMachineSignature ();
-    current_arch_start = FcCacheSkipToArch(cache->fd, 
-                                          current_arch_machine_name);
-    if (current_arch_start < 0)
-        goto bail1;
-
-    lseek (cache->fd, current_arch_start, SEEK_SET);
-    if (!FcCacheReadString (cache->fd, candidate_arch_machine_name, 
-                           sizeof (candidate_arch_machine_name)))
-       goto bail_and_destroy;
-    if (strlen(candidate_arch_machine_name) == 0)
-       goto bail_and_destroy;
-
-    while (1) 
-    {
-       off_t targ;
-
-       if (!FcCacheReadString (cache->fd, name_buf, sizeof (name_buf)) || 
-           !strlen(name_buf))
-           break;
-
-       /* Directory must be older than the global cache file; also
-          cache must be newer than the config file. */
-        if (stat ((char *) name_buf, &dir_stat) < 0 || 
-            dir_stat.st_mtime > cache_stat.st_mtime ||
-           (config_time.set && cache_stat.st_mtime < config_time.time))
-        {
-            FcCache md;
-           off_t off;
-
-           FcStrSetAdd (staleDirs, (FcChar8 *)name_buf);
-
-           /* skip subdirs */
-           while (FcCacheReadString (cache->fd, subdirName, 
-                                     sizeof (subdirName)) &&
-                  strlen (subdirName))
-               ;
-
-           if (read (cache->fd, &md, sizeof (FcCache)) != sizeof(FcCache)) 
-           {
-               perror ("read metadata");
-               goto bail1;
-           }
-           off = FcCacheNextOffset (lseek(cache->fd, 0, SEEK_CUR)) + md.count;
-           if (lseek (cache->fd, off, SEEK_SET) != off) 
-           {
-               perror ("lseek");
-               goto bail1;
-           }
-           continue;
-        }
-
-       d = malloc (sizeof (FcGlobalCacheDir));
-       if (!d)
-           goto bail1;
-
-       d->next = cache->dirs;
-       cache->dirs = d;
-
-       d->name = (char *)FcStrCopy ((FcChar8 *)name_buf);
-       d->ent = 0;
-       d->state = FcGCDirFileRead;
-
-       d->subdirs = FcStrSetCreate();
-       do
-       {
-           if (!FcCacheReadString (cache->fd, subdirName, 
-                                   sizeof (subdirName)) ||
-               !strlen (subdirName))
-               break;
-           FcStrSetAdd (d->subdirs, (FcChar8 *)subdirName);
-       } while (1);
-
-       d->offset = lseek (cache->fd, 0, SEEK_CUR);
-       if (read (cache->fd, &d->metadata, sizeof (FcCache)) != sizeof (FcCache))
-           goto bail1;
-       targ = FcCacheNextOffset (lseek(cache->fd, 0, SEEK_CUR)) + d->metadata.count;
-       if (lseek (cache->fd, targ, SEEK_SET) != targ)
-           goto bail1;
-    }
-    return;
-
- bail1:
-    for (d = cache->dirs; d; d = next)
-    {
-       next = d->next;
-       free (d);
-    }
-    cache->dirs = 0;
-
-    close (cache->fd);
-    cache->fd = -1;
-    return;
-
- bail_and_destroy:
-    close (cache->fd);
-    cache->fd = -1;
-
-    if (stat ((char *) cache_file, &cache_stat) == 0)
-        unlink ((char *)cache_file);
-
-    return;
-
-}
-
-FcBool
-FcGlobalCacheReadDir (FcFontSet *set, FcStrSet *dirs, FcGlobalCache * cache, const char *dir, FcConfig *config)
-{
-    FcGlobalCacheDir   *d;
-    int                        i;
-
-    if (cache->fd == -1)
-       return FcFalse;
-
-    for (d = cache->dirs; d; d = d->next)
-    {
-       if (strcmp (d->name, dir) == 0)
-       {
-           if (d->state == FcGCDirDisabled)
-               return FcFalse;
-
-           if (d->state == FcGCDirFileRead) 
-           {
-               lseek (cache->fd, d->offset, SEEK_SET);
-               if (!FcDirCacheConsume (cache->fd, d->name, set, config))
-                   return FcFalse;
-
-               for (i = 0; i < d->subdirs->num; i++)
-                   FcStrSetAdd (dirs, (FcChar8 *)d->subdirs->strs[i]);
-
-               d->state = FcGCDirConsumed;
-           }
-           return FcTrue;
-       }
-    }
-
-    return FcFalse;
-}
-
-static FcGlobalCacheDir *
-FcGlobalCacheDirFind (FcGlobalCache *cache, const char *name)
-{
-    FcGlobalCacheDir * d;
-
-    if (!cache || !name)
-       return NULL;
-
-    for (d = cache->dirs; d; d = d->next)
-       if (strcmp((const char *)d->name, (const char *)name) == 0)
-           return d;
-
-    return NULL;
- }
-
-FcBool
-FcGlobalCacheUpdate (FcGlobalCache  *cache,
-                    FcStrSet       *dirs,
-                    const char     *name,
-                    FcFontSet      *set,
-                    FcConfig       *config)
-{
-    FcGlobalCacheDir    *d;
-    int                        i;
-
-    d = FcGlobalCacheDirFind (cache, name);
-
-    if (!d)
-    {
-       d = malloc (sizeof (FcGlobalCacheDir));
-       if (!d)
-           return FcFalse;
-       d->next = cache->dirs;
-       cache->dirs = d;
-    } else {
-       /* free old resources */
-       FcStrFree ((FcChar8 *)d->name);
-       free (d->ent);
-       FcStrSetDestroy (d->subdirs);
-    }
-
-    cache->updated = FcTrue;
-
-    d->name = (char *)FcStrCopy ((FcChar8 *)name);
-    d->ent = FcDirCacheProduce (set, &d->metadata);
-    d->offset = 0;
-    d->subdirs = FcStrSetCreate();
-    d->state = FcGCDirUpdated;
-    for (i = 0; i < dirs->num; i++)
-       FcStrSetAdd (d->subdirs, dirs->strs[i]);
-    return FcTrue;
-}
-
-FcBool
-FcGlobalCacheSave (FcGlobalCache    *cache,
-                  const FcChar8    *cache_file,
-                  FcConfig         *config)
-{
-    int                        fd, fd_orig;
-    FcGlobalCacheDir   *dir;
-    FcAtomic           *atomic;
-    off_t              current_arch_start = 0, truncate_to;
-    char               * current_arch_machine_name, * header;
-
-    if (!cache->updated)
-       return FcTrue;
-    
-#if defined (HAVE_GETUID) && defined (HAVE_GETEUID)
-    /* Set-UID programs can't safely update the cache */
-    if (getuid () != geteuid ())
-       return FcFalse;
-#endif
-    
-    atomic = FcAtomicCreate (cache_file);
-    if (!atomic)
-       return FcFalse;
-
-    if (!FcAtomicLock (atomic))
-       goto bail1;
-    fd = open ((char *) FcAtomicNewFile(atomic), O_RDWR | O_CREAT | O_BINARY, 
-              S_IRUSR | S_IWUSR);
-    if (fd == -1)
-       goto bail2;
-    FcCacheWriteString (fd, FC_GLOBAL_MAGIC_COOKIE);
-
-    fd_orig = open ((char *) FcAtomicOrigFile(atomic), O_RDONLY | O_BINARY);
-
-    current_arch_machine_name = FcCacheMachineSignature ();
-    if (fd_orig == -1)
-        current_arch_start = 0;
-    else
-        current_arch_start = FcCacheSkipToArch (fd_orig, 
-                                                current_arch_machine_name);
-
-    if (current_arch_start < 0)
-    {
-       off_t i = lseek(fd_orig, 0, SEEK_END);
-       if (i < strlen (FC_GLOBAL_MAGIC_COOKIE)+1)
-           i = strlen (FC_GLOBAL_MAGIC_COOKIE)+1;
-       current_arch_start = FcCacheNextOffset (i);
-    }
-
-    if (!FcCacheCopyOld(fd, fd_orig, current_arch_start))
-       goto bail3;
-
-    current_arch_start = lseek(fd, 0, SEEK_CUR);
-#if defined (HAVE_FTRUNCATE)
-    if (ftruncate (fd, current_arch_start) == -1)
-       goto bail3;
-#else
-#if defined (HAVE_CHSIZE)
-    if (chsize (fd, current_arch_start) == -1)
-       goto bail3;
-#else
-    goto bail3;
-#endif
-#endif
-
-    header = malloc (10 + strlen (current_arch_machine_name));
-    if (!header)
-       goto bail3;
-
-    truncate_to = current_arch_start + strlen(current_arch_machine_name) + 11;
-    for (dir = cache->dirs; dir; dir = dir->next)
-    {
-       int i;
-
-       if (dir->state == FcGCDirDisabled)
-           continue;
-       truncate_to += strlen(dir->name) + 1;
-       truncate_to += sizeof (FcCache);
-       truncate_to = FcCacheNextOffset (truncate_to);
-       truncate_to += dir->metadata.count;
-
-       for (i = 0; i < dir->subdirs->size; i++)
-           truncate_to += strlen((char *)dir->subdirs->strs[i]) + 1;
-       truncate_to ++;
-    }
-    truncate_to -= current_arch_start;
-
-    sprintf (header, "%8x ", (int)truncate_to);
-    strcat (header, current_arch_machine_name);
-    if (!FcCacheWriteString (fd, header))
-       goto bail4;
-
-    free (header);
-
-    for (dir = cache->dirs; dir; dir = dir->next)
-    {
-       int i;
-       const char * d;
-       off_t off;
-
-       if (!dir->name || dir->state == FcGCDirDisabled)
-           continue;
-       d = dir->name;
-       if (!d) 
-           continue;
-           
-       if (dir->metadata.count && !dir->ent) 
-       {
-           if (dir->state == FcGCDirUpdated || fd_orig < 0) 
-           {
-               fprintf(stderr, "Invalid metadata entry for %s, skipping...\n", d);
-               continue;
-           }
-           /* copy the old content */
-           dir->ent = malloc (dir->metadata.count);
-           if (!dir->ent) 
-           {
-               perror("malloc error");
-               continue;
-           }
-           off = FcCacheNextOffset (dir->offset + sizeof(FcCache));
-           if (lseek (fd_orig, off, SEEK_SET) != off) 
-           {
-               perror("lseek");
-               free(dir->ent);
-               continue;
-           }
-           if (read (fd_orig, dir->ent, dir->metadata.count)
-               != dir->metadata.count) 
-           {
-               perror("read");
-               free(dir->ent);
-               continue;
-           }
-       }
-       
-       FcCacheWriteString (fd, d);
-
-       for (i = 0; i < dir->subdirs->size; i++)
-           FcCacheWriteString (fd, (char *)dir->subdirs->strs[i]);
-       FcCacheWriteString (fd, "");
-       
-       if (write (fd, &dir->metadata, sizeof(FcCache)) != sizeof(FcCache))
-       {
-           perror ("write metadata");
-           free (dir->ent);
-           continue;
-       }
-       off = FcCacheNextOffset (lseek(fd, 0, SEEK_CUR));
-       if (lseek (fd, off, SEEK_SET) != off)
-       {
-           perror ("lseek");
-           free (dir->ent);
-           continue;
-       }
-       if (dir->metadata.count)
-       {
-           if (write (fd, dir->ent, dir->metadata.count) != dir->metadata.count)
-           {
-               perror ("write dirent");
-               free (dir->ent);
-               continue;
-           }
-       }
-       free (dir->ent);
-    }
-    FcCacheWriteString (fd, "");
-
-    if (close (fd) == -1)
-       goto bail25;
-
-    close (fd_orig);
-    fd_orig = -1;
-    
-    if (!FcAtomicReplaceOrig (atomic))
-       goto bail25;
-    
-    FcAtomicUnlock (atomic);
-    FcAtomicDestroy (atomic);
-
-    cache->updated = FcFalse;
-    return FcTrue;
-
- bail4:
-    free (header);
- bail3:
-    if (fd_orig != -1)
-        close (fd_orig);
-
-    close (fd);
- bail25:
-    FcAtomicDeleteNew (atomic);
- bail2:
-    FcAtomicUnlock (atomic);
- bail1:
-    FcAtomicDestroy (atomic);
-    return FcFalse;
-}
-
 /* 
  * Find the next presumably-mmapable offset after the supplied file
  * position.
  */
-static int
+static off_t
 FcCacheNextOffset(off_t w)
 {
     static long pagesize = -1;
@@ -621,177 +124,21 @@ FcCacheNextOffset(off_t w)
        return ((w / pagesize)+1)*pagesize;
 }
 
-/* return the address of the segment for the provided arch,
- * or -1 if arch not found */
-static off_t
-FcCacheSkipToArch (int fd, const char * arch)
-{
-    char candidate_arch_machine_name_count[MACHINE_SIGNATURE_SIZE + 9];
-    char * candidate_arch;
-    off_t current_arch_start = 0;
-
-    lseek (fd, 0, SEEK_SET);
-    FcCacheSkipString (fd);
-    current_arch_start = lseek (fd, 0, SEEK_CUR);
-
-    /* skip arches that are not the current arch */
-    while (1)
-    {
-       long bs;
-
-       if (lseek (fd, current_arch_start, SEEK_SET) != current_arch_start)
-            return -1;
-
-       if (FcCacheReadString (fd, candidate_arch_machine_name_count, 
-                               sizeof (candidate_arch_machine_name_count)) == 0)
-            return -1;
-       if (!strlen(candidate_arch_machine_name_count))
-           return -1;
-       bs = strtol(candidate_arch_machine_name_count, &candidate_arch, 16);
-
-       /* count = 0 should probably be distinguished from the !bs condition */
-       if (!bs || bs < strlen (candidate_arch_machine_name_count))
-           return -1;
-
-       candidate_arch++; /* skip leading space */
-
-       if (strcmp (candidate_arch, arch)==0)
-           return current_arch_start;
-       current_arch_start += bs;
-       current_arch_start = FcCacheNextOffset (current_arch_start);
-    }
-}
-
-/* Cuts out the segment at the file pointer (moves everything else
- * down to cover it), and leaves the file pointer at the end of the
- * file. */
-static FcBool 
-FcCacheCopyOld (int fd, int fd_orig, off_t start)
-{
-    char * buf = malloc (8192);
-    char candidate_arch_machine_name[MACHINE_SIGNATURE_SIZE + 9];
-    long bs;
-    int c, bytes_skipped;
-    off_t loc;
-
-    if (!buf)
-       return FcFalse;
-
-    loc = 0;
-    lseek (fd, 0, SEEK_SET); lseek (fd_orig, 0, SEEK_SET);
-    FcCacheSkipString (fd); FcCacheSkipString (fd_orig);
-    do
-    {
-        int b = 8192;
-        if (loc + b > start)
-            b = start - loc;
-
-       if ((c = read (fd_orig, buf, b)) <= 0)
-           break;
-       if (write (fd, buf, c) < 0)
-           goto bail;
-
-        loc += c;
-    }
-    while (c > 0);
-
-    lseek (fd, start, SEEK_SET);
-    if (FcCacheReadString (fd, candidate_arch_machine_name, 
-                          sizeof (candidate_arch_machine_name)) == 0)
-       goto done;
-    if (!strlen(candidate_arch_machine_name))
-       goto done;
-
-    bs = strtol(candidate_arch_machine_name, 0, 16);
-    if (bs == 0)
-       goto done;
-
-    bytes_skipped = 0;
-    do
-    {
-       lseek (fd, start+bs+bytes_skipped, SEEK_SET);
-       if ((c = read (fd, buf, 8192)) <= 0)
-           break;
-       lseek (fd, start+bytes_skipped, SEEK_SET);
-       if (write (fd, buf, c) < 0)
-           goto bail;
-       bytes_skipped += c;
-    }
-    while (c > 0);
-    lseek (fd, start+bytes_skipped, SEEK_SET);
-
- done:
-    free (buf);
-    return FcTrue;
-
- bail:
-    free (buf);
-    return FcFalse;
-}
-
 /* Does not check that the cache has the appropriate arch section. */
 FcBool
 FcDirCacheValid (const FcChar8 *dir, FcConfig *config)
 {
-    int        fd;
+    FILE *file;
 
-    fd = FcDirCacheOpen (config, dir, NULL);
+    file = FcDirCacheOpen (config, dir, NULL);
 
-    if (fd < 0)
+    if (file != NULL)
        return FcFalse;
-    close (fd);
+    fclose (file);
 
     return FcTrue;
 }
 
-/* Assumes that the cache file in 'dir' exists.
- * Checks that the cache has the appropriate arch section. */
-FcBool
-FcDirCacheHasCurrentArch (const FcChar8 *dir, FcConfig *config)
-{
-    int        fd;
-    off_t      current_arch_start;
-    char       *current_arch_machine_name;
-    FcCache    metadata;
-    char       subdirName[FC_MAX_FILE_LEN + 1 + 12 + 1];
-
-    fd = FcDirCacheOpen (config, dir, NULL);
-    if (fd < 0)
-       goto bail;
-
-    current_arch_machine_name = FcCacheMachineSignature();
-    current_arch_start = FcCacheSkipToArch(fd, current_arch_machine_name);
-
-    if (current_arch_start >= 0)
-    {
-       if (lseek (fd, current_arch_start, SEEK_SET) != current_arch_start)
-           goto bail1;
-
-       FcCacheSkipString (fd);
-
-       while (FcCacheReadString (fd, subdirName, sizeof (subdirName)) && strlen (subdirName) > 0)
-           ;
-
-        if (read(fd, &metadata, sizeof(FcCache)) != sizeof(FcCache))
-           goto bail1;
-
-        if (metadata.magic != FC_CACHE_MAGIC)
-           goto bail1;
-    }
-
-    close (fd);
-
-    if (current_arch_start < 0)
-        return FcFalse;
-    
-    return FcTrue;
-
- bail1:
-    close (fd);
- bail:
-    return FcFalse;
-}
-
 #define CACHEBASE_LEN (1 + 32 + 1 + sizeof (FC_ARCHITECTURE) + sizeof (FC_CACHE_SUFFIX))
 
 static const char bin2hex[] = { '0', '1', '2', '3',
@@ -828,12 +175,10 @@ FcDirCacheBasename (const FcChar8 * dir, FcChar8 cache_base[CACHEBASE_LEN])
 FcBool
 FcDirCacheUnlink (const FcChar8 *dir, FcConfig *config)
 {
-    int                fd = -1;
     FcChar8    *cache_hashed = NULL;
     FcChar8    cache_base[CACHEBASE_LEN];
     FcStrList  *list;
     FcChar8    *cache_dir;
-    char       dir_buf[FC_MAX_FILE_LEN];
 
     FcDirCacheBasename (dir, cache_base);
 
@@ -846,19 +191,7 @@ FcDirCacheUnlink (const FcChar8 *dir, FcConfig *config)
         cache_hashed = FcStrPlus (cache_dir, cache_base);
         if (!cache_hashed)
            break;
-        fd = open((char *) cache_hashed, O_RDONLY | O_BINARY);
-       FcStrFree (cache_hashed);
-        if (fd >= 0) 
-       {
-           if (FcCacheReadString (fd, dir_buf, sizeof (dir_buf)) &&
-               strcmp (dir_buf, (char *) dir) == 0)
-           {
-               close (fd);
-               if (unlink ((char *) cache_hashed) < 0)
-                   break;
-           } else
-               close (fd);
-       }
+       (void) unlink ((char *) cache_hashed);
     }
     FcStrListDone (list);
     /* return FcFalse if something went wrong */
@@ -868,14 +201,13 @@ FcDirCacheUnlink (const FcChar8 *dir, FcConfig *config)
 }
 
 static int
-FcCacheReadDirs (FcConfig * config, FcGlobalCache * cache, 
+FcCacheReadDirs (FcConfig * config,
                 FcStrList *list, FcFontSet * set, FcStrSet *processed_dirs)
 {
     int                        ret = 0;
     FcChar8            *dir;
     FcStrSet           *subdirs;
     FcStrList          *sublist;
-    FcGlobalCacheDir   *d;
 
     /*
      * Read in the results from 'list'.
@@ -903,24 +235,9 @@ FcCacheReadDirs (FcConfig * config, FcGlobalCache * cache,
            continue;
        }
        
-       if (FcDirCacheRead (set, subdirs, dir, config))
-       {
-           /* if an old entry is found in the global cache, disable it */
-           if ((d = FcGlobalCacheDirFind (cache, (const char *)dir)) != NULL)
-           {
-               d->state = FcGCDirDisabled;
-               /* save the updated config later without this entry */
-               cache->updated = FcTrue;
-           }
-       }
-       else
-       {
-           if (FcDebug () & FC_DBG_FONTSET)
-               printf ("cache scan dir %s\n", dir);
-
-           FcDirScanConfig (set, subdirs, cache, 
-                            config->blanks, dir, FcFalse, config);
-       }
+       FcDirScanConfig (set, subdirs,
+                        config->blanks, dir, FcFalse, config);
+       
        sublist = FcStrListCreate (subdirs);
        FcStrSetDestroy (subdirs);
        if (!sublist)
@@ -929,14 +246,14 @@ FcCacheReadDirs (FcConfig * config, FcGlobalCache * cache,
            ret++;
            continue;
        }
-       ret += FcCacheReadDirs (config, cache, sublist, set, processed_dirs);
+       ret += FcCacheReadDirs (config, sublist, set, processed_dirs);
     }
     FcStrListDone (list);
     return ret;
 }
 
 FcFontSet *
-FcCacheRead (FcConfig *config, FcGlobalCache * cache)
+FcCacheRead (FcConfig *config)
 {
     FcFontSet  *s = FcFontSetCreate();
     FcStrSet   *processed_dirs;
@@ -948,7 +265,7 @@ FcCacheRead (FcConfig *config, FcGlobalCache * cache)
     if (!processed_dirs)
        goto bail;
 
-    if (FcCacheReadDirs (config, cache, FcConfigGetConfigDirs (config), s, processed_dirs))
+    if (FcCacheReadDirs (config, FcConfigGetConfigDirs (config), s, processed_dirs))
        goto bail1;
 
     FcStrSetDestroy (processed_dirs);
@@ -965,185 +282,130 @@ FcCacheRead (FcConfig *config, FcGlobalCache * cache)
  * This searches the list of cache dirs for the relevant cache file,
  * returning the first one found.
  */
-static int
+static FILE *
 FcDirCacheOpen (FcConfig *config, const FcChar8 *dir, FcChar8 **cache_path)
 {
-    int                fd = -1;
+    FILE       *file = NULL;
     FcChar8    *cache_hashed = NULL;
     FcChar8    cache_base[CACHEBASE_LEN];
     FcStrList  *list;
     FcChar8    *cache_dir;
-    char       dir_buf[FC_MAX_FILE_LEN];
     struct stat file_stat, dir_stat;
 
     if (stat ((char *) dir, &dir_stat) < 0)
-        return -1;
+        return NULL;
 
     FcDirCacheBasename (dir, cache_base);
 
     list = FcStrListCreate (config->cacheDirs);
     if (!list)
-        return -1;
+        return NULL;
        
     while ((cache_dir = FcStrListNext (list)))
     {
         cache_hashed = FcStrPlus (cache_dir, cache_base);
         if (!cache_hashed)
            break;
-        fd = open((char *) cache_hashed, O_RDONLY | O_BINARY);
-        if (fd >= 0) {
-           if (fstat (fd, &file_stat) >= 0 &&
+        file = fopen((char *) cache_hashed, "rb");
+        if (file != NULL) {
+           if (fstat (fileno (file), &file_stat) >= 0 &&
                dir_stat.st_mtime <= file_stat.st_mtime)
            {
                break;
            }
-           close (fd);
-           fd = -1;
+           fclose (file);
+           file = NULL;
        }
        FcStrFree (cache_hashed);
        cache_hashed = NULL;
     }
     FcStrListDone (list);
 
-    if (fd < 0)
-       return -1;
-    
-    if (!FcCacheReadString (fd, dir_buf, sizeof (dir_buf)) ||
-       strcmp (dir_buf, (char *) dir) != 0)
-    {
-       close (fd);
-        FcStrFree (cache_hashed);
-        return -1;
-    }
+    if (file == NULL)
+       return NULL;
     
     if (cache_path)
        *cache_path = cache_hashed;
     else
        FcStrFree (cache_hashed);
 
-    return fd;
+    return file;
 }
 
 /* read serialized state from the cache file */
 FcBool
-FcDirCacheRead (FcFontSet * set, FcStrSet * dirs, const FcChar8 *dir, FcConfig *config)
+FcDirCacheRead (FcFontSet * set, FcStrSet * dirs, 
+               const FcChar8 *dir, FcConfig *config)
 {
-    int        fd;
-    char       *current_arch_machine_name;
-    char       candidate_arch_machine_name[9+MACHINE_SIGNATURE_SIZE];
-    off_t      current_arch_start = 0;
-    char       subdirName[FC_MAX_FILE_LEN + 1 + 12 + 1];
-
-    fd = FcDirCacheOpen (config, dir, NULL);
-    if (fd < 0)
-       goto bail;
-
-    current_arch_machine_name = FcCacheMachineSignature();
-    current_arch_start = FcCacheSkipToArch(fd, 
-                                          current_arch_machine_name);
-    if (current_arch_start < 0)
-        goto bail1;
-
-    lseek (fd, current_arch_start, SEEK_SET);
-    if (FcCacheReadString (fd, candidate_arch_machine_name, 
-                          sizeof (candidate_arch_machine_name)) == 0)
-       goto bail1;
+    FILE       *file;
+    FcCache    metadata;
+    void       *current_dir_block;
+    char       subdir_name[FC_MAX_FILE_LEN + 1 + 12 + 1];
 
-    while (FcCacheReadString (fd, subdirName, sizeof (subdirName)) && strlen (subdirName) > 0)
-        FcStrSetAdd (dirs, (FcChar8 *)subdirName);
+    file = FcDirCacheOpen (config, dir, NULL);
+    if (file == NULL)
+       goto bail;
 
-    if (!FcDirCacheConsume (fd, (const char *)dir, set, config))
+    if (fread(&metadata, sizeof(FcCache), 1, file) != 1)
        goto bail1;
-       
-    close(fd);
-    return FcTrue;
-
- bail1:
-    close (fd);
- bail:
-    return FcFalse;
-}
-
-static FcBool
-FcDirCacheConsume (int fd, const char * dir, FcFontSet *set, FcConfig *config)
-{
-    FcCache metadata;
-    void * current_dir_block;
-    off_t pos, endpos;
-
-    if (read(fd, &metadata, sizeof(FcCache)) != sizeof(FcCache))
-       return FcFalse;
     if (metadata.magic != FC_CACHE_MAGIC)
-        return FcFalse;
-
-    if (!metadata.count)
-    {
-       pos = FcCacheNextOffset (lseek(fd, 0, SEEK_CUR));
-       lseek (fd, pos, SEEK_SET);
-       if (config)
-           FcConfigAddFontDir (config, (FcChar8 *)dir);
-       return FcTrue;
-    }
-
-    pos = FcCacheNextOffset (lseek(fd, 0, SEEK_CUR));
-
-    /* This is not failsafe (multi-arches can break it),
-     * but fd has got to have at least as many bytes as
-     * metadata.count, or something's going to go horribly wrong. */
-    if (pos == (off_t)-1)
-        return FcFalse;
+        goto bail1;
 
-    endpos = lseek (fd, 0, SEEK_END);
-    if (endpos == (off_t)-1 || endpos - pos < metadata.count)
-        return FcFalse;
-    if (lseek (fd, pos, SEEK_SET) == -1)
-        return FcFalse;
+    while (FcCacheReadString (file, subdir_name, sizeof (subdir_name)) && 
+          strlen (subdir_name) > 0)
+        FcStrSetAdd (dirs, (FcChar8 *)subdir_name);
 
+    if (metadata.count)
+    {
+       int     fd = fileno (file);
 #if defined(HAVE_MMAP) || defined(__CYGWIN__)
-    current_dir_block = mmap (0, metadata.count, 
-                             PROT_READ, MAP_SHARED, fd, pos);
-    if (current_dir_block == MAP_FAILED)
-       return FcFalse;
+       current_dir_block = mmap (0, metadata.count, 
+                                 PROT_READ, MAP_SHARED, fd, metadata.pos);
+       if (current_dir_block == MAP_FAILED)
+           goto bail1;
 #elif defined(_WIN32)
        {
-               HANDLE hFileMap;
+           HANDLE hFileMap;
 
-               hFileMap = CreateFileMapping((HANDLE) _get_osfhandle(fd), NULL, PAGE_READONLY, 0, 0, NULL);
-               if (hFileMap == NULL)
-                       return FcFalse;
-
-               current_dir_block = MapViewOfFile (hFileMap, FILE_MAP_READ, 0, 0, metadata.count + pos);
-               if (current_dir_block == NULL)
-               {
-                       CloseHandle (hFileMap);
-                       return FcFalse;
-               }
+           hFileMap = CreateFileMapping((HANDLE) _get_osfhandle(fd), NULL, PAGE_READONLY, 0, 0, NULL);
+           if (hFileMap == NULL)
+               goto bail1;
 
-               current_dir_block = (void *)((char *)current_dir_block + pos);
+           current_dir_block = MapViewOfFile (hFileMap, FILE_MAP_READ, 0, metadata.pos, metadata.count);
+           if (current_dir_block == NULL)
+           {
+               CloseHandle (hFileMap);
+               goto bail1;
+           }
        }
 #else
-    current_dir_block = malloc (metadata.count);
-    if (!current_dir_block)
-       return FcFalse;
+       if (lseek (fd, metatdata.pos, SEEK_SET) == -1)
+           goto bail1;
 
-    /* could also use CreateMappedViewOfFile under MinGW... */
-    if (read (fd, current_dir_block, metadata.count) != metadata.count)
-       goto bail;
-#endif
-    lseek (fd, pos+metadata.count, SEEK_SET);
+       current_dir_block = malloc (metadata.count);
+       if (!current_dir_block)
+           goto bail1;
 
-    FcCacheAddBankDir (metadata.bank, dir);
+       /* could also use CreateMappedViewOfFile under MinGW... */
+       if (read (fd, current_dir_block, metadata.count) != metadata.count)
+       {
+           free (current_dir_block);
+           goto bail1;
+       }
+#endif
+       FcCacheAddBankDir (metadata.bank, (char *) dir);
+       if (!FcFontSetUnserialize (&metadata, set, current_dir_block))
+           goto bail1;
+    }
     if (config)
        FcConfigAddFontDir (config, (FcChar8 *)dir);
 
-    if (!FcFontSetUnserialize (&metadata, set, current_dir_block))
-       goto bail;
-
+    fclose(file);
     return FcTrue;
+
+ bail1:
+    fclose (file);
  bail:
-#if !(defined(HAVE_MMAP) || defined(__CYGWIN__))
-    free (current_dir_block);
-#endif
     return FcFalse;
 }
 
@@ -1236,15 +498,14 @@ FcDirCacheWrite (FcFontSet *set, FcStrSet *dirs, const FcChar8 *dir, FcConfig *c
 {
     FcChar8        cache_base[CACHEBASE_LEN];
     FcChar8        *cache_hashed;
-    int            fd, fd_orig, i, dirs_count;
+    int            fd, i;
     FcAtomic       *atomic;
     FcCache        metadata;
-    off_t          current_arch_start = 0, truncate_to;
-    FcStrList      *list;
-    char            *current_arch_machine_name, * header;
     void           *current_dir_block = 0;
+    FcStrList      *list;
     FcChar8        *cache_dir = NULL;
     FcChar8        *test_dir;
+    off_t          header_len;
 
     /*
      * Write it to the first directory in the list which is writable
@@ -1297,76 +558,35 @@ FcDirCacheWrite (FcFontSet *set, FcStrSet *dirs, const FcChar8 *dir, FcConfig *c
     if (!FcAtomicLock (atomic))
        goto bail2;
 
-    /* open the original file to save relevant portions */
-    fd_orig = open((char *)FcAtomicOrigFile (atomic), O_RDONLY | O_BINARY);
-
     fd = open((char *)FcAtomicNewFile (atomic), O_RDWR | O_CREAT | O_BINARY, 0666);
     if (fd == -1)
        goto bail3;
-
-    FcCacheWriteString (fd, (char *) dir);
-
-    current_arch_machine_name = FcCacheMachineSignature ();
-    current_arch_start = 0;
-
-    if (fd_orig != -1)
-        current_arch_start = 
-            FcCacheSkipToArch(fd_orig, current_arch_machine_name);
-
-    if (current_arch_start < 0)
-    {
-       off_t offset = lseek(fd_orig, 0, SEEK_END);
-       current_arch_start = FcCacheNextOffset (offset);
-    }
-
-    if (fd_orig != -1 && !FcCacheCopyOld(fd, fd_orig, current_arch_start))
-       goto bail4;
-
-    if (fd_orig != -1)
-        close (fd_orig);
-
-    current_arch_start = lseek(fd, 0, SEEK_CUR);
-#if defined (HAVE_FTRUNCATE)
-    if (ftruncate (fd, current_arch_start) == -1)
-       goto bail4;
-#else
-#if defined (HAVE_CHSIZE)
-    if (chsize (fd, current_arch_start) == -1)
-       goto bail4;
-#else
-    goto bail4;
-#endif
-#endif
-
-    /* allocate space for subdir names in this block */
-    dirs_count = 0;
-    for (i = 0; i < dirs->size; i++)
-        dirs_count += strlen((char *)dirs->strs[i]) + 1;
-    dirs_count ++;
-
-    /* now write the address of the next offset */
-    truncate_to = FcCacheNextOffset (FcCacheNextOffset (current_arch_start + sizeof (FcCache) + dirs_count) + metadata.count) - current_arch_start;
-    header = malloc (10 + strlen (current_arch_machine_name));
-    if (!header)
-       goto bail4;
-    sprintf (header, "%8x ", (int)truncate_to);
-    strcat (header, current_arch_machine_name);
-    if (!FcCacheWriteString (fd, header))
-       goto bail5;
-
+    
+    /*
+     * Compute file header length -- the FcCache followed by the subdir names
+     */
+    header_len = sizeof (FcCache);
     for (i = 0; i < dirs->size; i++)
-        FcCacheWriteString (fd, (char *)dirs->strs[i]);
-    FcCacheWriteString (fd, "");
-
+       header_len += strlen ((char *)dirs->strs[i]) + 1;
+    
+    metadata.pos = FcCacheNextOffset (lseek (fd, 0, SEEK_CUR) + header_len);
+    metadata.subdirs = dirs->size;
+    
+    /*
+     * Write out the header
+     */
     if (write (fd, &metadata, sizeof(FcCache)) != sizeof(FcCache)) 
     {
        perror("write metadata");
        goto bail5;
     }
+    
+    for (i = 0; i < dirs->size; i++)
+        FcCacheWriteString (fd, (char *)dirs->strs[i]);
+
     if (metadata.count)
     {
-       off_t off = FcCacheNextOffset (lseek(fd, 0, SEEK_END));
-       if (lseek (fd, off, SEEK_SET) != off)
+       if (lseek (fd, metadata.pos, SEEK_SET) != metadata.pos)
            perror("lseek");
        else if (write (fd, current_dir_block, metadata.count) !=
                 metadata.count)
@@ -1375,20 +595,6 @@ FcDirCacheWrite (FcFontSet *set, FcStrSet *dirs, const FcChar8 *dir, FcConfig *c
         current_dir_block = 0;
     }
 
-    /* this actually serves to pad out the cache file, if needed */
-#if defined (HAVE_FTRUNCATE)
-    if (ftruncate (fd, current_arch_start + truncate_to) == -1)
-       goto bail5;
-#else
-#if defined (HAVE_CHSIZE)
-    if (chsize (fd, current_arch_start + truncate_to) == -1)
-       goto bail5;
-#else
-    goto bail5;
-#endif
-#endif
-
-    free (header);
     close(fd);
     if (!FcAtomicReplaceOrig(atomic))
         goto bail3;
@@ -1398,8 +604,6 @@ FcDirCacheWrite (FcFontSet *set, FcStrSet *dirs, const FcChar8 *dir, FcConfig *c
     return FcTrue;
 
  bail5:
-    free (header);
- bail4:
     close (fd);
  bail3:
     FcAtomicUnlock (atomic);
@@ -1412,45 +616,6 @@ FcDirCacheWrite (FcFontSet *set, FcStrSet *dirs, const FcChar8 *dir, FcConfig *c
     return FcFalse;
 }
 
-static char *
-FcCacheMachineSignature ()
-{
-    static char buf[MACHINE_SIGNATURE_SIZE];
-    int32_t magic = ENDIAN_TEST;
-    char * m = (char *)&magic;
-
-    sprintf (buf, "%2x%2x%2x%2x "
-            "%4x %4x %4x %4x %4x %4x %4x %4x %4x %4x %4x %4x "
-            "%4x %4x %4x %4x %4x %4x %4x %4x\n", 
-            m[0], m[1], m[2], m[3],
-            (unsigned int)sizeof (char),
-            (unsigned int)sizeof (char *),
-            (unsigned int)sizeof (int),
-            (unsigned int)sizeof (FcPattern),
-            (unsigned int)sizeof (FcPatternEltPtr),
-            (unsigned int)sizeof (struct _FcPatternElt *),
-            (unsigned int)sizeof (FcPatternElt),
-            (unsigned int)sizeof (FcObjectPtr),
-            (unsigned int)sizeof (FcValueListPtr),
-            (unsigned int)sizeof (FcValue),
-            (unsigned int)sizeof (FcValueBinding),
-            (unsigned int)sizeof (struct _FcValueList *),
-            (unsigned int)sizeof (FcCharSet),
-            (unsigned int)sizeof (FcCharLeaf **),
-            (unsigned int)sizeof (FcChar16 *),
-            (unsigned int)sizeof (FcChar16),
-            (unsigned int)sizeof (FcCharLeaf),
-            (unsigned int)sizeof (FcChar32),
-            (unsigned int)sizeof (FcCache),
-#if defined (HAVE_SYSCONF)
-            (unsigned int)sysconf(_SC_PAGESIZE));
-#else
-            (unsigned int)FC_HARDCODED_PAGESIZE);
-#endif
-
-    return buf;
-}
-
 static int banks_ptr = 0, banks_alloc = 0;
 int * _fcBankId = 0, * _fcBankIdx = 0;
 static const char ** bankDirs = 0;
index 3c3681a100620e9c803e811aae0f55195121e340..586c6567e4fca29fd6a9950ac617ebd5d7aa6bfe 100644 (file)
@@ -265,7 +265,6 @@ FcBool
 FcConfigBuildFonts (FcConfig *config)
 {
     FcFontSet      *fonts, *cached_fonts;
-    FcGlobalCache   *cache;
     FcStrList      *list;
     FcStrSet       *oldDirs;
     FcChar8        *dir;
@@ -274,18 +273,11 @@ FcConfigBuildFonts (FcConfig *config)
     if (!fonts)
        goto bail0;
     
-    cache = FcGlobalCacheCreate ();
-    if (!cache)
-       goto bail1;
-
     oldDirs = FcStrSetCreate ();
     if (!oldDirs)
         goto bail2;
 
-    if (config->cache)
-       FcGlobalCacheLoad (cache, oldDirs, config->cache, config);
-
-    cached_fonts = FcCacheRead(config, cache);
+    cached_fonts = FcCacheRead(config);
     if (!cached_fonts)
     {
        list = FcConfigGetFontDirs (config);
@@ -296,7 +288,7 @@ FcConfigBuildFonts (FcConfig *config)
        {
            if (FcDebug () & FC_DBG_FONTSET)
                printf ("build scan dir %s\n", dir);
-           FcDirScanConfig (fonts, config->fontDirs, cache, 
+           FcDirScanConfig (fonts, config->fontDirs,
                             config->blanks, dir, FcFalse, config);
        }
        
@@ -310,7 +302,7 @@ FcConfigBuildFonts (FcConfig *config)
         {
            if (FcDebug () & FC_DBG_FONTSET)
                printf ("scan dir %s\n", oldDirs->strs[i]);
-           FcDirScanConfig (fonts, config->fontDirs, cache, 
+           FcDirScanConfig (fonts, config->fontDirs,
                             config->blanks, oldDirs->strs[i], 
                              FcFalse, config);
        }
@@ -333,9 +325,6 @@ FcConfigBuildFonts (FcConfig *config)
     if (FcDebug () & FC_DBG_FONTSET)
        FcFontSetPrint (fonts);
 
-    if (config->cache)
-       FcGlobalCacheSave (cache, config->cache, config);
-    FcGlobalCacheDestroy (cache);
     FcStrSetDestroy (oldDirs);
 
     FcConfigSetFonts (config, fonts, FcSetSystem);
@@ -344,8 +333,6 @@ FcConfigBuildFonts (FcConfig *config)
 bail3:
     FcStrSetDestroy (oldDirs);
 bail2:
-    FcGlobalCacheDestroy (cache);
-bail1:
     FcFontSetDestroy (fonts);
 bail0:
     return FcFalse;
@@ -1799,7 +1786,7 @@ FcConfigAppFontAddFile (FcConfig    *config,
        FcConfigSetFonts (config, set, FcSetApplication);
     }
        
-    if (!FcFileScanConfig (set, subdirs, 0, config->blanks, file, FcFalse, config))
+    if (!FcFileScanConfig (set, subdirs, config->blanks, file, FcFalse, config))
     {
        FcStrSetDestroy (subdirs);
        return FcFalse;
@@ -1847,7 +1834,7 @@ FcConfigAppFontAddDir (FcConfig       *config,
        FcConfigSetFonts (config, set, FcSetApplication);
     }
     
-    if (!FcDirScanConfig (set, subdirs, 0, config->blanks, dir, FcFalse, config))
+    if (!FcDirScanConfig (set, subdirs, config->blanks, dir, FcFalse, config))
     {
        FcStrSetDestroy (subdirs);
        return FcFalse;
index 145226072571abe2d4a6880ee1727359edb26501..62314a325984452c437d9987e6aad0b2d2c3a4ff 100644 (file)
@@ -35,26 +35,17 @@ FcFileIsDir (const FcChar8 *file)
     return S_ISDIR(statb.st_mode);
 }
 
-FcBool
-FcFileScanConfig (FcFontSet    *set,
-                 FcStrSet      *dirs,
-                 FcGlobalCache *cache,
-                 FcBlanks      *blanks,
-                 const FcChar8 *file,
-                 FcBool        force,
-                 FcConfig      *config)
+static FcBool
+FcFileScanFontConfig (FcFontSet                *set,
+                     FcBlanks          *blanks,
+                     const FcChar8     *file,
+                     FcConfig          *config)
 {
-    int                        id;
-    FcPattern          *font;
-    FcBool             ret = FcTrue;
-    int                        count = 0;
+    FcPattern  *font;
+    FcBool     ret = FcTrue;
+    int                id;
+    int                count = 0;
     
-    if (config && !FcConfigAcceptFilename (config, file))
-       return FcTrue;
-
-    if (FcFileIsDir (file))
-       return FcStrSetAdd (dirs, file);
-
     id = 0;
     do
     {
@@ -89,15 +80,32 @@ FcFileScanConfig (FcFontSet *set,
     return ret;
 }
 
+FcBool
+FcFileScanConfig (FcFontSet    *set,
+                 FcStrSet      *dirs,
+                 FcBlanks      *blanks,
+                 const FcChar8 *file,
+                 FcBool        force,
+                 FcConfig      *config)
+{
+    if (config && !FcConfigAcceptFilename (config, file))
+       return FcTrue;
+
+    if (FcFileIsDir (file))
+       return FcStrSetAdd (dirs, file);
+    else
+       return FcFileScanFontConfig (set, blanks, file, config);
+}
+
 FcBool
 FcFileScan (FcFontSet      *set,
            FcStrSet        *dirs,
-           FcGlobalCache   *cache,
+           FcFileCache     *cache, /* XXX unused */
            FcBlanks        *blanks,
            const FcChar8   *file,
            FcBool          force)
 {
-    return FcFileScanConfig (set, dirs, cache, blanks, file, force, 0);
+    return FcFileScanConfig (set, dirs, blanks, file, force, 0);
 }
 
 /*
@@ -118,7 +126,6 @@ cmpstringp(const void *p1, const void *p2)
 FcBool
 FcDirScanConfig (FcFontSet     *set,
                 FcStrSet       *dirs,
-                FcGlobalCache  *cache,
                 FcBlanks       *blanks,
                 const FcChar8  *dir,
                 FcBool         force,
@@ -126,8 +133,7 @@ FcDirScanConfig (FcFontSet  *set,
 {
     DIR                        *d;
     struct dirent      *e;
-    FcChar8            **dirlist;
-    int                        dirlistlen, dirlistalloc;
+    FcStrSet           *dirlist, *filelist;
     FcChar8            *file;
     FcChar8            *base;
     FcBool             ret = FcTrue;
@@ -139,18 +145,13 @@ FcDirScanConfig (FcFontSet        *set,
 
     if (!force)
     {
-       /*
-        * Check ~/.fonts.cache-<version> file
-        */
-       if (cache && FcGlobalCacheReadDir (set, dirs, cache, (char *)dir, config))
-           return FcTrue;
-
-       if (FcDirCacheValid (dir, config) && 
-           FcDirCacheHasCurrentArch (dir, config) &&
-           FcDirCacheRead (set, dirs, dir, config))
+       if (FcDirCacheRead (set, dirs, dir, config))
            return FcTrue;
     }
     
+    if (FcDebug () & FC_DBG_FONTSET)
+       printf ("cache scan dir %s\n", dir);
+
     /* freed below */
     file = (FcChar8 *) malloc (strlen ((char *) dir) + 1 + FC_MAX_FILE_LEN + 1);
     if (!file)
@@ -180,75 +181,80 @@ FcDirScanConfig (FcFontSet        *set,
        goto bail0;
     }
 
-    dirlistlen = 0;
-    dirlistalloc = 8;
-    dirlist = malloc(dirlistalloc * sizeof(FcChar8 *));
+    dirlist = FcStrSetCreate ();
     if (!dirlist) 
     {
        ret = FcFalse;
        goto bail1;
     }
+    filelist = FcStrSetCreate ();
+    if (!filelist)
+    {
+       ret = FcFalse;
+       goto bail2;
+    }
     while ((e = readdir (d)))
     {
        if (e->d_name[0] != '.' && strlen (e->d_name) < FC_MAX_FILE_LEN)
        {
-           if (dirlistlen == dirlistalloc)
-           {
-                FcChar8        **tmp_dirlist;
-
-               dirlistalloc *= 2;
-               tmp_dirlist = realloc(dirlist, 
-                                      dirlistalloc * sizeof(FcChar8 *));
-               if (!tmp_dirlist) 
-                {
+           strcpy ((char *) base, (char *) e->d_name);
+           if (FcFileIsDir (file)) {
+               if (!FcStrSetAdd (dirlist, file)) {
                    ret = FcFalse;
-                   goto bail2;
+                   goto bail3;
+               }
+           } else {
+               if (!FcStrSetAdd (filelist, file)) {
+                   ret = FcFalse;
+                   goto bail3;
                }
-                dirlist = tmp_dirlist;
-           }
-           dirlist[dirlistlen] = malloc(strlen (e->d_name) + 1);
-           if (!dirlist[dirlistlen]) 
-            {
-               ret = FcFalse;
-               goto bail2;
            }
-           strcpy((char *)dirlist[dirlistlen], e->d_name);
-           dirlistlen++;
        }
     }
-    qsort(dirlist, dirlistlen, sizeof(FcChar8 *), cmpstringp);
-    i = 0;
-    while (ret && i < dirlistlen)
+    /*
+     * Sort files and dirs to make things prettier
+     */
+    qsort(dirlist->strs, dirlist->num, sizeof(FcChar8 *), cmpstringp);
+    qsort(filelist->strs, filelist->num, sizeof(FcChar8 *), cmpstringp);
+    
+    for (i = 0; i < filelist->num; i++)
+       FcFileScanFontConfig (tmpSet, blanks, filelist->strs[i], config);
+    
+    if (FcShouldWriteFiles ())
     {
-       strcpy ((char *) base, (char *) dirlist[i]);
-       ret = FcFileScanConfig (tmpSet, dirs, cache, blanks, file, force, config);
-       i++;
+       /*
+        * Now that the directory has been scanned,
+        * write out the cache file
+        */
+       FcDirCacheWrite (tmpSet, dirlist, dir, config);
     }
+
     /*
-     * Now that the directory has been scanned,
-     * add the cache entry 
+     * Add the discovered fonts to our internal non-cache list
      */
-    if (ret && cache)
-       FcGlobalCacheUpdate (cache, dirs, (char *)dir, tmpSet, config);
-
     for (i = 0; i < tmpSet->nfont; i++)
        FcFontSetAdd (set, tmpSet->fonts[i]);
 
-    if (tmpSet->fonts)
-    {
-       FcMemFree (FC_MEM_FONTPTR, tmpSet->sfont * sizeof (FcPattern *));
-       free (tmpSet->fonts);
-    }
-    FcMemFree (FC_MEM_FONTSET, sizeof (FcFontSet));
+    /*
+     * the patterns in tmpset now belong to set; don't free them 
+     */
+    tmpSet->nfont = 0;
 
+    /*
+     * Add the discovered directories to the list to be scanned
+     */
+    for (i = 0; i < dirlist->num; i++)
+       if (!FcStrSetAdd (dirs, dirlist->strs[i])) {
+           ret = FcFalse;
+           goto bail3;
+       }
+    
+ bail3:
+    FcStrSetDestroy (filelist);
  bail2:
-    for (i = 0; i < dirlistlen; i++)
-       free(dirlist[i]);
-
-    free (dirlist);
-
+    FcStrSetDestroy (dirlist);
  bail1:
-    free (tmpSet);
+    FcFontSetDestroy (tmpSet);
     
  bail0:
     closedir (d);
@@ -260,12 +266,12 @@ FcDirScanConfig (FcFontSet        *set,
 FcBool
 FcDirScan (FcFontSet       *set,
           FcStrSet         *dirs,
-          FcGlobalCache    *cache,
+          FcFileCache      *cache, /* XXX unused */
           FcBlanks         *blanks,
           const FcChar8    *dir,
           FcBool           force)
 {
-    return FcDirScanConfig (set, dirs, cache, blanks, dir, force, 0);
+    return FcDirScanConfig (set, dirs, blanks, dir, force, 0);
 }
 
 FcBool
index 6a61b3912e8b5c3565672b3a83814a790b96ea76..494552ef44b241be96b9a75e479414e9e272ea07 100644 (file)
@@ -259,7 +259,9 @@ typedef struct _FcStrBuf {
 
 typedef struct _FcCache {
     int            magic;              /* FC_CACHE_MAGIC */
-    int            count;              /* number of bytes of data in block */
+    int            subdirs;            /* number of subdir strings */
+    off_t   pos;               /* position of data block in file */
+    off_t   count;              /* number of bytes of data in block */
     int     bank;               /* bank ID */
     int     pattern_count;      /* number of FcPatterns */
     int     patternelt_count;   /* number of FcPatternElts */
@@ -323,39 +325,8 @@ typedef struct _FcCaseFold {
 
 #define fc_alignof(type) offsetof (struct { char c; type member; }, member)
 
-/*
- * The per-user ~/.fonts.cache-<version> file is loaded into
- * this data structure.  Each directory gets a substructure
- * which is validated by comparing the directory timestamp with
- * that saved in the cache.  When valid, the entire directory cache
- * can be immediately loaded without reading the directory.  Otherwise,
- * the files are checked individually; updated files are loaded into the
- * cache which is then rewritten to the users home directory
- */
-
 #define FC_CACHE_MAGIC 0xFC02FC04
 
-typedef struct _FcGlobalCacheDir FcGlobalCacheDir;
-
-enum FcGCDirState {
-       FcGCDirDisabled, FcGCDirFileRead, FcGCDirConsumed, FcGCDirUpdated
-};
-struct _FcGlobalCacheDir {
-    struct _FcGlobalCacheDir   *next;
-    char                       *name;
-    FcCache                    metadata;
-    off_t                      offset;
-    FcStrSet                   *subdirs;
-    void                       *ent;
-    enum FcGCDirState          state;
-};
-
-typedef struct _FcGlobalCache {
-    FcGlobalCacheDir           *dirs;
-    FcBool                     updated;
-    int                                fd;
-} FcGlobalCache;
-
 struct _FcAtomic {
     FcChar8    *file;          /* original file name */
     FcChar8    *new;           /* temp file name -- write data here */
@@ -445,39 +416,8 @@ typedef struct _FcCharMap FcCharMap;
 
 /* fccache.c */
 
-FcGlobalCache *
-FcGlobalCacheCreate (void);
-
-void
-FcGlobalCacheDestroy (FcGlobalCache *cache);
-
-FcBool
-FcGlobalCacheReadDir (FcFontSet     *set, 
-                     FcStrSet      *dirs, 
-                     FcGlobalCache *cache, 
-                     const char    *dir, 
-                     FcConfig      *config);
-
-void
-FcGlobalCacheLoad (FcGlobalCache    *cache,
-                   FcStrSet        *staleDirs,
-                  const FcChar8    *cache_file,
-                  FcConfig         *config);
-
-FcBool
-FcGlobalCacheUpdate (FcGlobalCache  *cache,
-                    FcStrSet       *dirs,
-                    const char     *file,
-                    FcFontSet      *set,
-                    FcConfig       *config);
-
-FcBool
-FcGlobalCacheSave (FcGlobalCache    *cache,
-                  const FcChar8    *cache_file,
-                  FcConfig         *config);
-
 FcFontSet *
-FcCacheRead (FcConfig *config, FcGlobalCache * cache);
+FcCacheRead (FcConfig *config);
 
 FcBool
 FcDirCacheWrite (FcFontSet *set, FcStrSet * dirs, const FcChar8 *dir, FcConfig *config);
@@ -654,7 +594,6 @@ FcFileIsDir (const FcChar8 *file);
 FcBool
 FcFileScanConfig (FcFontSet    *set,
                  FcStrSet      *dirs,
-                 FcFileCache   *cache,
                  FcBlanks      *blanks,
                  const FcChar8 *file,
                  FcBool        force,
@@ -663,7 +602,6 @@ FcFileScanConfig (FcFontSet *set,
 FcBool
 FcDirScanConfig (FcFontSet     *set,
                 FcStrSet       *dirs,
-                FcFileCache    *cache,
                 FcBlanks       *blanks,
                 const FcChar8  *dir,
                 FcBool         force,