Stop saving notification setting on unload
Review Request #3024 — Created March 12, 2024 and submitted — Latest diff uploaded
When libpurple is shutting down, the settings backend is gone before the plugins (and maybe this should be fixed elsewhere), so all the saved settings leak as they attempt to write to a
NULL
settings object.All places in this plugin that change the settings then save them, so there's no need to save on unload.
Ran in valgrind with the plugin enabled, and there were no longer any leaks of settings values from unload at exit.