<div dir="ltr"><div>For the series:<br><br></div>Reviewed-by: Jasper St. Pierre <<a href="mailto:jstpierre@mecheye.net">jstpierre@mecheye.net</a>><br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Jan 8, 2014 at 2:35 AM, Alan Coopersmith <span dir="ltr"><<a href="mailto:alan.coopersmith@oracle.com" target="_blank">alan.coopersmith@oracle.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">catalogue.c: In function 'CatalogueOpenFont':<br>
catalogue.c:290:22: warning: variable 'dir' set but not used [-Wunused-but-set-variable]<br>
catalogue.c: In function 'CatalogueListFonts':<br>
catalogue.c:324:22: warning: variable 'dir' set but not used [-Wunused-but-set-variable]<br>
fpe.c: In function 'BuiltinResetFPE':<br>
fpe.c:57:22: warning: variable 'dir' set but not used [-Wunused-but-set-variable]<br>
<br>
Signed-off-by: Alan Coopersmith <<a href="mailto:alan.coopersmith@oracle.com">alan.coopersmith@oracle.com</a>><br>
---<br>
 src/builtins/fpe.c       |    3 ---<br>
 src/fontfile/catalogue.c |    4 ----<br>
 2 files changed, 7 deletions(-)<br>
<br>
diff --git a/src/builtins/fpe.c b/src/builtins/fpe.c<br>
index 83905da..689f22a 100644<br>
--- a/src/builtins/fpe.c<br>
+++ b/src/builtins/fpe.c<br>
@@ -54,9 +54,6 @@ BuiltinInitFPE (FontPathElementPtr fpe)<br>
 static int<br>
 BuiltinResetFPE (FontPathElementPtr fpe)<br>
 {<br>
-    FontDirectoryPtr   dir;<br>
-<br>
-    dir = (FontDirectoryPtr) fpe->private;<br>
     /* builtins can't change! */<br>
     return Successful;<br>
 }<br>
diff --git a/src/fontfile/catalogue.c b/src/fontfile/catalogue.c<br>
index 09ca2ae..6b6fc09 100644<br>
--- a/src/fontfile/catalogue.c<br>
+++ b/src/fontfile/catalogue.c<br>
@@ -287,7 +287,6 @@ CatalogueOpenFont (pointer client, FontPathElementPtr fpe, Mask flags,<br>
 {<br>
     CataloguePtr cat = fpe->private;<br>
     FontPathElementPtr subfpe;<br>
-    FontDirectoryPtr dir;<br>
     int i, status;<br>
<br>
     CatalogueRescan (fpe, FALSE);<br>
@@ -295,7 +294,6 @@ CatalogueOpenFont (pointer client, FontPathElementPtr fpe, Mask flags,<br>
     for (i = 0; i < cat->fpeCount; i++)<br>
     {<br>
        subfpe = cat->fpeList[i];<br>
-       dir = subfpe->private;<br>
        status = FontFileOpenFont(client, subfpe, flags,<br>
                                  name, namelen, format, fmask, id,<br>
                                  pFont, aliasName, non_cachable_font);<br>
@@ -321,7 +319,6 @@ CatalogueListFonts (pointer client, FontPathElementPtr fpe, char *pat,<br>
 {<br>
     CataloguePtr cat = fpe->private;<br>
     FontPathElementPtr subfpe;<br>
-    FontDirectoryPtr dir;<br>
     int i;<br>
<br>
     CatalogueRescan (fpe, FALSE);<br>
@@ -329,7 +326,6 @@ CatalogueListFonts (pointer client, FontPathElementPtr fpe, char *pat,<br>
     for (i = 0; i < cat->fpeCount; i++)<br>
     {<br>
        subfpe = cat->fpeList[i];<br>
-       dir = subfpe->private;<br>
        FontFileListFonts(client, subfpe, pat, len, max, names);<br>
     }<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
1.7.9.2<br>
<br>
_______________________________________________<br>
<a href="mailto:xorg-devel@lists.x.org">xorg-devel@lists.x.org</a>: X.Org development<br>
Archives: <a href="http://lists.x.org/archives/xorg-devel" target="_blank">http://lists.x.org/archives/xorg-devel</a><br>
Info: <a href="http://lists.x.org/mailman/listinfo/xorg-devel" target="_blank">http://lists.x.org/mailman/listinfo/xorg-devel</a><br>
</font></span></blockquote></div><br><br clear="all"><br>-- <br>  Jasper<br>
</div>