A handful of changes after applying pull request #191 from

Alessandro De Laurenzis.  That pull request cleaned up the vast
majority of compiler warnings.  However, that cleanup exposed a
few additional warnings pointing to errors in the code that needed
fixing.  The code now compiles cleanly except for one warning
about redefined CAD_DIR that I have not looked into.

VERSION

@@ -1 +1 @@
-8.3.332
+8.3.333

calma/CalmaRead.c

@@ -299,7 +299,7 @@ done:
     HashKill(&calmaDefInitHash);
     UndoEnable();
 
-    if (calmaErrorFile != NULL) FCLOSE(calmaErrorFile);
+    if (calmaErrorFile != NULL) fclose(calmaErrorFile);
 }
 
 /*

cif/CIFrdpoly.c

@@ -21,6 +21,7 @@ static char rcsid[] __attribute__ ((unused)) = "$Header: /usr/cvsroot/magic-8.0/
 #endif  /* not lint */
 
 #include <stdio.h>
+#include <stdlib.h>		/* For qsort() */
 #include "utils/magic.h"
 #include "utils/geometry.h"
 #include "tiles/tile.h"

commands/CmdCD.c

@@ -130,7 +130,8 @@ CmdCalma(w, cmd)
     char **msg, *namep, *dotptr;
     char writeMode[3];
     CellDef *rootDef;
-    FILE *f;
+    FILETYPE f;
+    FILE *fp;
 #ifdef HAVE_ZLIB
     gzFile fz;
 #endif
@@ -966,19 +967,19 @@ outputCalma:
     {
 
 #endif
-	f = PaOpen(namep, "w", (dotptr == NULL) ? ".gds" : "", ".", (char *) NULL, (char **)NULL);
+	fp = PaOpen(namep, "w", (dotptr == NULL) ? ".gds" : "", ".", (char *) NULL, (char **)NULL);
-	if (f == (FILE *)NULL)
+	if (fp == (FILE *)NULL)
 	{
 	    TxError("Cannot open %s%s to write GDS-II stream output\n", namep,
 			(dotptr == NULL) ? ".gds" : "");
 	    return;
 	}
-	if (!CalmaWrite(rootDef, f))
+	if (!CalmaWrite(rootDef, fp))
 	{
 	    TxError("I/O error in writing file %s.\n", namep);
 	    TxError("File may be incompletely written.\n");
 	}
-	(void) fclose(f);
+	(void) fclose(fp);
 
 #ifdef HAVE_ZLIB
     }

garouter/gaChannel.c

@@ -43,7 +43,7 @@ static char rcsid[] __attribute__ ((unused)) = "$Header: /usr/cvsroot/magic-8.0/
 #include "debug/debug.h"
 
 /* C99 compat */
-#include "gaRouter/gaInternal.h"
+#include "garouter/gaInternal.h"
 
 /* List of all active channels */
 GCRChannel *gaChannelList = NULL;

graphics/grTOGL1.c

@@ -1337,6 +1337,7 @@ GrTOGLLower(w)
 
 #ifdef CAIRO_OFFSCREEN_RENDER
 extern void GrTCairoLock();
+extern void GrTCairoUnlock();
 extern void TCairoOffScreen();
 #endif
 

grouter/groutePen.c

@@ -22,6 +22,7 @@ static char rcsid[] __attribute__ ((unused)) = "$Header: /usr/cvsroot/magic-8.0/
 #endif	/* lint */
 
 #include <stdio.h>
+#include <stdlib.h>			/* For qsort() */
 #include "utils/magic.h"
 #include "utils/geometry.h"
 #include "utils/hash.h"

utils/child.c

@@ -10,9 +10,7 @@
  *************************************************************************/
 #include <stdio.h>
 #include <string.h>
-#ifndef	SYSV
 #include <sys/wait.h>
-#endif /* SYSV */
 #include <errno.h>
 
 #include "utils/utils.h"

utils/utils.h

@@ -25,6 +25,7 @@
 #include <zlib.h>
 #endif
 
+#include "utils/tech.h"
 #include "utils/magic.h"
 
 /*