kwo pushed a commit to branch master. http://git.enlightenment.org/e16/e16-epplets.git/commit/?id=b436df8d7a6355a7c1f5dcded8c5bf810abb3d03
commit b436df8d7a6355a7c1f5dcded8c5bf810abb3d03 Author: Kim Woelders <k...@woelders.dk> Date: Thu Aug 26 21:54:45 2021 +0200 Centralize setting up atexit(Epplet_cleanup) --- api/epplet.c | 2 ++ epplets/ConfigTestEpplet.c | 1 - epplets/E-Areas.c | 1 - epplets/E-Bandwidth.c | 1 - epplets/E-Biff.c | 1 - epplets/E-Clock.c | 1 - epplets/E-Cpu.c | 2 -- epplets/E-Disk.c | 1 - epplets/E-Exec.c | 2 -- epplets/E-Load.c | 1 - epplets/E-LoadMeter.c | 2 -- epplets/E-Magic.c | 1 - epplets/E-Mixer.c | 2 -- epplets/E-MoonClock.c | 2 -- epplets/E-Mountbox.c | 1 - epplets/E-Net.c | 2 -- epplets/E-NetFlame.c | 2 -- epplets/E-NetGraph.c | 1 - epplets/E-OpenGL-Demo.c | 2 -- epplets/E-Pants.c | 2 -- epplets/E-Pinger.c | 2 -- epplets/E-PlayCD.c | 2 -- epplets/E-Power.c | 1 - epplets/E-SD.c | 10 +--------- epplets/E-ScreenSave.c | 9 +-------- epplets/E-ScreenShoot.c | 10 +--------- epplets/E-Slides.c | 1 - epplets/E-Sys.c | 2 -- epplets/E-Time.c | 2 -- epplets/E-Toolbox.c | 1 - epplets/E-UrlWatch.c | 10 +--------- epplets/E-Wireless.c | 2 -- epplets/E-Xss.c | 2 -- epplets/Emix.c | 1 - epplets/TestEpplet.c | 1 - 35 files changed, 6 insertions(+), 80 deletions(-) diff --git a/api/epplet.c b/api/epplet.c index bdcd146..8101264 100644 --- a/api/epplet.c +++ b/api/epplet.c @@ -560,6 +560,8 @@ Epplet_Init(const char *name, const char *version, const char *info, sa.sa_flags = SA_RESTART; sigemptyset(&sa.sa_mask); sigaction(SIGCHLD, &sa, (struct sigaction *)0); + + atexit(Epplet_cleanup); } static Window diff --git a/epplets/ConfigTestEpplet.c b/epplets/ConfigTestEpplet.c index 35ce297..2f76c79 100644 --- a/epplets/ConfigTestEpplet.c +++ b/epplets/ConfigTestEpplet.c @@ -6,7 +6,6 @@ cb_close(void *data __UNUSED__) { Epplet_unremember(); Esync(); - Epplet_cleanup(); exit(0); } diff --git a/epplets/E-Areas.c b/epplets/E-Areas.c index a800b59..732f439 100644 --- a/epplets/E-Areas.c +++ b/epplets/E-Areas.c @@ -77,7 +77,6 @@ main(int argc, char **argv) { Epplet_gadget b_up, b_down, b_left, b_right; - atexit(Epplet_cleanup); Epplet_Init("E-Areas", "0.1", "A desktop areas navigator; John Slee <j...@chirp.com.au>", 3, 3, argc, argv, 0); diff --git a/epplets/E-Bandwidth.c b/epplets/E-Bandwidth.c index d5b260c..736bee5 100644 --- a/epplets/E-Bandwidth.c +++ b/epplets/E-Bandwidth.c @@ -265,7 +265,6 @@ main(int argc, char **argv) { Epplet_adjust_priority(10); - atexit(Epplet_cleanup); Epplet_Init("E-Bandwidth", "0.2", "Enlightenment Network I/O Monitor Epplet", 3, 3, argc, argv, 0); Epplet_load_config(); diff --git a/epplets/E-Biff.c b/epplets/E-Biff.c index fdbb44b..9252152 100644 --- a/epplets/E-Biff.c +++ b/epplets/E-Biff.c @@ -419,7 +419,6 @@ main(int argc, char **argv) { Epplet_adjust_priority(10); - atexit(Epplet_cleanup); Epplet_Init("E-Biff", "0.5", "Enlightenment Mailbox Checker Epplet", 3, 3, argc, argv, 0); Epplet_load_config(); diff --git a/epplets/E-Clock.c b/epplets/E-Clock.c index 37c2b5c..511291c 100644 --- a/epplets/E-Clock.c +++ b/epplets/E-Clock.c @@ -104,7 +104,6 @@ draw_arms(void) int main(int argc, char **argv) { - atexit(Epplet_cleanup); Epplet_Init("E-Clock", "0.1", "Enlightenment Clock Epplet", 3, 3, argc, argv, 0); Epplet_timer(cb_timer, NULL, 0.5, "TIMER"); diff --git a/epplets/E-Cpu.c b/epplets/E-Cpu.c index a116050..7c44a5c 100644 --- a/epplets/E-Cpu.c +++ b/epplets/E-Cpu.c @@ -418,8 +418,6 @@ main(int argc, char **argv) Epplet_adjust_priority(10); - atexit(Epplet_cleanup); - cpus = count_cpus(); load_val = calloc(cpus, sizeof(int)); prev_val = calloc(cpus, sizeof(double)); diff --git a/epplets/E-Disk.c b/epplets/E-Disk.c index 0e91040..9952ebb 100644 --- a/epplets/E-Disk.c +++ b/epplets/E-Disk.c @@ -243,7 +243,6 @@ main(int argc, char **argv) { Epplet_adjust_priority(10); - atexit(Epplet_cleanup); Epplet_Init("E-Disk", "0.2", "Enlightenment Disk I/O Monitor Epplet", 3, 3, argc, argv, 0); Epplet_load_config(); diff --git a/epplets/E-Exec.c b/epplets/E-Exec.c index 2966290..b310c8a 100644 --- a/epplets/E-Exec.c +++ b/epplets/E-Exec.c @@ -233,8 +233,6 @@ cb_configure(void *data __UNUSED__) int main(int argc, char *argv[]) { - atexit(Epplet_cleanup); - if ((argc > 1) && (!strcmp("--proper-size", argv[1]))) { Epplet_Init(EPPLET_NAME, EPPLET_VERSION, EPPLET_INFO, 6, 3, argc, argv, diff --git a/epplets/E-Load.c b/epplets/E-Load.c index 3773a90..d7e90a6 100644 --- a/epplets/E-Load.c +++ b/epplets/E-Load.c @@ -135,7 +135,6 @@ main(int argc, char **argv) { int i; - atexit(Epplet_cleanup); cpus = count_cpus(); load_val = malloc(sizeof(int) * cpus); prev_val = malloc(sizeof(double) * cpus); diff --git a/epplets/E-LoadMeter.c b/epplets/E-LoadMeter.c index 36ad963..6d58436 100644 --- a/epplets/E-LoadMeter.c +++ b/epplets/E-LoadMeter.c @@ -98,8 +98,6 @@ delete_cb(void *data __UNUSED__, Window win __UNUSED__) int main(int argc, char **argv) { - atexit(Epplet_cleanup); - Epplet_Init("E-LoadMeter", "0.1", "Enlightenment CPU Load Epplet", 3, 3, argc, argv, 0); diff --git a/epplets/E-Magic.c b/epplets/E-Magic.c index d06453c..b8cfb8d 100644 --- a/epplets/E-Magic.c +++ b/epplets/E-Magic.c @@ -318,7 +318,6 @@ main(int argc, char **argv) int w = 6, h = 3; srand(getpid() ^ time(NULL)); - atexit(Epplet_cleanup); Epplet_Init("E-Magic", "0.1", "Enlightenment Magic Eight Ball Epplet", w, h, argc, argv, 0); diff --git a/epplets/E-Mixer.c b/epplets/E-Mixer.c index 109d8e1..d68d6a0 100644 --- a/epplets/E-Mixer.c +++ b/epplets/E-Mixer.c @@ -302,8 +302,6 @@ main(int argc, char **argv) { int i; - atexit(Epplet_cleanup); - #ifdef SGI_AUDIO openMixer("audout"); #else diff --git a/epplets/E-MoonClock.c b/epplets/E-MoonClock.c index 061eef1..dfbfe1d 100644 --- a/epplets/E-MoonClock.c +++ b/epplets/E-MoonClock.c @@ -101,8 +101,6 @@ main(int argc, char **argv) { Epplet_adjust_priority(10); - atexit(Epplet_cleanup); - Epplet_Init("E-MoonClock", "0.1", "Enlightenment MoonClock Epplet", 3, 3, argc, argv, 0); Epplet_load_config(); diff --git a/epplets/E-Mountbox.c b/epplets/E-Mountbox.c index 7ce97f6..5fc8aef 100644 --- a/epplets/E-Mountbox.c +++ b/epplets/E-Mountbox.c @@ -1773,7 +1773,6 @@ SyncConfigs(void) int main(int argc, char **argv) { - atexit(Epplet_cleanup); Epplet_Init("E-Mountbox", "0.1", "Enlightenment Mount Epplet", 3, 3, argc, argv, 0); Epplet_load_config(); diff --git a/epplets/E-Net.c b/epplets/E-Net.c index 64d6a88..5b39ebf 100644 --- a/epplets/E-Net.c +++ b/epplets/E-Net.c @@ -104,8 +104,6 @@ main(int argc, char **argv) Epplet_adjust_priority(10); - atexit(Epplet_cleanup); - Epplet_Init("E-Net", "0.1", "Enlightenment Network Load Epplet", 5, 2, argc, argv, 0); Epplet_load_config(); diff --git a/epplets/E-NetFlame.c b/epplets/E-NetFlame.c index 977f6cb..2e4d512 100644 --- a/epplets/E-NetFlame.c +++ b/epplets/E-NetFlame.c @@ -391,8 +391,6 @@ main(int argc, char **argv) Epplet_adjust_priority(10); - atexit(Epplet_cleanup); - load_val = calloc(DIVISIONS, sizeof(int)); prev_val = calloc(DIVISIONS, sizeof(double)); diff --git a/epplets/E-NetGraph.c b/epplets/E-NetGraph.c index 28f20ca..7762eaa 100644 --- a/epplets/E-NetGraph.c +++ b/epplets/E-NetGraph.c @@ -408,7 +408,6 @@ main(int argc, char **argv) Epplet_load_config(); load_config(); - atexit(Epplet_cleanup); while (i < argc) { diff --git a/epplets/E-OpenGL-Demo.c b/epplets/E-OpenGL-Demo.c index c49676c..8ba8c40 100644 --- a/epplets/E-OpenGL-Demo.c +++ b/epplets/E-OpenGL-Demo.c @@ -379,8 +379,6 @@ main(int argc, char **argv) * bit to the sign since the texture is 8 bits per color component. */ GLubyte textureArray[3 * 64 * 64]; - atexit(Epplet_cleanup); - Epplet_Init("E-OpenGL-Demo", "0.1", "Enlightenment OpenGL Demo", 4, 4, argc, argv, 0); Epplet_load_config(); diff --git a/epplets/E-Pants.c b/epplets/E-Pants.c index 92cbcc4..fc87e5c 100644 --- a/epplets/E-Pants.c +++ b/epplets/E-Pants.c @@ -190,8 +190,6 @@ cb_timer(void *data __UNUSED__) int main(int argc, char **argv) { - atexit(Epplet_cleanup); - Epplet_Init("E-Pants", "0.1", "Enlightenment Pants Epplet", 4, 1, argc, argv, 0); Epplet_timer(cb_timer, NULL, 0.05, "TIMER"); diff --git a/epplets/E-Pinger.c b/epplets/E-Pinger.c index 88acc1f..4de3362 100644 --- a/epplets/E-Pinger.c +++ b/epplets/E-Pinger.c @@ -452,8 +452,6 @@ main(int argc, char **argv) { int filedes[2]; - atexit(Epplet_cleanup); - Epplet_Init("E-Pinger", "0.1", "Enlightenment Pinger Epplet", 6, 1, argc, argv, 0); Epplet_timer(cb_timer, NULL, 0.05, "TIMER"); diff --git a/epplets/E-PlayCD.c b/epplets/E-PlayCD.c index d0128f0..2bdc11c 100644 --- a/epplets/E-PlayCD.c +++ b/epplets/E-PlayCD.c @@ -534,8 +534,6 @@ main(int argc, char **argv) Epplet_adjust_priority(10); - atexit(Epplet_cleanup); - Epplet_Init("E-PlayCD", "0.5", "Enlightenment CD Player Epplet", 5, 3, argc, argv, 0); Epplet_load_config(); diff --git a/epplets/E-Power.c b/epplets/E-Power.c index 0652fa2..fa86808 100644 --- a/epplets/E-Power.c +++ b/epplets/E-Power.c @@ -455,7 +455,6 @@ main(int argc, char **argv) Epplet_Init("E-Power", "0.1", "Enlightenment Laptop Power Epplet", 3, 3, argc, argv, 0); - atexit(Epplet_cleanup); Epplet_timer(cb_timer, NULL, 10.0, "TIMER"); b_close = Epplet_create_button(NULL, NULL, 2, 2, 0, 0, "CLOSE", 0, NULL, cb_close, NULL); diff --git a/epplets/E-SD.c b/epplets/E-SD.c index 4b78ad8..88916cd 100644 --- a/epplets/E-SD.c +++ b/epplets/E-SD.c @@ -526,25 +526,17 @@ create_epplet_layout(void) Epplet_register_mouse_leave_handler(cb_out, NULL); } -static void -clean_exit(void) -{ - save_config(); - Epplet_cleanup(); -} - int main(int argc, char **argv) { Epplet_adjust_priority(10); - atexit(clean_exit); - /* Initialise random numbers */ srand(time(0)); Epplet_Init("E-SD", "0.4", "Enlightenment ESD Monitor", 3, 3, argc, argv, 0); Epplet_load_config(); + atexit(save_config); esd.fd = esd_monitor_stream(ESD_BITS16 | ESD_STEREO | ESD_STREAM | ESD_PLAY, diff --git a/epplets/E-ScreenSave.c b/epplets/E-ScreenSave.c index 45c8ff6..3f1d961 100644 --- a/epplets/E-ScreenSave.c +++ b/epplets/E-ScreenSave.c @@ -597,24 +597,17 @@ create_epplet_layout(void) Epplet_register_delete_event_handler(delete_cb, NULL); } -static void -clean_exit(void) -{ - save_config(); - Epplet_cleanup(); -} - int main(int argc, char **argv) { Epplet_adjust_priority(10); - atexit(clean_exit); /* Initialise random numbers */ srand(time(0)); Epplet_Init("E-ScreenSave", "0.7", "Enlightenment Screen Savin' Epplet", 3, 3, argc, argv, 0); Epplet_load_config(); + atexit(save_config); load_config(); create_epplet_layout(); Epplet_show(); diff --git a/epplets/E-ScreenShoot.c b/epplets/E-ScreenShoot.c index b3299e3..4c99844 100644 --- a/epplets/E-ScreenShoot.c +++ b/epplets/E-ScreenShoot.c @@ -933,25 +933,17 @@ create_epplet_layout(void) Epplet_register_mouse_leave_handler(cb_out, NULL); } -static void -clean_exit(void) -{ - save_config(); - Epplet_cleanup(); -} - int main(int argc, char **argv) { Epplet_adjust_priority(10); - atexit(clean_exit); - /* Initialise random numbers */ srand(time(0)); Epplet_Init("E-ScreenShoot", "0.7", "Enlightenment Screen Shootin' Epplet", 3, 3, argc, argv, 0); + atexit(save_config); Epplet_load_config(); load_config(); diff --git a/epplets/E-Slides.c b/epplets/E-Slides.c index 680c36b..cc823ea 100644 --- a/epplets/E-Slides.c +++ b/epplets/E-Slides.c @@ -688,7 +688,6 @@ main(int argc, char **argv) Epplet_adjust_priority(10); - atexit(Epplet_cleanup); srand(getpid() * time(NULL) % ((unsigned int)-1)); for (j = 1; j < argc; j++) diff --git a/epplets/E-Sys.c b/epplets/E-Sys.c index aaa08b1..6e405d1 100644 --- a/epplets/E-Sys.c +++ b/epplets/E-Sys.c @@ -123,8 +123,6 @@ main(int argc, char **argv) { Epplet_adjust_priority(10); - atexit(Epplet_cleanup); - Epplet_Init("E-Sys", "0.1", "Enlightenment Uptime Epplet", 3, 3, argc, argv, 0); diff --git a/epplets/E-Time.c b/epplets/E-Time.c index ec16046..af814a1 100644 --- a/epplets/E-Time.c +++ b/epplets/E-Time.c @@ -226,8 +226,6 @@ parse_config(void) int main(int argc, char **argv) { - atexit(Epplet_cleanup); - Epplet_Init("E-Time", "0.1", "Enlightenment Digital Clock Epplet", 3, 3, argc, argv, 0); Epplet_load_config(); diff --git a/epplets/E-Toolbox.c b/epplets/E-Toolbox.c index ff1e050..00c2653 100644 --- a/epplets/E-Toolbox.c +++ b/epplets/E-Toolbox.c @@ -690,7 +690,6 @@ main(int argc, char **argv) int j = 0, c = 0; prog_name = argv[0]; - atexit(Epplet_cleanup); for (j = 1; j < argc; j++) { diff --git a/epplets/E-UrlWatch.c b/epplets/E-UrlWatch.c index 5bd7350..65b2e54 100644 --- a/epplets/E-UrlWatch.c +++ b/epplets/E-UrlWatch.c @@ -766,13 +766,6 @@ create_epplet_layout(void) display_string("Welcome to E-UrlWatch ;-)"); } -static void -clean_exit(void) -{ - save_config(); - Epplet_cleanup(); -} - static void in_cb(void *data __UNUSED__, Window w) { @@ -816,11 +809,10 @@ main(int argc, char **argv) { Epplet_adjust_priority(10); - atexit(clean_exit); - Epplet_Init("E-UrlWatch", "0.1", "Enlightenment Url Watcher Epplet", 6, 3, argc, argv, 0); Epplet_load_config(); + atexit(save_config); load_config(); create_epplet_layout(); Epplet_register_focus_in_handler(in_cb, NULL); diff --git a/epplets/E-Wireless.c b/epplets/E-Wireless.c index 34d7d2f..36cd73b 100644 --- a/epplets/E-Wireless.c +++ b/epplets/E-Wireless.c @@ -225,8 +225,6 @@ main(int argc, char *argv[]) Epplet_adjust_priority(10); - atexit(Epplet_cleanup); - cpu_hist_size = WIDTH; cpu_hist = (int *)malloc(sizeof(int) * cpu_hist_size); memset(cpu_hist, 0x00, sizeof(int) * cpu_hist_size); diff --git a/epplets/E-Xss.c b/epplets/E-Xss.c index 1d233bc..d4d6290 100644 --- a/epplets/E-Xss.c +++ b/epplets/E-Xss.c @@ -331,8 +331,6 @@ main(int argc, char **argv) { Epplet_adjust_priority(10); - atexit(Epplet_cleanup); - Epplet_Init("E-Xss", "0.2", "Enlightenment Xscreensaver Epplet", 3, 3, argc, argv, 0); Epplet_load_config(); diff --git a/epplets/Emix.c b/epplets/Emix.c index 5fc8937..1ad6ffc 100644 --- a/epplets/Emix.c +++ b/epplets/Emix.c @@ -151,7 +151,6 @@ main(int argc, char *argv[]) return 1; initialize_channels(); - atexit(Epplet_cleanup); Epplet_Init("Emix", "0.9", "Audio mixer Epplet", 3, 3, argc, argv, 0); vs_master = Epplet_create_vslider(2, 2, 44, 0, 20, 1, 5, &channels[0].value, set_volume, diff --git a/epplets/TestEpplet.c b/epplets/TestEpplet.c index 62580dc..eb79c35 100644 --- a/epplets/TestEpplet.c +++ b/epplets/TestEpplet.c @@ -35,7 +35,6 @@ cb_close(void *data __UNUSED__) { Epplet_unremember(); Esync(); - Epplet_cleanup(); exit(0); } --