- move the common code to a function in the e_modules file.

SVN revision: 16851

AUTHORS

@@ -3,3 +3,4 @@ Ibukun Olumuyiwa <ibukun@computer.org>
 Sebastian Dransfeld <sebastid@tango.flipp.net>
 HandyAndE (Andrew Williams) <andy@handyande.co.uk>
 CodeWarrior (Hisham Mardam Bey) <hisham@hisham.cc>
+dj2 (dan sinclair) <dj2@everburning.com>

src/bin/e_module.c

@@ -456,3 +456,22 @@ _e_module_control_menu_enabled(void *data, E_Menu *m, E_Menu_Item *mi)
      }
    e_menu_item_toggle_set(mi, e_module_enabled_get(mod));
 }
+
+void
+e_module_dialog_show(char *title, char *body)
+{
+   E_Dialog *dia;
+
+   dia = e_dialog_new (e_container_current_get (e_manager_current_get ()));
+   if (!dia) return;
+
+   e_dialog_title_set (dia, title);
+   e_dialog_icon_set (dia, "enlightenment/e", 64);
+   e_dialog_text_set (dia, body);
+   e_dialog_button_add (dia, _("Ok"), NULL, NULL, NULL);
+   e_win_centered_set (dia->win, 1);
+   e_dialog_show (dia);
+}
+
+
+

src/bin/e_module.h

@@ -65,6 +65,7 @@ EAPI int          e_module_save_all(void);
 EAPI E_Module    *e_module_find(char *name);
 EAPI Evas_List   *e_module_list(void);
 EAPI E_Menu      *e_module_menu_new(void);
+EAPI void         e_module_dialog_show(char *title, char *body);
 
 #endif
 #endif

src/modules/battery/e_mod_main.c

