callisto-debian/gimp
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

app: A '|' in identifiers is not a special case

Browse Source
This commit is contained in:
Martin Nordholts
2012-01-10 20:51:08 +01:00
parent 6335190ff7
commit 40d98b79cb
1 changed files with 16 additions and 30 deletions
Download Patch File Download Diff File Expand all files Collapse all files

20
app/widgets/gimpdialogfactory.c
View File

@ -734,6 +734,8 @@ gimp_dialog_factory_dialog_raise (GimpDialogFactory *factory,
gint view_size)
{
GtkWidget *dialog;
gchar **ids;
gint i;
g_return_val_if_fail (GIMP_IS_DIALOG_FACTORY (factory), NULL);
g_return_val_if_fail (GDK_IS_SCREEN (screen), NULL);
@ -742,11 +744,7 @@ gimp_dialog_factory_dialog_raise (GimpDialogFactory *factory,
/* If the identifier is a list, try to find a matching dialog and
* raise it. If there's no match, use the first list item.
*/
if (strchr (identifiers, '|'))
{
gchar **ids = g_strsplit (identifiers, "|", 0);
gint i;
ids = g_strsplit (identifiers, "|", 0);
for (i = 0; ids[i]; i++)
{
if (gimp_dialog_factory_find_widget (factory, ids[i]))
@ -762,18 +760,6 @@ gimp_dialog_factory_dialog_raise (GimpDialogFactory *factory,
TRUE,
TRUE);
g_strfreev (ids);
}
else
{
dialog = gimp_dialog_factory_dialog_new_internal (factory,
screen,
NULL,
NULL,
identifiers,
view_size,
TRUE,
TRUE);
}
return dialog;
}