Eliminate NormalizeDir. Eliminate gratuitous stat/access calls per dir.

[fontconfig.git] / src / fcdir.c

diff --git a/src/fcdir.c b/src/fcdir.c
index f505bf533b3396641c25296589a6d2cb3de0b92d..145226072571abe2d4a6880ee1727359edb26501 100644 (file)
--- a/src/fcdir.c
+++ b/src/fcdir.c
@@ -129,7 +129,6 @@ FcDirScanConfig (FcFontSet  *set,
     FcChar8            **dirlist;
     int                        dirlistlen, dirlistalloc;
     FcChar8            *file;
-    const FcChar8      *d_can = 0;
     FcChar8            *base;
     FcBool             ret = FcTrue;
     FcFontSet          *tmpSet;
@@ -138,11 +137,6 @@ FcDirScanConfig (FcFontSet *set,
     if (config && !FcConfigAcceptFilename (config, dir))
        return FcTrue;
 
-    if (config)
-       d_can = FcConfigNormalizeFontDir (config, dir);
-    if (d_can)
-       dir = d_can;
-
     if (!force)
     {
        /*
@@ -151,8 +145,8 @@ FcDirScanConfig (FcFontSet  *set,
        if (cache && FcGlobalCacheReadDir (set, dirs, cache, (char *)dir, config))
            return FcTrue;
 
-       if (FcDirCacheValid (dir) && 
-           FcDirCacheHasCurrentArch (dir) &&
+       if (FcDirCacheValid (dir, config) && 
+           FcDirCacheHasCurrentArch (dir, config) &&
            FcDirCacheRead (set, dirs, dir, config))
            return FcTrue;
     }
@@ -180,31 +174,44 @@ FcDirScanConfig (FcFontSet        *set,
     }
 
     tmpSet = FcFontSetCreate();
-    if (!tmpSet)
-    {  
-       free (file);
-       return FcFalse;
+    if (!tmpSet) 
+    {
+       ret = FcFalse;
+       goto bail0;
     }
 
     dirlistlen = 0;
     dirlistalloc = 8;
     dirlist = malloc(dirlistalloc * sizeof(FcChar8 *));
-    if (!dirlist)
-       return FcFalse;
+    if (!dirlist) 
+    {
+       ret = FcFalse;
+       goto bail1;
+    }
     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;
-               dirlist = realloc(dirlist, dirlistalloc * sizeof(FcChar8 *));
-               if (!dirlist)
-                   return FcFalse;
+               tmp_dirlist = realloc(dirlist, 
+                                      dirlistalloc * sizeof(FcChar8 *));
+               if (!tmp_dirlist) 
+                {
+                   ret = FcFalse;
+                   goto bail2;
+               }
+                dirlist = tmp_dirlist;
            }
            dirlist[dirlistlen] = malloc(strlen (e->d_name) + 1);
-           if (!dirlist[dirlistlen])
-               return FcFalse;
+           if (!dirlist[dirlistlen]) 
+            {
+               ret = FcFalse;
+               goto bail2;
+           }
            strcpy((char *)dirlist[dirlistlen], e->d_name);
            dirlistlen++;
        }
@@ -217,11 +224,6 @@ FcDirScanConfig (FcFontSet *set,
        ret = FcFileScanConfig (tmpSet, dirs, cache, blanks, file, force, config);
        i++;
     }
-    for (i = 0; i < dirlistlen; i++)
-       free(dirlist[i]);
-    free (dirlist);
-    free (file);
-    closedir (d);
     /*
      * Now that the directory has been scanned,
      * add the cache entry 
@@ -238,8 +240,20 @@ FcDirScanConfig (FcFontSet *set,
        free (tmpSet->fonts);
     }
     FcMemFree (FC_MEM_FONTSET, sizeof (FcFontSet));
+
+ bail2:
+    for (i = 0; i < dirlistlen; i++)
+       free(dirlist[i]);
+
+    free (dirlist);
+
+ bail1:
     free (tmpSet);
-       
+    
+ bail0:
+    closedir (d);
+    
+    free (file);
     return ret;
 }
 
@@ -257,5 +271,5 @@ FcDirScan (FcFontSet            *set,
 FcBool
 FcDirSave (FcFontSet *set, FcStrSet * dirs, const FcChar8 *dir)
 {
-    return FcDirCacheWrite (set, dirs, dir);
+    return FcDirCacheWrite (set, dirs, dir, FcConfigGetCurrent ());
 }