Commit 3682b8e9 authored by Sven Neumann's avatar Sven Neumann Committed by Sven Neumann

warn if a plug-in attempts to register a menu path for a procedure that it

2006-01-11  Sven Neumann  <sven@gimp.org>

	* tools/pdbgen/pdb/plug_in.pdb (plugin_menu_register): warn if a
	plug-in attempts to register a menu path for a procedure that it
	didn't install.

	* app/pdb/plug_in_cmds.c: regenerated.
parent 244d9ee7
2006-01-11 Sven Neumann <sven@gimp.org>
* tools/pdbgen/pdb/plug_in.pdb (plugin_menu_register): warn if a
plug-in attempts to register a menu path for a procedure that it
didn't install.
* app/pdb/plug_in_cmds.c: regenerated.
2006-01-07 Michael Natterer <mitch@gimp.org> 2006-01-07 Michael Natterer <mitch@gimp.org>
* libgimpwidgets/gimpdialog.c: implement GtkWidget::hide() and set * libgimpwidgets/gimpdialog.c: implement GtkWidget::hide() and set
......
...@@ -412,13 +412,13 @@ plugin_menu_register_invoker (Gimp *gimp, ...@@ -412,13 +412,13 @@ plugin_menu_register_invoker (Gimp *gimp,
} }
else else
{ {
g_message ("Plug-In \"%s\"\n(%s)\n\n" g_message ("Plug-in \"%s\"\n(%s)\n\n"
"attempted to install additional menu_path \"%s\"\n" "attempted to register the menu item \"%s\" "
"for procedure \"%s\".\n" "for procedure \"%s\".\n"
"However the menu_path given in " "The menu label given in gimp_install_procedure() "
"gimp_install_procedure() already contained " "already contained a path. To make this work, "
"a path. To make this work, pass just the menu's " "pass just the menu's label to "
"label to gimp_install_procedure().", "gimp_install_procedure().",
gimp_filename_to_utf8 (gimp->current_plug_in->name), gimp_filename_to_utf8 (gimp->current_plug_in->name),
gimp_filename_to_utf8 (gimp->current_plug_in->prog), gimp_filename_to_utf8 (gimp->current_plug_in->prog),
menu_path, canonical); menu_path, canonical);
...@@ -427,7 +427,18 @@ plugin_menu_register_invoker (Gimp *gimp, ...@@ -427,7 +427,18 @@ plugin_menu_register_invoker (Gimp *gimp,
} }
} }
else else
success = FALSE; {
g_message ("Plug-in \"%s\"\n(%s)\n\n"
"attempted to register the menu item \"%s\" "
"for the procedure \"%s\"\n."
"It has however not installed that procedure. This "
"is not allowed.",
gimp_filename_to_utf8 (gimp->current_plug_in->name),
gimp_filename_to_utf8 (gimp->current_plug_in->prog),
menu_path, canonical);
success = FALSE;
}
g_free (canonical); g_free (canonical);
} }
......
...@@ -248,22 +248,33 @@ HELP ...@@ -248,22 +248,33 @@ HELP
} }
else else
{ {
g_message ("Plug-In \"%s\"\n(%s)\n\n" g_message ("Plug-in \"%s\"\n(%s)\n\n"
"attempted to install additional menu_path \"%s\"\n" "attempted to register the menu item \"%s\" "
"for procedure \"%s\".\n" "for procedure \"%s\".\n"
"However the menu_path given in " "The menu label given in gimp_install_procedure() "
"gimp_install_procedure() already contained " "already contained a path. To make this work, "
"a path. To make this work, pass just the menu's " "pass just the menu's label to "
"label to gimp_install_procedure().", "gimp_install_procedure().",
gimp_filename_to_utf8 (gimp->current_plug_in->name), gimp_filename_to_utf8 (gimp->current_plug_in->name),
gimp_filename_to_utf8 (gimp->current_plug_in->prog), gimp_filename_to_utf8 (gimp->current_plug_in->prog),
menu_path, canonical); menu_path, canonical);
success = FALSE; success = FALSE;
} }
} }
else else
success = FALSE; {
g_message ("Plug-in \"%s\"\n(%s)\n\n"
"attempted to register the menu item \"%s\" "
"for the procedure \"%s\"\n."
"It has however not installed that procedure. This "
"is not allowed.",
gimp_filename_to_utf8 (gimp->current_plug_in->name),
gimp_filename_to_utf8 (gimp->current_plug_in->prog),
menu_path, canonical);
success = FALSE;
}
g_free (canonical); g_free (canonical);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment