smyrna: use an 'unsigned' for 'topviewcache.selnode_id'

Squashes 4 -Wsign-conversion warnings. Note that this also changes the sentinel
from `-1` to `UINT_MAX`.

diff --git a/cmd/smyrna/smyrnadefs.h b/cmd/smyrna/smyrnadefs.h
index 44a4ffd655f61284e6758e0d54e8a292a510381f..b94ec3e8cc4ca823d2b994bf502a87c3712be9e7 100644 (file)
--- a/cmd/smyrna/smyrnadefs.h
+++ b/cmd/smyrna/smyrnadefs.h
@@ -142,7 +142,7 @@ typedef struct
     {
        unsigned node_id;
        unsigned edge_id;
-       int selnode_id;
+       unsigned selnode_id;
        int seledge_id;
        int nodelabel_id;
        int edgelabel_id;

diff --git a/cmd/smyrna/topviewfuncs.c b/cmd/smyrna/topviewfuncs.c
index 564fa0fa36898d5cfd84e22d75054f9cc7582669..bf393552d017a4f895b27949e0532e6c425c30de 100644 (file)
--- a/cmd/smyrna/topviewfuncs.c
+++ b/cmd/smyrna/topviewfuncs.c
@@ -780,7 +780,7 @@ void cacheSelectedEdges(Agraph_t * g,topview* t)
 }
 void cacheSelectedNodes(Agraph_t * g,topview* t)
 {
-    if(t->cache.selnode_id!=-1)        /*clean existing cache*/
+    if (t->cache.selnode_id != UINT_MAX) // clean existing cache
        glDeleteLists(t->cache.selnode_id,1);
     t->cache.selnode_id=glGenLists(1);
     glNewList(t->cache.selnode_id,GL_COMPILE);
@@ -864,7 +864,7 @@ void initSmGraph(Agraph_t * g,topview* rv)
 
     rv->fisheyeParams.active = 0;
     rv->cache.node_id = UINT_MAX;
-    rv->cache.selnode_id=-1;
+    rv->cache.selnode_id = UINT_MAX;
     rv->cache.edge_id = UINT_MAX;
     rv->cache.seledge_id=-1;
     rv->sel.selectEdges=0;