From f38dcc64b518d3b7ba41d6b2702ec72303934376 Mon Sep 17 00:00:00 2001 From: Ben Wiederhake Date: Sat, 27 May 2023 01:08:58 +0200 Subject: [PATCH] AudioApplet: Store default config value in a single place The hope is that this can later be used to: - verify that all accesses to the same key use the same default value, - and extract the default values more easily. --- Userland/Applets/Audio/main.cpp | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/Userland/Applets/Audio/main.cpp b/Userland/Applets/Audio/main.cpp index 8bce27ab30..be5e3070e3 100644 --- a/Userland/Applets/Audio/main.cpp +++ b/Userland/Applets/Audio/main.cpp @@ -24,6 +24,8 @@ #include #include +static constexpr bool audio_applet_show_percent_default = false; + class AudioWidget final : public GUI::Widget { C_OBJECT_ABSTRACT(AudioWidget) @@ -53,7 +55,7 @@ private: AudioWidget(NonnullRefPtr audio_client, Array volume_level_bitmaps) : m_audio_client(move(audio_client)) , m_volume_level_bitmaps(move(volume_level_bitmaps)) - , m_show_percent(Config::read_bool("AudioApplet"sv, "Applet"sv, "ShowPercent"sv, false)) + , m_show_percent(Config::read_bool("AudioApplet"sv, "Applet"sv, "ShowPercent"sv, audio_applet_show_percent_default)) { m_audio_volume = static_cast(m_audio_client->get_main_mix_volume() * 100); m_audio_muted = m_audio_client->is_main_mix_muted(); @@ -246,7 +248,7 @@ ErrorOr serenity_main(Main::Arguments arguments) window->show(); // This positioning code depends on the window actually existing. - static_cast(window->main_widget())->set_audio_widget_size(Config::read_bool("AudioApplet"sv, "Applet"sv, "ShowPercent"sv, false)); + static_cast(window->main_widget())->set_audio_widget_size(Config::read_bool("AudioApplet"sv, "Applet"sv, "ShowPercent"sv, audio_applet_show_percent_default)); TRY(Core::System::pledge("stdio recvfd sendfd rpath"));