AuroraMiddleware/gtk2
1
0
Fork 0
forked from AuroraMiddleware/gtk
Code Issues Pull Requests Projects Releases Wiki Activity

Menu|Item: Fix FIXME re non-const interned strings

Browse Source 
These are members of the private struct, so it hurts no one to fix this.
This commit is contained in:
Daniel Boles 2018-04-17 20:03:07 +01:00
parent 4fae962a03
commit d13e78831b
4 changed files with 5 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
gtk/gtkmenu.c
View File

@ -2388,8 +2388,7 @@ gtk_menu_set_accel_path (GtkMenu *menu,
if (accel_path)
g_return_if_fail (accel_path[0] == '<' && strchr (accel_path, '/')); /* simplistic check */
/* FIXME: accel_path should be defined as const gchar* */
priv->accel_path = (gchar*)g_intern_string (accel_path);
priv->accel_path = g_intern_string (accel_path);
if (priv->accel_path)
_gtk_menu_refresh_accel_paths (menu, FALSE);
}

4
gtk/gtkmenuitem.c
View File

@ -1693,7 +1693,7 @@ _gtk_menu_item_refresh_accel_path (GtkMenuItem *menu_item,
if (postfix)
{
new_path = g_strconcat (prefix, "/", postfix, NULL);
path = priv->accel_path = (char*)g_intern_string (new_path);
path = priv->accel_path = g_intern_string (new_path);
g_free (new_path);
}
}
@ -1745,7 +1745,7 @@ gtk_menu_item_set_accel_path (GtkMenuItem *menu_item,
widget = GTK_WIDGET (menu_item);
/* store new path */
priv->accel_path = (char*)g_intern_string (accel_path);
priv->accel_path = g_intern_string (accel_path);
/* forget accelerators associated with old path */
gtk_widget_set_accel_path (widget, NULL, NULL);

2
gtk/gtkmenuitemprivate.h
View File

@ -34,7 +34,7 @@ struct _GtkMenuItemPrivate
guint timer;
gchar *accel_path;
const char *accel_path;
GtkActionHelper *action_helper;

2
gtk/gtkmenuprivate.h
View File

@ -45,7 +45,7 @@ struct _GtkMenuPrivate
GtkWidget *old_active_menu_item;
GtkAccelGroup *accel_group;
gchar *accel_path;
const char *accel_path;
GtkMenuPositionFunc position_func;
gpointer position_func_data;