Fix compile warnings (#648)

* Fix compile warnings for GCC

* Add empty default case

* Initialise adj and free unconditionaly

* Remove blank lines

Author: Joseph-Edwards

configure.ac

@@ -87,8 +87,7 @@ AS_IF([test "x$enable_compile_warnings" != "xno"],
 AS_IF([test "x$enable_compile_warnings" != "xno"],
   [ax_enable_compile_warnings=yes
    AX_COMPILER_FLAGS_CFLAGS([WARNING_CFLAGS], [], [ dnl
-       -pedantic dnl
-       -Wold-style-cast])
+       -pedantic dnl])
    AC_SUBST([WARNING_CFLAGS])])
 
 # Check for system planarity suite

extern/bliss-0.73/bliss_C.cc

@@ -29,6 +29,11 @@ extern "C" {
 #pragma clang diagnostic push
 #pragma clang diagnostic ignored "-Wold-style-cast"
 #pragma clang diagnostic ignored "-Wsign-compare"
+#elif defined(__GNUC__)
+#pragma GCC diagnostic push
+#pragma GCC diagnostic ignored "-Wold-style-cast"
+#pragma GCC diagnostic ignored "-Wsign-compare"
+#pragma GCC diagnostic ignored "-Wmissing-declarations"
 #endif
 
 struct bliss_digraphs_graph_struct {
@@ -238,4 +243,6 @@ void bliss_digraphs_free_blissstats(BlissStats *stats)
 
 #if defined(__clang__)
 #pragma clang diagnostic pop
+#elif defined(__GNUC__)
+#pragma GCC diagnostic pop
 #endif

extern/bliss-0.73/defs.cc

@@ -25,6 +25,11 @@
 #pragma clang diagnostic push
 #pragma clang diagnostic ignored "-Wformat-nonliteral"
 #pragma clang diagnostic ignored "-Wmissing-noreturn"
+#elif defined(__GNUC__)
+#pragma GCC diagnostic push
+#pragma GCC diagnostic ignored "-Wformat-nonliteral"
+#pragma GCC diagnostic ignored "-Wmissing-noreturn"
+#pragma GCC diagnostic ignored "-Wsuggest-attribute=format"
 #endif
 namespace bliss_digraphs {
 
@@ -42,4 +47,6 @@ namespace bliss_digraphs {
 
 #if defined(__clang__)
 #pragma clang diagnostic pop
+#elif defined(__GNUC__)
+#pragma GCC diagnostic pop
 #endif

extern/bliss-0.73/graph.cc

@@ -33,6 +33,9 @@
 #if defined(__clang__)
 #pragma clang diagnostic push
 #pragma clang diagnostic ignored "-Wold-style-cast"
+#elif defined(__GNUC__)
+#pragma GCC diagnostic push
+#pragma GCC diagnostic ignored "-Wold-style-cast"
 #endif
 
 namespace bliss_digraphs {

extern/bliss-0.73/timer.cc

@@ -24,6 +24,10 @@
 #if defined(__clang__)
 #pragma clang diagnostic push
 #pragma clang diagnostic ignored "-Wold-style-cast"
+#elif defined(__GNUC__)
+#pragma GCC diagnostic push
+#pragma GCC diagnostic ignored "-Wold-style-cast"
+#pragma GCC diagnostic ignored "-Wuninitialized"
 #endif
 
 namespace bliss_digraphs {
@@ -60,4 +64,6 @@ double Timer::get_duration()
 } // namespace bliss_digraphs
 #if defined(__clang__)
 #pragma clang diagnostic pop
+#elif defined(__GNUC__)
+#pragma GCC diagnostic pop
 #endif

extern/edge-addition-planarity-suite-Version_3.0.1.0/c/graphDFSUtils.c

@@ -31,6 +31,10 @@ extern void _ClearVertexVisitedFlags(graphP theGraph, int);
 #pragma clang diagnostic ignored "-Wstrict-prototypes"
 #pragma clang diagnostic ignored "-Wmissing-prototypes"
 #pragma clang diagnostic ignored "-Wdeprecated-non-prototype"
+#elif defined(__GNUC__)
+#pragma GCC diagnostic push
+#pragma GCC diagnostic ignored "-Wstrict-prototypes"
+#pragma GCC diagnostic ignored "-Wmissing-prototypes"
 #endif
 
 int  gp_CreateDFSTree(graphP theGraph)
@@ -459,4 +463,6 @@ printf("LeastAncestor in %.3lf seconds.\n", platform_GetDuration(start,end));
 }
 #if defined(__clang__)
 #pragma clang diagnostic pop
+#elif defined(__GNUC__)
+#pragma GCC diagnostic pop
 #endif

extern/edge-addition-planarity-suite-Version_3.0.1.0/c/graphDrawPlanar.c

@@ -9,6 +9,10 @@ See the LICENSE.TXT file for licensing information.
 #pragma clang diagnostic ignored "-Wstrict-prototypes"
 #pragma clang diagnostic ignored "-Wmissing-prototypes"
 #pragma clang diagnostic ignored "-Wdeprecated-non-prototype"
+#elif defined(__GNUC__)
+#pragma GCC diagnostic push
+#pragma GCC diagnostic ignored "-Wstrict-prototypes"
+#pragma GCC diagnostic ignored "-Wmissing-prototypes"
 #endif
 
 #include "graphDrawPlanar.h"
@@ -1047,4 +1051,6 @@ int v, e, eTwin, EsizeOccupied, epos, eposIndex;
 }
 #if defined(__clang__)
 #pragma clang diagnostic pop
+#elif defined(__GNUC__)
+#pragma GCC diagnostic pop
 #endif

extern/edge-addition-planarity-suite-Version_3.0.1.0/c/graphDrawPlanar_Extensions.c

@@ -3,20 +3,22 @@ Copyright (c) 1997-2020, John M. Boyer
 All rights reserved.
 See the LICENSE.TXT file for licensing information.
 */
-
-#if defined(__clang__)
-#pragma clang diagnostic push
-#pragma clang diagnostic ignored "-Wstrict-prototypes"
-#pragma clang diagnostic ignored "-Wmissing-prototypes"
-#pragma clang diagnostic ignored "-Wdeprecated-non-prototype"
-#endif
+
+#if defined(__clang__)
+#pragma clang diagnostic push
+#pragma clang diagnostic ignored "-Wstrict-prototypes"
+#pragma clang diagnostic ignored "-Wmissing-prototypes"
+#pragma clang diagnostic ignored "-Wdeprecated-non-prototype"
+#elif defined(__GNUC__)
+#pragma GCC diagnostic push
+#pragma GCC diagnostic ignored "-Wstrict-prototypes"
+#pragma GCC diagnostic ignored "-Wmissing-prototypes"
+#endif
 
 #include <stdlib.h>
 
 #include "graphDrawPlanar.private.h"
 #include "graphDrawPlanar.h"
-
-
 
 extern void _ClearVertexVisitedFlags(graphP theGraph, int);
 
@@ -59,7 +61,7 @@ void _DrawPlanar_FreeContext(void *);
 /****************************************************************************
  * DRAWPLANAR_ID - the variable used to hold the integer identifier for this
  * extension, enabling this feature's extension context to be distinguished
- * from other features' extension contexts that may be attached to a graph.
+ * from other features' extension contexts that may be attached to a graph.
  ****************************************************************************/
 
 int DRAWPLANAR_ID = 0;
@@ -687,6 +689,8 @@ int  _DrawPlanar_WritePostprocess(graphP theGraph, void **pExtraData, long *pExt
 
     return NOTOK;
 }
-#if defined(__clang__)
-#pragma clang diagnostic pop
-#endif
+#if defined(__clang__)
+#pragma clang diagnostic pop
+#elif defined(__GNUC__)
+#pragma GCC diagnostic pop
+#endif

extern/edge-addition-planarity-suite-Version_3.0.1.0/c/graphEmbed.c

@@ -13,6 +13,10 @@ See the LICENSE.TXT file for licensing information.
 #pragma clang diagnostic ignored "-Wstrict-prototypes"
 #pragma clang diagnostic ignored "-Wmissing-prototypes"
 #pragma clang diagnostic ignored "-Wdeprecated-non-prototype"
+#elif defined(__GNUC__)
+#pragma GCC diagnostic push
+#pragma GCC diagnostic ignored "-Wstrict-prototypes"
+#pragma GCC diagnostic ignored "-Wmissing-prototypes"
 #endif
 
 /* Imported functions */
@@ -1369,4 +1373,6 @@ int  e_u, e_v, e_ulink, e_vlink;
 
 #if defined(__clang__)
 #pragma clang diagnostic pop
+#elif defined(__GNUC__)
+#pragma GCC diagnostic pop
 #endif

extern/edge-addition-planarity-suite-Version_3.0.1.0/c/graphExtensions.c

@@ -13,6 +13,13 @@ See the LICENSE.TXT file for licensing information.
 #include "graphExtensions.h"
 #include "graphFunctionTable.h"
 
+#ifdef __GNUC__
+    #ifndef __clang__
+    #pragma GCC diagnostic push
+    #pragma GCC diagnostic ignored "-Wshadow"
+    #endif
+#endif
+
 /* Imported functions */
 
 extern void _InitFunctionTable(graphP theGraph);
@@ -30,7 +37,7 @@ graphExtensionP _FindNearestOverload(graphP theGraph, graphExtensionP target, in
  * An ID identifies an extension, which may be added to multiple
  * graphs.  It is used in lieu of identifying extensions by a string
  * name, which is noticeably expensive when a frequently called
- * overload function seeks the extension context for a graph.
+ * overload function seeks the extension context for a graph.
  ********************************************************************/
 
 static int moduleIDGenerator = 0;
@@ -544,3 +551,9 @@ void _FreeExtension(graphExtensionP extension)
     }
     free(extension);
 }
+
+#ifdef __GNUC__
+    #ifndef __clang__
+    #pragma GCC diagnostic pop
+    #endif
+#endif