Zander671/Waybar
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

modules: systemd_failed_units: Introduce RequestFailedUnits member

Browse Source 
Split-out request of failed units from systemd into a separate member
function.

This increases the readability and extendability, but preserves the
current functionality (non-functional change).

Signed-off-by: Steffen Kothe <steffen.kothe@skothe.net>
This commit is contained in:
Steffen Kothe
2025-06-21 15:34:44 +00:00
parent 4bb06b86bc
commit 0731117679
2 changed files with 7 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
include/modules/systemd_failed_units.hpp
View File

@ -25,6 +25,7 @@ class SystemdFailedUnits : public ALabel {
void notify_cb(const Glib::ustring &sender_name, const Glib::ustring &signal_name, void notify_cb(const Glib::ustring &sender_name, const Glib::ustring &signal_name,
const Glib::VariantContainerBase &arguments); const Glib::VariantContainerBase &arguments);
void RequestFailedUnits();
void updateData(); void updateData();
}; };

9
src/modules/systemd_failed_units.cpp
View File

@ -68,9 +68,7 @@ auto SystemdFailedUnits::notify_cb(const Glib::ustring& sender_name,
} }
} }
void SystemdFailedUnits::updateData() { void SystemdFailedUnits::RequestFailedUnits() {
update_pending = false;
auto load = [](const char* kind, Glib::RefPtr<Gio::DBus::Proxy>& proxy) -> uint32_t { auto load = [](const char* kind, Glib::RefPtr<Gio::DBus::Proxy>& proxy) -> uint32_t {
try { try {
if (!proxy) return 0; if (!proxy) return 0;
@ -93,6 +91,11 @@ void SystemdFailedUnits::updateData() {
nr_failed_system = load("systemwide", system_proxy); nr_failed_system = load("systemwide", system_proxy);
nr_failed_user = load("user", user_proxy); nr_failed_user = load("user", user_proxy);
nr_failed = nr_failed_system + nr_failed_user; nr_failed = nr_failed_system + nr_failed_user;
}
void SystemdFailedUnits::updateData() {
update_pending = false;
RequestFailedUnits();
dp.emit(); dp.emit();
} }