changeset 1592:752949e9aec4 trunk

[svn] Actually cast variables to glib types before feeding them into glib compare functions. Resolves another 16 signedness warnings on PPC.
author chainsaw
date Thu, 24 Aug 2006 12:23:54 -0700
parents 02841f72b897
children 6647d5cc717c
files ChangeLog Plugins/Visualization/paranormal/pn/pnactuator.c Plugins/Visualization/paranormal/pn/pnbooleanoption.c Plugins/Visualization/paranormal/pn/pncontainer.c Plugins/Visualization/paranormal/pn/pnfloatoption.c Plugins/Visualization/paranormal/pn/pnintegeroption.c Plugins/Visualization/paranormal/pn/pnlistoption.c Plugins/Visualization/paranormal/pn/pnstringoption.c Plugins/Visualization/paranormal/pn/pnvis.c
diffstat 9 files changed, 34 insertions(+), 13 deletions(-) [+]
line wrap: on
line diff
--- a/ChangeLog	Thu Aug 24 11:40:00 2006 -0700
+++ b/ChangeLog	Thu Aug 24 12:23:54 2006 -0700
@@ -1,3 +1,21 @@
+2006-08-24 18:40:00 +0000  Tony Vroon <chainsaw@gentoo.org>
+  revision [2105]
+  Cast variable to proper type before passing it into libxml2. Include necessary header to make this possible. Resolves 35 pointer signedness warnings on PPC.
+
+  Changes:        Modified:
+  +1 -1           trunk/Plugins/Visualization/paranormal/client.c  
+  +2 -2           trunk/Plugins/Visualization/paranormal/pn/pnactuator.c  
+  +3 -3           trunk/Plugins/Visualization/paranormal/pn/pnbooleanoption.c  
+  +2 -2           trunk/Plugins/Visualization/paranormal/pn/pncontainer.c  
+  +2 -2           trunk/Plugins/Visualization/paranormal/pn/pnfloatoption.c  
+  +2 -2           trunk/Plugins/Visualization/paranormal/pn/pnintegeroption.c  
+  +2 -2           trunk/Plugins/Visualization/paranormal/pn/pnlistoption.c  
+  +2 -2           trunk/Plugins/Visualization/paranormal/pn/pnstringoption.c  
+  +1 -1           trunk/Plugins/Visualization/paranormal/pn/pnuserobject.c  
+  +5 -5           trunk/Plugins/Visualization/paranormal/pn/pnvis.c  
+  +2 -1           trunk/Plugins/Visualization/paranormal/pn/pnxml.h  
+
+
 2006-08-24 18:11:30 +0000  Tony Vroon <chainsaw@gentoo.org>
   revision [2103]
   Sync with upstream. ADL is now dual licensed, and RIX has had a pitch fixup. Free warning fix included.
--- a/Plugins/Visualization/paranormal/pn/pnactuator.c	Thu Aug 24 11:40:00 2006 -0700
+++ b/Plugins/Visualization/paranormal/pn/pnactuator.c	Thu Aug 24 12:23:54 2006 -0700
@@ -166,7 +166,7 @@
 
   /* find the 'options' node */
   for (options_node = node->xmlChildrenNode; options_node; options_node = options_node->next)
-    if (g_strcasecmp (options_node->name, "Options") == 0)
+    if (g_strcasecmp ((gchar *) options_node->name, "Options") == 0)
       break;
 
   /* load each of the options */
@@ -174,7 +174,7 @@
     {
       for (option_node = options_node->xmlChildrenNode; option_node; option_node = option_node->next)
 	{
-	  if (!g_strcasecmp(option_node->name, "text"))
+	  if (!g_strcasecmp((gchar *) option_node->name, "text"))
             continue;
 
 	  option = pn_actuator_get_option_by_name (actuator, option_node->name);
--- a/Plugins/Visualization/paranormal/pn/pnbooleanoption.c	Thu Aug 24 11:40:00 2006 -0700
+++ b/Plugins/Visualization/paranormal/pn/pnbooleanoption.c	Thu Aug 24 12:23:54 2006 -0700
@@ -125,7 +125,7 @@
   for (boolean_option_node = node->xmlChildrenNode;
        boolean_option_node;
        boolean_option_node = boolean_option_node->next)
-    if (g_strcasecmp (boolean_option_node->name, "Value") == 0)
+    if (g_strcasecmp ((gchar *) boolean_option_node->name, "Value") == 0)
       break;
   if (! boolean_option_node)
     {
--- a/Plugins/Visualization/paranormal/pn/pncontainer.c	Thu Aug 24 11:40:00 2006 -0700
+++ b/Plugins/Visualization/paranormal/pn/pncontainer.c	Thu Aug 24 12:23:54 2006 -0700
@@ -165,7 +165,7 @@
 
   /* find the 'actuators' node */
   for (actuators_node = node->xmlChildrenNode; actuators_node; actuators_node = actuators_node->next)
-    if (g_strcasecmp (actuators_node->name, "Actuators") == 0)
+    if (g_strcasecmp ((gchar *) actuators_node->name, "Actuators") == 0)
       break;
 
   /* load each of the actuators */
@@ -173,7 +173,7 @@
     {
       for (actuator_node = actuators_node->xmlChildrenNode; actuator_node; actuator_node = actuator_node->next)
 	{
-	  if (!g_strcasecmp(actuator_node->name, "text"))
+	  if (!g_strcasecmp((gchar *) actuator_node->name, "text"))
 	    continue;
 
 	  actuator = pn_actuator_factory_new_actuator_from_xml (actuator_node);
--- a/Plugins/Visualization/paranormal/pn/pnfloatoption.c	Thu Aug 24 11:40:00 2006 -0700
+++ b/Plugins/Visualization/paranormal/pn/pnfloatoption.c	Thu Aug 24 12:23:54 2006 -0700
@@ -134,7 +134,7 @@
   for (float_option_node = node->xmlChildrenNode;
        float_option_node;
        float_option_node = float_option_node->next)
-    if (g_strcasecmp (float_option_node->name, "Value") == 0)
+    if (g_strcasecmp ((gchar *) float_option_node->name, "Value") == 0)
       break;
   if (! float_option_node)
     {
--- a/Plugins/Visualization/paranormal/pn/pnintegeroption.c	Thu Aug 24 11:40:00 2006 -0700
+++ b/Plugins/Visualization/paranormal/pn/pnintegeroption.c	Thu Aug 24 12:23:54 2006 -0700
@@ -134,7 +134,7 @@
   for (integer_option_node = node->xmlChildrenNode;
        integer_option_node;
        integer_option_node = integer_option_node->next)
-    if (g_strcasecmp (integer_option_node->name, "Value") == 0)
+    if (g_strcasecmp ((gchar *) integer_option_node->name, "Value") == 0)
       break;
   if (! integer_option_node)
     {
--- a/Plugins/Visualization/paranormal/pn/pnlistoption.c	Thu Aug 24 11:40:00 2006 -0700
+++ b/Plugins/Visualization/paranormal/pn/pnlistoption.c	Thu Aug 24 12:23:54 2006 -0700
@@ -131,7 +131,7 @@
   for (list_option_node = node->xmlChildrenNode;
        list_option_node;
        list_option_node = list_option_node->next)
-    if (g_strcasecmp (list_option_node->name, "Value") == 0)
+    if (g_strcasecmp ((gchar *) list_option_node->name, "Value") == 0)
       break;
   if (! list_option_node)
     {
--- a/Plugins/Visualization/paranormal/pn/pnstringoption.c	Thu Aug 24 11:40:00 2006 -0700
+++ b/Plugins/Visualization/paranormal/pn/pnstringoption.c	Thu Aug 24 12:23:54 2006 -0700
@@ -128,7 +128,7 @@
   for (string_option_node = node->xmlChildrenNode;
        string_option_node;
        string_option_node = string_option_node->next)
-    if (g_strcasecmp (string_option_node->name, "Value") == 0)
+    if (g_strcasecmp ((gchar *) string_option_node->name, "Value") == 0)
       break;
   if (! string_option_node)
     {
--- a/Plugins/Visualization/paranormal/pn/pnvis.c	Thu Aug 24 11:40:00 2006 -0700
+++ b/Plugins/Visualization/paranormal/pn/pnvis.c	Thu Aug 24 12:23:54 2006 -0700
@@ -155,7 +155,10 @@
 pn_vis_load_thyself (PnUserObject *user_object, const xmlNodePtr node)
 {
   PnVis *vis;
-  xmlNodePtr actuators_node, actuator_node, tptr;
+  xmlNodePtr actuators_node, actuator_node;
+#if 0
+, tptr;
+#endif
   PnActuator *root_actuator = NULL;
 
   g_return_if_fail (user_object != NULL);
@@ -166,7 +169,7 @@
 
   /* find the 'Actuators' node */
   for (actuators_node = node->xmlChildrenNode; actuators_node; actuators_node = actuators_node->next)
-    if (g_strcasecmp (actuators_node->name, "Actuators") == 0)
+    if (g_strcasecmp ((gchar *) actuators_node->name, "Actuators") == 0)
       break;
   if (! actuators_node)
     {
@@ -175,7 +178,7 @@
     }
       
   /* get the root actuator's node */
-  for (actuator_node = actuators_node->xmlChildrenNode; actuator_node != NULL && !g_strcasecmp(actuator_node->name, "text");
+  for (actuator_node = actuators_node->xmlChildrenNode; actuator_node != NULL && !g_strcasecmp((gchar *) actuator_node->name, "text");
 	actuator_node = actuator_node->next);
   if (! actuator_node)
     goto done;
@@ -305,7 +308,7 @@
       return FALSE;
     }
 
-  if (g_strcasecmp (root_node->name, pn_user_object_get_name (PN_USER_OBJECT (vis))))
+  if (g_strcasecmp ((gchar *) root_node->name, pn_user_object_get_name (PN_USER_OBJECT (vis))))
     {
       pn_error ("invalid file format: this file does not contain a Paranormal visualization");
       return FALSE;