@@ -73,23 +73,13 @@ e_modapi_init(E_Module *m)
    /* check module api version */
    if (m->api->version < E_MODULE_API_VERSION)
      {
-	E_Dialog *dia;
 	char buf[4096];
-
+	snprintf(buf, sizeof(buf), _("Module API Error<br>Error initializing Module: %s<br>"
-	dia = e_dialog_new(e_container_current_get(e_manager_current_get()));
-	if (!dia) return NULL;
-
-	snprintf(buf, sizeof(buf), _("Module API Error<br>Error initializing Module: Battery<br>"
 				     "It requires a minimum module API version of: %i.<br>"
 				     "The module API advertized by Enlightenment is: %i.<br>"), 
-				   E_MODULE_API_VERSION, m->api->version);
+				   _("Battery"), E_MODULE_API_VERSION, m->api->version);
 
-	e_dialog_title_set(dia, "Enlightenment Battery Module");
+	e_module_dialog_show(_("Enlightenment Battery Module"), buf);
-	e_dialog_icon_set(dia, "enlightenment/e", 64);
-	e_dialog_text_set(dia, buf);
-	e_dialog_button_add(dia, _("Ok"), NULL, NULL, NULL);
-	e_win_centered_set(dia->win, 1);
-	e_dialog_show(dia);
 	return NULL;
      }
    /* actually init battery */
@@ -136,20 +126,12 @@ e_modapi_info(E_Module *m)
 int
 e_modapi_about(E_Module *m)
 {
-   E_Dialog *dia;
+   e_module_dialog_show(_("Enlightenment Battery Module"),
-
+		       _("A basic battery meter that uses either"
-   dia = e_dialog_new(e_container_current_get(e_manager_current_get()));
+		 	 "<hilight>ACPI</hilight> or <hilight>APM</hilight><br>"
-   if (!dia) return 0;
+			 "on Linux to monitor your battery and AC power adaptor<br>"
-   e_dialog_title_set(dia, "Enlightenment Battery Module");
+			 "status. This will work under Linux and FreeBSD and is only<br>"
-   e_dialog_icon_set(dia, "enlightenment/e", 64);
+			 "as accurate as your BIOS or kernel drivers."));
-   e_dialog_text_set(dia, _("A basic battery meter that uses either"
-			    "<hilight>ACPI</hilight> or <hilight>APM</hilight><br>"
-			    "on Linux to monitor your battery and AC power adaptor<br>"
-			    "status. This will work under Linux and FreeBSD and is only<br>"
-			    "as accurate as your BIOS or kernel drivers."));
-   e_dialog_button_add(dia, _("Ok"), NULL, NULL, NULL);
-   e_win_centered_set(dia->win, 1);
-   e_dialog_show(dia);
    return 1;
 }
 

src/modules/clock/e_mod_main.c

@@ -46,23 +46,13 @@ e_modapi_init(E_Module *module)
    /* check module api version */
    if (module->api->version < E_MODULE_API_VERSION)
      {
-	E_Dialog *dia;
 	char buf[4096];
-
+	snprintf(buf, sizeof(buf), _("Module API Error<br>Error initializing Module: %s<br>"
-	dia = e_dialog_new(e_container_current_get(e_manager_current_get()));
-	if (!dia) return NULL;
-
-	snprintf(buf, sizeof(buf), _("Module API Error<br>Error initializing Module: Clock<br>"
 				     "It requires a minimum module API version of: %i.<br>"
 				     "The module API advertized by Enlightenment is: %i.<br>"), 
-				   E_MODULE_API_VERSION, module->api->version);
+				   _("Clock"), E_MODULE_API_VERSION, module->api->version);
 
-	e_dialog_title_set(dia, "Enlightenment Clock Module");
+	e_module_dialog_show(_("Enlightenment Clock Module"), buf);
-	e_dialog_icon_set(dia, "enlightenment/e", 64);
-	e_dialog_text_set(dia, buf);
-	e_dialog_button_add(dia, _("Ok"), NULL, NULL, NULL);
-	e_win_centered_set(dia->win, 1);
-	e_dialog_show(dia);
 	return NULL;
      }
 
@@ -111,16 +101,8 @@ e_modapi_info(E_Module *module)
 int
 e_modapi_about(E_Module *module)
 {
-   E_Dialog *dia;
+   e_module_dialog_show(_("Enlightenment Clock Module"), 
-
+		       _("A simple module to give E17 a clock."));
-   dia = e_dialog_new(e_container_current_get(e_manager_current_get()));
-   if (!dia) return 0;
-   e_dialog_title_set(dia, "Enlightenment Clock Module");
-   e_dialog_icon_set(dia, "enlightenment/e", 64);
-   e_dialog_text_set(dia, _("A simple module to give E17 a clock."));
-   e_dialog_button_add(dia, _("Ok"), NULL, NULL, NULL);
-   e_win_centered_set(dia->win, 1);
-   e_dialog_show(dia);
    return 1;
 }
 

src/modules/cpufreq/e_mod_main.c

@@ -64,23 +64,13 @@ e_modapi_init(E_Module *module)
    
    if (module->api->version < E_MODULE_API_VERSION)
      {
-	E_Dialog *dia;
 	char buf[4096];
-
+	snprintf(buf, sizeof(buf), _("Module API Error<br>Error initializing Module: %s<br>"
-	dia = e_dialog_new(e_container_current_get(e_manager_current_get()));
-	if (!dia) return NULL;
-
-	snprintf(buf, sizeof(buf), _("Module API Error<br>Error initializing Module: Cpufreq<br>"
 				     "It requires a minimum module API version of: %i.<br>"
 				     "The module API advertized by Enlightenment is: %i.<br>"), 
-				   E_MODULE_API_VERSION, module->api->version);
+				   _("Cpufreq"), E_MODULE_API_VERSION, module->api->version);
 
-	e_dialog_title_set(dia, "Enlightenment Cpufreq Module");
+	e_module_dialog_show(_("Enlightenment Cpufreq Module"), buf);
-	e_dialog_icon_set(dia, "enlightenment/e", 64);
-	e_dialog_text_set(dia, buf);
-	e_dialog_button_add(dia, _("Ok"), NULL, NULL, NULL);
-	e_win_centered_set(dia->win, 1);
-	e_dialog_show(dia);
 	return NULL;
      }
 
@@ -128,17 +118,9 @@ e_modapi_info(E_Module *module)
 int
 e_modapi_about(E_Module *module)
 {
-   E_Dialog *dia;
+   e_module_dialog_show(_("CPU Frequency Controller Module"), 
-
+			_("A simple module to control the frequency of the system CPU.<br>"
-   dia = e_dialog_new(e_container_current_get(e_manager_current_get()));
+			  "This is especially useful to save power on laptops."));
-   if (!dia) return 0;
-   e_dialog_title_set(dia, "CPU Frequency Controller Module");
-   e_dialog_icon_set(dia, "enlightenment/e", 64);
-   e_dialog_text_set(dia, _("A simple module to control the frequency of the system CPU.<br>"
-			    "This is especially useful to save power on laptops."));
-   e_dialog_button_add(dia, _("Ok"), NULL, NULL, NULL);
-   e_win_centered_set(dia->win, 1);
-   e_dialog_show(dia);
    return 1;
 }
 

src/modules/dropshadow/e_mod_main.c

@@ -97,23 +97,13 @@ e_modapi_init(E_Module *m)
    
    if (m->api->version < E_MODULE_API_VERSION)
      {
-	E_Dialog *dia;
 	char buf[4096];
-
+	snprintf(buf, sizeof(buf), _("Module API Error<br>Error initializing Module: %s<br>"
-	dia = e_dialog_new(e_container_current_get(e_manager_current_get()));
-	if (!dia) return NULL;
-
-	snprintf(buf, sizeof(buf), _("Module API Error<br>Error initializing Module: Dropshadow<br>"
 		 		     "It requires a minimum module API version of: %i.<br>"
 		 		     "The module API advertized by Enlightenment is: %i.<br>"), 
-		 		   E_MODULE_API_VERSION, m->api->version);
+		 		   _("Dropshadow"), E_MODULE_API_VERSION, m->api->version);
 
-	e_dialog_title_set(dia, "Enlightenment Dropshadow Module");
+	e_module_dialog_show(_("Enlightenment Dropshadow Module"), buf);
-	e_dialog_icon_set(dia, "enlightenment/e", 64);
-	e_dialog_text_set(dia, buf);
-	e_dialog_button_add(dia, _("Ok"), NULL, NULL, NULL);
-	e_win_centered_set(dia->win, 1);
-	e_dialog_show(dia);
 	return NULL;
      }
    ds = _ds_init(m);
@@ -200,18 +190,10 @@ e_modapi_info(E_Module *m)
 int
 e_modapi_about(E_Module *m)
 {
-   E_Dialog *dia;
+   e_module_dialog_show(_("Enlightenment Dropshadow Module"),
-
+			_("This is the dropshadow module that allows dropshadows to be cast<br>"
-   dia = e_dialog_new(e_container_current_get(e_manager_current_get()));
+			  "on the desktop background - without special X-Server extensions<br>"
-   if (!dia) return 0;
+			  "or hardware acceleration."));
-   e_dialog_title_set(dia, "Enlightenment Dropshadow Module");
-   e_dialog_icon_set(dia, "enlightenment/e", 64);
-   e_dialog_text_set(dia, _("This is the dropshadow module that allows dropshadows to be cast<br>"
-			    "on the desktop background - without special X-Server extensions<br>"
-			    "or hardware acceleration."));
-   e_dialog_button_add(dia, _("Ok"), NULL, NULL, NULL);
-   e_win_centered_set(dia->win, 1);
-   e_dialog_show(dia);
    return 1;
 }
 

src/modules/ibar/e_mod_main.c

@@ -118,23 +118,13 @@ e_modapi_init(E_Module *m)
    /* check module api version */
    if (m->api->version < E_MODULE_API_VERSION)
      {
-	E_Dialog *dia;
 	char buf[4096];
-
+	snprintf(buf, sizeof(buf), _("Module API Error<br>Error initializing Module: %s<br>"
-	dia = e_dialog_new(e_container_current_get(e_manager_current_get()));
-	if (!dia) return NULL;
-
-	snprintf(buf, sizeof(buf), _("Module API Error<br>Error initializing Module: IBar<br>"
 				     "It requires a minimum module API version of: %i.<br>"
 				     "The module API advertized by Enlightenment is: %i.<br>"), 
-				   E_MODULE_API_VERSION, m->api->version);
+				   _("IBar"), E_MODULE_API_VERSION, m->api->version);
 
-	e_dialog_title_set(dia, "Enlightenment IBar Module");
+	e_module_dialog_show(_("Enlightenment IBar Module"), buf);
-	e_dialog_icon_set(dia, "enlightenment/e", 64);
-	e_dialog_text_set(dia, buf);
-	e_dialog_button_add(dia, _("Ok"), NULL, NULL, NULL);
-	e_win_centered_set(dia->win, 1);
-	e_dialog_show(dia);
 	return NULL;
      }
    /* actually init ibar */
@@ -181,19 +171,11 @@ e_modapi_info(E_Module *m)
 int
 e_modapi_about(E_Module *m)
 {
-   E_Dialog *dia;
+   e_module_dialog_show(_("Enlightenment IBar Module"), 
-
+			_("This is the IBar Application Launcher bar module for Enlightenment.<br>"
-   dia = e_dialog_new(e_container_current_get(e_manager_current_get()));
+			  "It is a first example module and is being used to flesh out several<br>"
-   if (!dia) return 0;
+			  "interfaces in Enlightenment 0.17.0. It is under heavy development,<br>"
-   e_dialog_title_set(dia, "Enlightenment IBar Module");
+			  "so expect it to <hilight>break often</hilight> and change as it improves."));
-   e_dialog_icon_set(dia, "enlightenment/e", 64);
-   e_dialog_text_set(dia, _("This is the IBar Application Launcher bar module for Enlightenment.<br>"
-			    "It is a first example module and is being used to flesh out several<br>"
-			    "interfaces in Enlightenment 0.17.0. It is under heavy development,<br>"
-			    "so expect it to <hilight>break often</hilight> and change as it improves."));
-   e_dialog_button_add(dia, _("Ok"), NULL, NULL, NULL);
-   e_win_centered_set(dia->win, 1);
-   e_dialog_show(dia);
    return 1;
 }
 

src/modules/ibox/e_mod_main.c

@@ -100,23 +100,13 @@ e_modapi_init(E_Module *m)
    /* check module api version */
    if (m->api->version < E_MODULE_API_VERSION)
      {
-	E_Dialog *dia;
 	char buf[4096];
-
+	snprintf(buf, sizeof(buf), _("Module API Error<br>Error initializing Module: %s<br>"
-	dia = e_dialog_new(e_container_current_get(e_manager_current_get()));
-	if (!dia) return NULL;
-
-	snprintf(buf, sizeof(buf), _("Module API Error<br>Error initializing Module: IBox<br>"
 				     "It requires a minimum module API version of: %i.<br>"
 				     "The module API advertized by Enlightenment is: %i.<br>"), 
-				   E_MODULE_API_VERSION, m->api->version);
+				   _("IBox"), E_MODULE_API_VERSION, m->api->version);
 
-	e_dialog_title_set(dia, "Enlightenment IBox Module");
+	e_module_dialog_show(_("Enlightenment IBox Module"), buf);
-	e_dialog_icon_set(dia, "enlightenment/e", 64);
-	e_dialog_text_set(dia, buf);
-	e_dialog_button_add(dia, _("Ok"), NULL, NULL, NULL);
-	e_win_centered_set(dia->win, 1);
-	e_dialog_show(dia);
 	return NULL;
      }
    /* actually init ibox */
@@ -163,17 +153,9 @@ e_modapi_info(E_Module *m)
 int
 e_modapi_about(E_Module *m)
 {
-   E_Dialog *dia;
+   e_module_dialog_show(_("Enlightenment IBox Module"), 
-
+			_("This is the IBox Iconified Application module for Enlightenment.<br>"
-   dia = e_dialog_new(e_container_current_get(e_manager_current_get()));
+			  "It will hold minimized applications"));
-   if (!dia) return 0;
-   e_dialog_title_set(dia, "Enlightenment IBox Module");
-   e_dialog_icon_set(dia, "enlightenment/e", 64);
-   e_dialog_text_set(dia, _("This is the IBox Iconified Application module for Enlightenment.<br>"
-			    "It will hold minimized applications"));
-   e_dialog_button_add(dia, _("Ok"), NULL, NULL, NULL);
-   e_win_centered_set(dia->win, 1);
-   e_dialog_show(dia);
    return 1;
 }
 

src/modules/pager/e_mod_main.c

@@ -112,23 +112,13 @@ e_modapi_init(E_Module *module)
    /* check module api version */
    if (module->api->version < E_MODULE_API_VERSION)
      {
-	E_Dialog *dia;
 	char buf[4096];
-
+	snprintf(buf, sizeof(buf), _("Module API Error<br>Error initializing Module: %s<br>"
-	dia = e_dialog_new(e_container_current_get(e_manager_current_get()));
-	if (!dia) return NULL;
-
-	snprintf(buf, sizeof(buf), _("Module API Error<br>Error initializing Module: Pager<br>"
 				     "It requires a minimum module API version of: %i.<br>"
 				     "The module API advertized by Enlightenment is: %i.<br>"), 
-				   E_MODULE_API_VERSION, module->api->version);
+				   _("Pager"), E_MODULE_API_VERSION, module->api->version);
 
-	e_dialog_title_set(dia, "Enlightenment Pager Module");
+	e_module_dialog_show(_("Enlightenment Pager Module"), buf);
-	e_dialog_icon_set(dia, "enlightenment/e", 64);
-	e_dialog_text_set(dia, buf);
-	e_dialog_button_add(dia, _("Ok"), NULL, NULL, NULL);
-	e_win_centered_set(dia->win, 1);
-	e_dialog_show(dia);
 	return NULL;
      }
    /* actually init pager */
@@ -178,16 +168,8 @@ e_modapi_info(E_Module *module)
 int
 e_modapi_about(E_Module *module)
 {
-   E_Dialog *dia;
+   e_module_dialog_show(_("Enlightenment Pager Module"),
-
+			_("A pager module to navigate virtual desktops."));
-   dia = e_dialog_new(e_container_current_get(e_manager_current_get()));
-   if (!dia) return 0;
-   e_dialog_title_set(dia, "Enlightenment Pager Module");
-   e_dialog_icon_set(dia, "enlightenment/e", 64);
-   e_dialog_text_set(dia, _("A pager module to navigate virtual desktops."));
-   e_dialog_button_add(dia, _("Ok"), NULL, NULL, NULL);
-   e_win_centered_set(dia->win, 1);
-   e_dialog_show(dia);
    return 1;
 }
 

src/modules/randr/e_mod_main.c

@@ -34,23 +34,13 @@ e_modapi_init(E_Module *m)
    
    if (m->api->version < E_MODULE_API_VERSION)
      {
-	E_Dialog *dia;
 	char buf[4096];
-
+	snprintf(buf, sizeof(buf), _("Module API Error<br>Error initializing Module: %s<br>"
-	dia = e_dialog_new(e_container_current_get(e_manager_current_get()));
-	if (!dia) return NULL;
-
-	snprintf(buf, sizeof(buf), _("Module API Error<br>Error initializing Module: Randr<br>"
 				     "It requires a minimum module API version of: %i.<br>"
 				     "The module API advertized by Enlightenment is: %i.<br>"), 
-				   E_MODULE_API_VERSION, m->api->version);
+				   _("RandR"), E_MODULE_API_VERSION, m->api->version);
 
-	e_dialog_title_set(dia, "Enlightenment Randr Module");
+	e_module_dialog_show(_("Enlightenment Randr Module"), buf);
-	e_dialog_icon_set(dia, "enlightenment/e", 64);
-	e_dialog_text_set(dia, buf);
-	e_dialog_button_add(dia, _("Ok"), NULL, NULL, NULL);
-	e_win_centered_set(dia->win, 1);
-	e_dialog_show(dia);
 	return NULL;
    }
    
@@ -101,16 +91,8 @@ e_modapi_info(E_Module *m)
 int
 e_modapi_about(E_Module *m)
 {
-   E_Dialog *dia;
+   e_module_dialog_show(_("Enlightenment Randr Module"), 
-
+			_("Module to change screen resolution for E17"));
-   dia = e_dialog_new(e_container_current_get(e_manager_current_get()));
-   if (!dia) return 0;
-   e_dialog_title_set(dia, "Enlightenment Randr Module");
-   e_dialog_icon_set(dia, "enlightenment/e", 64);
-   e_dialog_text_set(dia, _("Module to change screen resolution for E17"));
-   e_dialog_button_add(dia, _("Ok"), NULL, NULL, NULL);
-   e_win_centered_set(dia->win, 1);
-   e_dialog_show(dia);
    return 1;
 }
 

src/modules/start/e_mod_main.c

@@ -29,23 +29,13 @@ e_modapi_init(E_Module *m)
    
    if (m->api->version < E_MODULE_API_VERSION)
      {
-	E_Dialog *dia;
 	char buf[4096];
-
+	snprintf(buf, sizeof(buf), _("Module API Error<br>Error initializing Module: %s<br>"
-	dia = e_dialog_new(e_container_current_get(e_manager_current_get()));
-	if (!dia) return NULL;
-
-	snprintf(buf, sizeof(buf), _("Module API Error<br>Error initializing Module: Start<br>"
 				     "It requires a minimum module API version of: %i.<br>"
 				     "The module API advertized by Enlightenment is: %i.<br>"), 
-				   E_MODULE_API_VERSION, m->api->version);
+				   _("Start"), E_MODULE_API_VERSION, m->api->version);
 
-	e_dialog_title_set(dia, "Enlightenment Start Module");
+	e_module_dialog_show(_("Enlightenment Start Module"), buf);
-	e_dialog_icon_set(dia, "enlightenment/e", 64);
-	e_dialog_text_set(dia, buf);
-	e_dialog_button_add(dia, _("Ok"), NULL, NULL, NULL);
-	e_win_centered_set(dia->win, 1);
-	e_dialog_show(dia);
 	return NULL;
      }
    
@@ -92,16 +82,8 @@ e_modapi_info(E_Module *m)
 int
 e_modapi_about(E_Module *m)
 {
-   E_Dialog *dia;
+   e_module_dialog_show(_("Enlightenment Start Module"),
-
+			_("Experimental Button module for E17"));
-   dia = e_dialog_new(e_container_current_get(e_manager_current_get()));
-   if (!dia) return 0;
-   e_dialog_title_set(dia, "Enlightenment Start Module");
-   e_dialog_icon_set(dia, "enlightenment/e", 64);
-   e_dialog_text_set(dia, _("Experimental Button module for E17"));
-   e_dialog_button_add(dia, _("Ok"), NULL, NULL, NULL);
-   e_win_centered_set(dia->win, 1);
-   e_dialog_show(dia);
    return 1;
 }
 

src/modules/temperature/e_mod_main.c

@@ -46,23 +46,13 @@ e_modapi_init(E_Module *m)
    /* check module api version */
    if (m->api->version < E_MODULE_API_VERSION)
      {
-	E_Dialog *dia;
 	char buf[4096];
-
+	snprintf(buf, sizeof(buf), _("Module API Error<br>Error initializing Module: %s<br>"
-	dia = e_dialog_new(e_container_current_get(e_manager_current_get()));
-	if (!dia) return NULL;
-
-	snprintf(buf, sizeof(buf), _("Module API Error<br>Error initializing Module: Temperature<br>"
 				     "It requires a minimum module API version of: %i.<br>"
 				     "The module API advertized by Enlightenment is: %i.<br>"), 
-				   E_MODULE_API_VERSION, m->api->version);
+				   _("Temperature"), E_MODULE_API_VERSION, m->api->version);
 
-	e_dialog_title_set(dia, "Enlightenment Temperature Module");
+	e_module_dialog_show(_("Enlightenment Temperature Module"), buf);
-	e_dialog_icon_set(dia, "enlightenment/e", 64);
-	e_dialog_text_set(dia, buf);
-	e_dialog_button_add(dia, _("Ok"), NULL, NULL, NULL);
-	e_win_centered_set(dia->win, 1);
-	e_dialog_show(dia);
 	return NULL;
      }
    /* actually init temperature */
@@ -109,18 +99,10 @@ e_modapi_info(E_Module *m)
 int
 e_modapi_about(E_Module *m)
 {
-   E_Dialog *dia;
+   e_module_dialog_show(_("Enlightenment Temperature Module"),
-
+			_("A module to measure the <hilight>ACPI Thermal sensor</hilight> on Linux.<br>"
-   dia = e_dialog_new(e_container_current_get(e_manager_current_get()));
+			  "It is especially useful for modern Laptops with high speed<br>"
-   if (!dia) return 0;
+			  "CPUs that generate a lot of heat."));
-   e_dialog_title_set(dia, "Enlightenment Temperature Module");
-   e_dialog_icon_set(dia, "enlightenment/e", 64);
-   e_dialog_text_set(dia, _("A module to measure the <hilight>ACPI Thermal sensor</hilight> on Linux.<br>"
-			    "It is especially useful for modern Laptops with high speed<br>"
-			    "CPUs that generate a lot of heat."));
-   e_dialog_button_add(dia, _("Ok"), NULL, NULL, NULL);
-   e_win_centered_set(dia->win, 1);
-   e_dialog_show(dia);
    return 1;
 }
 

src/modules/test/e_mod_main.c

@@ -9,23 +9,14 @@ e_modapi_init(E_Module *m)
 {
    if (m->api->version < E_MODULE_API_VERSION)
      {
-	E_Dialog *dia;
 	char buf[4096];
 
-	dia = e_dialog_new(e_container_current_get(e_manager_current_get()));
+	snprintf(buf, sizeof(buf), _("Module API Error<br>Error initializing Module: %s<br>"
-	if (!dia) return NULL;
-
-	snprintf(buf, sizeof(buf), _("Module API Error<br>Error initializing Module: Test<br>"
 				     "It requires a minimum module API version of: %i.<br>"
 				     "The module API advertized by Enlightenment is: %i.<br>"), 
-				   E_MODULE_API_VERSION, m->api->version);
+				   _("Test"), E_MODULE_API_VERSION, m->api->version);
 
-	e_dialog_title_set(dia, "Enlightenment Test Module");
+	e_module_dialog_show(_("Enlightenment Test Module"), buf);
-	e_dialog_icon_set(dia, "enlightenment/e", 64);
-	e_dialog_text_set(dia, buf);
-	e_dialog_button_add(dia, _("Ok"), NULL, NULL, NULL);
-	e_win_centered_set(dia->win, 1);
-	e_dialog_show(dia);
 	return NULL;
      }
      {
@@ -76,17 +67,9 @@ e_modapi_info(E_Module *m)
 int
 e_modapi_about(E_Module *m)
 {
-   E_Dialog *dia;
+   e_module_dialog_show(_("Enlightenment Test Module"),
-
+			_("This module is VERY simple and is only used to test the basic<br>"
-   dia = e_dialog_new(e_container_current_get(e_manager_current_get()));
+			  "interface of the Enlightenment 0.17.0 module system. Please<br>"
-   if (!dia) return 0;
+			  "ignore this module unless you are working on the module system."));
-   e_dialog_title_set(dia, "Enlightenment Test Module");
-   e_dialog_icon_set(dia, "enlightenment/e", 64);
-   e_dialog_text_set(dia, _("This module is VERY simple and is only used to test the basic<br>"
-			    "interface of the Enlightenment 0.17.0 module system. Please<br>"
-			    "ignore this module unless you are working on the module system."));
-   e_dialog_button_add(dia, _("Ok"), NULL, NULL, NULL);
-   e_win_centered_set(dia->win, 1);
-   e_dialog_show(dia);
    return 1;
 }