Fix warnings about xmlChar cast

author: rbuj <[email protected]> 2021-02-21 10:30:00 +0100
committer: raveit65 <[email protected]> 2022-07-21 19:33:50 +0200
commit: 202a9c5ad39758616f73e23aabcbe43aaebea26b (patch)
tree: 402b5a9a629f3148c44dab9950f5af8bed066892
parent: 76f11ad9fd2c4a46211a115fe16c71dca9756503 (diff)
download: caja-202a9c5ad39758616f73e23aabcbe43aaebea26b.tar.bz2
caja-202a9c5ad39758616f73e23aabcbe43aaebea26b.tar.xz
5 files changed, 140 insertions, 126 deletions
diff --git a/eel/eel-xml-extensions.c b/eel/eel-xml-extensions.c
index da2cf6b5..855096c1 100644
--- a/eel/eel-xml-extensions.c
+++ b/eel/eel-xml-extensions.c
@@ -43,25 +43,25 @@ eel_xml_get_children (xmlNodePtr parent)
 }
 
 xmlNodePtr
-eel_xml_get_child_by_name_and_property (xmlNodePtr parent,
- const char *child_name,
- const char *property_name,
- const char *property_value)
+eel_xml_get_child_by_name_and_property (xmlNodePtr parent,
+ const xmlChar *child_name,
+ const xmlChar *property_name,
+ const xmlChar *property_value)
 {
 xmlNodePtr child;
 xmlChar *property;
 gboolean match;
 
- if (parent == NULL)
+ if (parent == NULL || property_name == NULL || property_value == NULL)
 {
 return NULL;
 }
- for (child = eel_xml_get_children (parent); child != NULL; child = child->next)
+ for (child = parent->children; child != NULL; child = child->next)
 {
- if (strcmp (child->name, child_name) == 0)
+ if (child->name != NULL && xmlStrcmp (child->name, child_name) == 0)
 {
 property = xmlGetProp (child, property_name);
- match = eel_strcmp (property, property_value) == 0;
+ match = property != NULL && xmlStrcmp (property, property_value) == 0;
 xmlFree (property);
 if (match)
 {
@@ -73,14 +73,13 @@ eel_xml_get_child_by_name_and_property (xmlNodePtr parent,
 }
 
 xmlNodePtr
-eel_xml_get_root_child_by_name_and_property (xmlDocPtr document,
- const char *child_name,
- const char *property_name,
- const char *property_value)
+eel_xml_get_root_child_by_name_and_property (xmlDocPtr document,
+ const xmlChar *child_name,
+ const xmlChar *property_name,
+ const xmlChar *property_value)
 {
- return eel_xml_get_child_by_name_and_property
- (xmlDocGetRootElement (document),
- child_name,
- property_name,
- property_value);
+ return eel_xml_get_child_by_name_and_property (xmlDocGetRootElement (document),
+ child_name,
+ property_name,
+ property_value);
 }
diff --git a/eel/eel-xml-extensions.h b/eel/eel-xml-extensions.h
index fba9ba57..7e746886 100644
--- a/eel/eel-xml-extensions.h
+++ b/eel/eel-xml-extensions.h
@@ -28,14 +28,14 @@
 #include <libxml/tree.h>
 #include <glib.h>
 
-xmlNodePtr eel_xml_get_children (xmlNodePtr parent);
-xmlNodePtr eel_xml_get_child_by_name_and_property (xmlNodePtr parent,
- const char *child_name,
- const char *property_name,
- const char *property_value);
-xmlNodePtr eel_xml_get_root_child_by_name_and_property (xmlDocPtr document,
- const char *child_name,
- const char *property_name,
- const char *property_value);
+xmlNodePtr eel_xml_get_children (xmlNodePtr parent);
+xmlNodePtr eel_xml_get_child_by_name_and_property (xmlNodePtr parent,
+ const xmlChar *child_name,
+ const xmlChar *property_name,
+ const xmlChar *property_value);
+xmlNodePtr eel_xml_get_root_child_by_name_and_property (xmlDocPtr document,
+ const xmlChar *child_name,
+ const xmlChar *property_name,
+ const xmlChar *property_value);
 
 #endif /* EEL_XML_EXTENSIONS_H */
diff --git a/libcaja-private/caja-customization-data.c b/libcaja-private/caja-customization-data.c
index d1f2a180..8caecba6 100644
--- a/libcaja-private/caja-customization-data.c
+++ b/libcaja-private/caja-customization-data.c
@@ -469,19 +469,24 @@ load_name_map_hash_table (CajaCustomizationData *data)
 if ((browser_data = xmlParseFile (CAJA_DATADIR "/browser.xml")) != NULL)
 {
 /* get the category node */
- category_node = eel_xml_get_root_child_by_name_and_property (browser_data, "category", "name", data->customization_name);
+ category_node = eel_xml_get_root_child_by_name_and_property (browser_data,
+ (const xmlChar *) "category",
+ (const xmlChar *) "name",
+ (const xmlChar *) data->customization_name);
 current_node = category_node->children;
 
 /* loop through the entries, adding a mapping to the hash table */
 while (current_node != NULL)
 {
- char *filename, *display_name;
+ xmlChar *filename, *display_name;
 
- display_name = xmlGetProp (current_node, "display_name");
- filename = xmlGetProp (current_node, "filename");
+ display_name = xmlGetProp (current_node, (const xmlChar *) "display_name");
+ filename = xmlGetProp (current_node, (const xmlChar *) "filename");
 if (display_name && filename)
 {
- g_hash_table_insert (data->name_map_hash, g_strdup (filename), g_strdup (_(display_name)));
+ g_hash_table_replace (data->name_map_hash,
+ g_strdup ((const char *) filename),
+ g_strdup (_((const char *) display_name)));
 }
 xmlFree (filename);
 xmlFree (display_name);
diff --git a/src/caja-application.c b/src/caja-application.c
index 7289d161..5c4f9cf6 100644
--- a/src/caja-application.c
+++ b/src/caja-application.c
@@ -1390,12 +1390,12 @@ caja_application_get_session_data (CajaApplication *self)
 xmlSaveCtxtPtr ctx;
 xmlBufferPtr buffer;
 
- doc = xmlNewDoc ("1.0");
+ doc = xmlNewDoc ((const xmlChar *) "1.0");
 
- root_node = xmlNewNode (NULL, "session");
+ root_node = xmlNewNode (NULL, (const xmlChar *) "session");
 xmlDocSetRootElement (doc, root_node);
 
- history_node = xmlNewChild (root_node, NULL, "history", NULL);
+ history_node = xmlNewChild (root_node, NULL, (const xmlChar *) "history", NULL);
 
 n_processed = 0;
 for (l = caja_get_history_list (); l != NULL; l = l->next) {
@@ -1406,10 +1406,10 @@ caja_application_get_session_data (CajaApplication *self)
 
 bookmark = l->data;
 
- bookmark_node = xmlNewChild (history_node, NULL, "bookmark", NULL);
+ bookmark_node = xmlNewChild (history_node, NULL, (const xmlChar *) "bookmark", NULL);
 
 tmp = caja_bookmark_get_name (bookmark);
- xmlNewProp (bookmark_node, "name", tmp);
+ xmlNewProp (bookmark_node, (const xmlChar *) "name", (const xmlChar *) tmp);
 g_free (tmp);
 
 icon = caja_bookmark_get_icon (bookmark);
@@ -1417,16 +1417,16 @@ caja_application_get_session_data (CajaApplication *self)
 tmp = g_icon_to_string (icon);
 g_object_unref (icon);
 if (tmp) {
- xmlNewProp (bookmark_node, "icon", tmp);
+ xmlNewProp (bookmark_node, (const xmlChar *) "icon", (const xmlChar *) tmp);
 g_free (tmp);
 }
 
 tmp = caja_bookmark_get_uri (bookmark);
- xmlNewProp (bookmark_node, "uri", tmp);
+ xmlNewProp (bookmark_node, (const xmlChar *) "uri", (const xmlChar *) tmp);
 g_free (tmp);
 
 if (caja_bookmark_get_has_custom_name (bookmark)) {
- xmlNewProp (bookmark_node, "has_custom_name", "TRUE");
+ xmlNewProp (bookmark_node, (const xmlChar *) "has_custom_name", (const xmlChar *) "TRUE");
 }
 
 if (++n_processed > 50) { /* prevent history list from growing arbitrarily large. */
@@ -1457,49 +1457,49 @@ caja_application_get_session_data (CajaApplication *self)
 continue;
 }
 
- win_node = xmlNewChild (root_node, NULL, "window", NULL);
+ win_node = xmlNewChild (root_node, NULL, (const xmlChar *) "window", NULL);
 
- xmlNewProp (win_node, "location", tmp);
+ xmlNewProp (win_node, (const xmlChar *) "location", (const xmlChar *) tmp);
 g_free (tmp);
 
- xmlNewProp (win_node, "type", CAJA_IS_NAVIGATION_WINDOW (window) ? "navigation" : "spatial");
+ xmlNewProp (win_node, (const xmlChar *) "type", CAJA_IS_NAVIGATION_WINDOW (window) ? (const xmlChar *) "navigation" : (const xmlChar *) "spatial");
 
 if (CAJA_IS_NAVIGATION_WINDOW (window)) { /* spatial windows store their state as file metadata */
 GdkWindow *gdk_window;
 
 tmp = eel_gtk_window_get_geometry_string (GTK_WINDOW (window));
- xmlNewProp (win_node, "geometry", tmp);
+ xmlNewProp (win_node, (const xmlChar *) "geometry", (const xmlChar *) tmp);
 g_free (tmp);
 
 gdk_window = gtk_widget_get_window (GTK_WIDGET (window));
 
 if (gdk_window &&
 gdk_window_get_state (gdk_window) & GDK_WINDOW_STATE_MAXIMIZED) {
- xmlNewProp (win_node, "maximized", "TRUE");
+ xmlNewProp (win_node, (const xmlChar *) "maximized", (const xmlChar *) "TRUE");
 }
 
 if (gdk_window &&
 gdk_window_get_state (gdk_window) & GDK_WINDOW_STATE_STICKY) {
- xmlNewProp (win_node, "sticky", "TRUE");
+ xmlNewProp (win_node, (const xmlChar *) "sticky", (const xmlChar *) "TRUE");
 }
 
 if (gdk_window &&
 gdk_window_get_state (gdk_window) & GDK_WINDOW_STATE_ABOVE) {
- xmlNewProp (win_node, "keep-above", "TRUE");
+ xmlNewProp (win_node, (const xmlChar *) "keep-above", (const xmlChar *) "TRUE");
 }
 }
 
 for (m = slots; m != NULL; m = m->next) {
 slot = CAJA_WINDOW_SLOT (m->data);
 
- slot_node = xmlNewChild (win_node, NULL, "slot", NULL);
+ slot_node = xmlNewChild (win_node, NULL, (const xmlChar *) "slot", NULL);
 
 tmp = caja_window_slot_get_location_uri (slot);
- xmlNewProp (slot_node, "location", tmp);
+ xmlNewProp (slot_node, (const xmlChar *) "location", (const xmlChar *) tmp);
 g_free (tmp);
 
 if (slot == active_slot) {
- xmlNewProp (slot_node, "active", "TRUE");
+ xmlNewProp (slot_node, (const xmlChar *) "active", (const xmlChar *) "TRUE");
 }
 }
 
@@ -1515,7 +1515,7 @@ caja_application_get_session_data (CajaApplication *self)
 }
 
 xmlSaveClose(ctx);
- data = g_strndup (buffer->content, buffer->use);
+ data = g_strndup ((const char *) buffer->content, buffer->use);
 xmlBufferFree (buffer);
 
 xmlFreeDoc (doc);
@@ -1564,12 +1564,13 @@ caja_application_load_session (CajaApplication *application)
 
 for (node = root_node->children; node != NULL; node = node->next)
 {
-
- if (g_strcmp0 (node->name, "text") == 0)
+ if (node->name == NULL || *node->name == '\0' ||
+ xmlStrcmp (node->name, (const xmlChar *) "text") == 0)
 {
 continue;
 }
- else if (g_strcmp0 (node->name, "history") == 0)
+
+ if (xmlStrcmp (node->name, (const xmlChar *) "history") == 0)
 {
 xmlNodePtr bookmark_node;
 gboolean emit_change;
@@ -1578,29 +1579,31 @@ caja_application_load_session (CajaApplication *application)
 
 for (bookmark_node = node->children; bookmark_node != NULL; bookmark_node = bookmark_node->next)
 {
- if (g_strcmp0 (bookmark_node->name, "text") == 0)
+ if (bookmark_node->name == NULL || *bookmark_node->name == '\0' ||
+ xmlStrcmp (bookmark_node->name, (const xmlChar *) "text") == 0)
 {
 continue;
 }
- else if (g_strcmp0 (bookmark_node->name, "bookmark") == 0)
+
+ if (xmlStrcmp (bookmark_node->name, (const xmlChar *) "bookmark") == 0)
 {
 xmlChar *name, *icon_str, *uri;
 gboolean has_custom_name;
 GIcon *icon;
 GFile *location;
 
- uri = xmlGetProp (bookmark_node, "uri");
- name = xmlGetProp (bookmark_node, "name");
- has_custom_name = xmlHasProp (bookmark_node, "has_custom_name") ? TRUE : FALSE;
- icon_str = xmlGetProp (bookmark_node, "icon");
+ uri = xmlGetProp (bookmark_node, (const xmlChar *) "uri");
+ name = xmlGetProp (bookmark_node, (const xmlChar *) "name");
+ has_custom_name = xmlHasProp (bookmark_node, (const xmlChar *) "has_custom_name") ? TRUE : FALSE;
+ icon_str = xmlGetProp (bookmark_node, (const xmlChar *) "icon");
 icon = NULL;
 if (icon_str)
 {
- icon = g_icon_new_for_string (icon_str, NULL);
+ icon = g_icon_new_for_string ((const char *) icon_str, NULL);
 }
- location = g_file_new_for_uri (uri);
+ location = g_file_new_for_uri ((const char *) uri);
 
- emit_change |= caja_add_to_history_list_no_notify (location, name, has_custom_name, icon);
+ emit_change |= caja_add_to_history_list_no_notify (location, (const char *) name, has_custom_name, icon);
 
 g_object_unref (location);
 
@@ -1626,7 +1629,7 @@ caja_application_load_session (CajaApplication *application)
 }
 }
 
- else if (g_strcmp0 (node->name, "window") == 0)
+ else if (xmlStrcmp (node->name, (const xmlChar *) "window") == 0)
 
 {
 CajaWindow *window;
@@ -1634,7 +1637,7 @@ caja_application_load_session (CajaApplication *application)
 xmlNodePtr slot_node;
 GFile *location;
 
- type = xmlGetProp (node, "type");
+ type = xmlGetProp (node, (const xmlChar *) "type");
 if (type == NULL)
 {
 g_message ("empty type node while parsing session data");
@@ -1642,7 +1645,7 @@ caja_application_load_session (CajaApplication *application)
 continue;
 }
 
- location_uri = xmlGetProp (node, "location");
+ location_uri = xmlGetProp (node, (const xmlChar *) "location");
 if (location_uri == NULL)
 {
 g_message ("empty location node while parsing session data");
@@ -1651,25 +1654,24 @@ caja_application_load_session (CajaApplication *application)
 continue;
 }
 
- if (g_strcmp0 (type, "navigation") == 0)
+ if (xmlStrcmp (type, (const xmlChar *) "navigation") == 0)
 {
 xmlChar *geometry;
 int i;
 
 window = caja_application_create_navigation_window (application, gdk_screen_get_default ());
- geometry = xmlGetProp (node, "geometry");
- if (geometry != NULL)
+ geometry = xmlGetProp (node, (const xmlChar *) "geometry");
+ if (geometry != NULL && *geometry != '\0')
 {
- eel_gtk_window_set_initial_geometry_from_string
- (GTK_WINDOW (window),
- geometry,
- CAJA_NAVIGATION_WINDOW_MIN_WIDTH,
- CAJA_NAVIGATION_WINDOW_MIN_HEIGHT,
- FALSE);
+ eel_gtk_window_set_initial_geometry_from_string (GTK_WINDOW (window),
+ (const char *) geometry,
+ CAJA_NAVIGATION_WINDOW_MIN_WIDTH,
+ CAJA_NAVIGATION_WINDOW_MIN_HEIGHT,
+ FALSE);
 }
 xmlFree (geometry);
 
- if (xmlHasProp (node, "maximized"))
+ if (xmlHasProp (node, (const xmlChar *) "maximized"))
 {
 gtk_window_maximize (GTK_WINDOW (window));
 }
@@ -1678,7 +1680,7 @@ caja_application_load_session (CajaApplication *application)
 gtk_window_unmaximize (GTK_WINDOW (window));
 }
 
- if (xmlHasProp (node, "sticky"))
+ if (xmlHasProp (node, (const xmlChar *) "sticky"))
 {
 gtk_window_stick (GTK_WINDOW (window));
 }
@@ -1687,7 +1689,7 @@ caja_application_load_session (CajaApplication *application)
 gtk_window_unstick (GTK_WINDOW (window));
 }
 
- if (xmlHasProp (node, "keep-above"))
+ if (xmlHasProp (node, (const xmlChar *) "keep-above"))
 {
 gtk_window_set_keep_above (GTK_WINDOW (window), TRUE);
 }
@@ -1698,11 +1700,11 @@ caja_application_load_session (CajaApplication *application)
 
 for (i = 0, slot_node = node->children; slot_node != NULL; slot_node = slot_node->next)
 {
- if (g_strcmp0 (slot_node->name, "slot") == 0)
+ if (slot_node->name && xmlStrcmp (slot_node->name, (const xmlChar *) "slot") == 0)
 {
 xmlChar *slot_uri;
 
- slot_uri = xmlGetProp (slot_node, "location");
+ slot_uri = xmlGetProp (slot_node, (const xmlChar *) "location");
 if (slot_uri != NULL)
 {
 CajaWindowSlot *slot;
@@ -1716,10 +1718,10 @@ caja_application_load_session (CajaApplication *application)
 slot = caja_window_open_slot (window->details->active_pane, CAJA_WINDOW_OPEN_SLOT_APPEND);
 }
 
- location = g_file_new_for_uri (slot_uri);
+ location = g_file_new_for_uri ((const char *) slot_uri);
 caja_window_slot_open_location (slot, location, FALSE);
 
- if (xmlHasProp (slot_node, "active"))
+ if (xmlHasProp (slot_node, (const xmlChar *) "active"))
 {
 caja_window_set_active_slot (slot->pane->window, slot);
 }
@@ -1733,20 +1735,18 @@ caja_application_load_session (CajaApplication *application)
 if (i == 0)
 {
 /* This may be an old session file */
- location = g_file_new_for_uri (location_uri);
+ location = g_file_new_for_uri ((const char *) location_uri);
 caja_window_slot_open_location (window->details->active_pane->active_slot, location, FALSE);
 g_object_unref (location);
 }
 }
- else if (g_strcmp0 (type, "spatial") == 0)
+ else if (xmlStrcmp (type, (const xmlChar *) "spatial") == 0)
 {
- location = g_file_new_for_uri (location_uri);
+ location = g_file_new_for_uri ((const char *) location_uri);
 window = caja_application_get_spatial_window (application, NULL, NULL,
- location, gdk_screen_get_default (),
- NULL);
-
- caja_window_go_to (window, location);
-
+ location, gdk_screen_get_default (),
+ NULL);
+ caja_window_go_to (window, location);
 g_object_unref (location);
 }
 else
diff --git a/src/caja-property-browser.c b/src/caja-property-browser.c
index c029ad90..2f36a2a2 100644
--- a/src/caja-property-browser.c
+++ b/src/caja-property-browser.c
@@ -903,19 +903,23 @@ write_browser_xml (CajaPropertyBrowser *property_browser,
 static xmlNodePtr
 get_color_category (xmlDocPtr document)
 {
- return eel_xml_get_root_child_by_name_and_property (document, "category", "name", "colors");
+ return eel_xml_get_root_child_by_name_and_property (document,
+ (const xmlChar *) "category",
+ (const xmlChar *) "name",
+ (const xmlChar *) "colors");
 }
 
 /* routines to remove specific category types. First, handle colors */
 
 static void
-remove_color (CajaPropertyBrowser *property_browser, const char* color_name)
+remove_color (CajaPropertyBrowser *property_browser,
+ const xmlChar *color_name)
 {
 /* load the local xml file to remove the color */
 xmlDocPtr document;
 xmlNodePtr cur_node, color_node;
 gboolean match;
- char *name;
+ xmlChar *name;
 
 document = read_browser_xml (property_browser);
 if (document == NULL)
@@ -938,9 +942,8 @@ remove_color (CajaPropertyBrowser *property_browser, const char* color_name)
 continue;
 }
 
- name = xmlGetProp (color_node, "name");
- match = name != NULL
- && strcmp (name, color_name) == 0;
+ name = xmlGetProp (color_node, (const xmlChar *) "name");
+ match = name != NULL && xmlStrcmp (name, color_name) == 0;
 xmlFree (name);
 
 if (match)
@@ -1022,7 +1025,7 @@ caja_property_browser_remove_element (CajaPropertyBrowser *property_browser, Eel
 break;
 case CAJA_PROPERTY_COLOR:
 color_name = eel_labeled_image_get_text (child);
- remove_color (property_browser, color_name);
+ remove_color (property_browser, (const xmlChar *) color_name);
 g_free (color_name);
 break;
 case CAJA_PROPERTY_EMBLEM:
@@ -1360,7 +1363,9 @@ add_new_pattern (CajaPropertyBrowser *property_browser)
 /* here's where we add the passed in color to the file that defines the colors */
 
 static void
-add_color_to_file (CajaPropertyBrowser *property_browser, const char *color_spec, const char *color_name)
+add_color_to_file (CajaPropertyBrowser *property_browser,
+ const xmlChar *color_spec,
+ const xmlChar *color_name)
 {
 xmlNodePtr cur_node, new_color_node, children_node;
 xmlDocPtr document;
@@ -1382,7 +1387,7 @@ add_color_to_file (CajaPropertyBrowser *property_browser, const char *color_spec
 {
 xmlChar *child_color_name;
 
- child_color_name = xmlGetProp (children_node, "name");
+ child_color_name = xmlGetProp (children_node, (const xmlChar *) "name");
 if (xmlStrcmp (color_name, child_color_name) == 0)
 {
 color_name_exists = TRUE;
@@ -1397,10 +1402,10 @@ add_color_to_file (CajaPropertyBrowser *property_browser, const char *color_spec
 /* add a new color node */
 if (!color_name_exists)
 {
- new_color_node = xmlNewChild (cur_node, NULL, "color", NULL);
+ new_color_node = xmlNewChild (cur_node, NULL, (const xmlChar *) "color", NULL);
 xmlNodeSetContent (new_color_node, color_spec);
- xmlSetProp (new_color_node, "local", "1");
- xmlSetProp (new_color_node, "name", color_name);
+ xmlSetProp (new_color_node, (const xmlChar *) "local", (const xmlChar *) "1");
+ xmlSetProp (new_color_node, (const xmlChar *) "name", color_name);
 
 write_browser_xml (property_browser, document);
 }
@@ -1442,7 +1447,7 @@ add_color_to_browser (GtkWidget *widget, gint which_button, gpointer data)
 }
 else
 {
- add_color_to_file (property_browser, color_spec, stripped_color_name);
+ add_color_to_file (property_browser, (const xmlChar *) color_spec, (const xmlChar *) stripped_color_name);
 caja_property_browser_update_contents(property_browser);
 }
 g_free (stripped_color_name);
@@ -1960,7 +1965,7 @@ make_properties_from_directories (CajaPropertyBrowser *property_browser)
 object_pixbuf = gdk_pixbuf_new_from_file (path, NULL);
 }
 g_free (path);
- property_image = labeled_image_new (_("Erase"), object_pixbuf, "erase", PANGO_SCALE_LARGE);
+ property_image = labeled_image_new (_("Erase"), object_pixbuf, (const char *) "erase", PANGO_SCALE_LARGE);
 eel_labeled_image_set_fixed_image_height (EEL_LABELED_IMAGE (property_image), MAX_EMBLEM_HEIGHT);
 
 gtk_container_add (GTK_CONTAINER (image_table), property_image);
@@ -2035,7 +2040,7 @@ make_properties_from_xml_node (CajaPropertyBrowser *property_browser,
 xmlNodePtr child_node;
 GdkPixbuf *pixbuf;
 GtkWidget *new_property;
- char *deleted, *local, *color, *name;
+ xmlChar *deleted, *local, *color, *name;
 
 gboolean local_only = property_browser->details->remove_mode;
 
@@ -2060,8 +2065,8 @@ make_properties_from_xml_node (CajaPropertyBrowser *property_browser,
 /* We used to mark colors that were removed with the "deleted" attribute.
 * To prevent these colors from suddenly showing up now, this legacy remains.
 */
- deleted = xmlGetProp (child_node, "deleted");
- local = xmlGetProp (child_node, "local");
+ deleted = xmlGetProp (child_node, (const xmlChar *) "deleted");
+ local = xmlGetProp (child_node, (const xmlChar *) "local");
 
 if (deleted == NULL && (!local_only || local != NULL))
 {
@@ -2071,13 +2076,18 @@ make_properties_from_xml_node (CajaPropertyBrowser *property_browser,
 }
 
 color = xmlNodeGetContent (child_node);
- name = xmlGetProp (child_node, "name");
+ name = xmlGetProp (child_node, (const xmlChar *) "name");
 
 /* make the image from the color spec */
- pixbuf = make_color_drag_image (property_browser, color, FALSE);
+ pixbuf = make_color_drag_image (property_browser,
+ (const char *) color,
+ FALSE);
 
 /* make the tile from the pixmap and name */
- new_property = labeled_image_new (_(name), pixbuf, color, PANGO_SCALE_LARGE);
+ new_property = labeled_image_new (_((const char *) name),
+ pixbuf,
+ (const char *) color,
+ PANGO_SCALE_LARGE);
 
 gtk_container_add (GTK_CONTAINER (property_browser->details->content_table), new_property);
 gtk_widget_show (new_property);
@@ -2195,7 +2205,7 @@ caja_property_browser_update_contents (CajaPropertyBrowser *property_browser)
 GtkWidget *viewport;
 GtkRadioButton *group;
 gboolean got_categories;
- char *name, *image, *type, *description, *display_name, *path, *mode;
+ xmlChar *name, *image, *type, *description, *display_name, *path, *mode;
 
 /* load the xml document corresponding to the path and selection */
 document = read_browser_xml (property_browser);
@@ -2257,24 +2267,24 @@ caja_property_browser_update_contents (CajaPropertyBrowser *property_browser)
 continue;
 }
 
- if (strcmp (cur_node->name, "category") == 0)
+ if (cur_node->name && xmlStrcmp (cur_node->name, (const xmlChar *) "category") == 0)
 {
- name = xmlGetProp (cur_node, "name");
+ name = xmlGetProp (cur_node, (const xmlChar *) "name");
 
 if (property_browser->details->category != NULL
- && strcmp (property_browser->details->category, name) == 0)
+ && strcmp (property_browser->details->category, (const char *) name) == 0)
 {
- path = xmlGetProp (cur_node, "path");
- mode = xmlGetProp (cur_node, "mode");
- description = xmlGetProp (cur_node, "description");
- type = xmlGetProp (cur_node, "type");
+ path = xmlGetProp (cur_node, (const xmlChar *) "path");
+ mode = xmlGetProp (cur_node, (const xmlChar *) "mode");
+ description = xmlGetProp (cur_node, (const xmlChar *) "description");
+ type = xmlGetProp (cur_node, (const xmlChar *) "type");
 
 make_category (property_browser,
- path,
- mode,
+ (const char *) path,
+ (const char *) mode,
 cur_node,
- _(description));
- caja_property_browser_set_drag_type (property_browser, type);
+ _((const char *) description));
+ caja_property_browser_set_drag_type (property_browser, (const char *) type);
 
 xmlFree (path);
 xmlFree (mode);
@@ -2284,13 +2294,13 @@ caja_property_browser_update_contents (CajaPropertyBrowser *property_browser)
 
 if (!got_categories)
 {
- display_name = xmlGetProp (cur_node, "display_name");
- image = xmlGetProp (cur_node, "image");
+ display_name = xmlGetProp (cur_node, (const xmlChar *) "display_name");
+ image = xmlGetProp (cur_node, (const xmlChar *) "image");
 
 make_category_link (property_browser,
- name,
- _(display_name),
- image,
+ (const char *) name,
+ _((const char *) display_name),
+ (const char *) image,
 &group);
 
 xmlFree (display_name);
author	rbuj <[email protected]>	2021-02-21 10:30:00 +0100
committer	raveit65 <[email protected]>	2022-07-21 19:33:50 +0200
commit	202a9c5ad39758616f73e23aabcbe43aaebea26b (patch)
tree	402b5a9a629f3148c44dab9950f5af8bed066892
parent	76f11ad9fd2c4a46211a115fe16c71dca9756503 (diff)
download	caja-202a9c5ad39758616f73e23aabcbe43aaebea26b.tar.bz2 caja-202a9c5ad39758616f73e23aabcbe43aaebea26b.tar.xz