From 2864d019f05b8c8bb3159f2c8913ed834e487698 Mon Sep 17 00:00:00 2001 From: Patric Stout Date: Tue, 8 Dec 2020 10:24:59 +0100 Subject: Fix: useless warning with -snull and no BaseSounds available (#8361) If I explicitly tell the system I do not want sound, I still get presented a nice message I do not have any BaseSounds available on my system, and that I should download one to enjoy sound. Well, let me tell you, with "-snull" that is really really not going to help. So please, be quiet, and let me enjoy the game without "boooooo" and "DING DING DING". Thank you. --- src/sound/null_s.h | 1 + src/sound/sound_driver.hpp | 11 +++++++++++ 2 files changed, 12 insertions(+) (limited to 'src/sound') diff --git a/src/sound/null_s.h b/src/sound/null_s.h index c01eae32a..9bc660b92 100644 --- a/src/sound/null_s.h +++ b/src/sound/null_s.h @@ -19,6 +19,7 @@ public: void Stop() override { } const char *GetName() const override { return "null"; } + bool HasOutput() const override { return false; } }; /** Factory for the null sound driver. */ diff --git a/src/sound/sound_driver.hpp b/src/sound/sound_driver.hpp index 6de66d74e..062e7704d 100644 --- a/src/sound/sound_driver.hpp +++ b/src/sound/sound_driver.hpp @@ -18,6 +18,17 @@ public: /** Called once every tick */ virtual void MainLoop() {} + /** + * Whether the driver has an output from which the user can hear sound. + * Or in other words, whether we should warn the user if no soundset is + * loaded and that loading one would fix the sound problems. + * @return True for all drivers except null. + */ + virtual bool HasOutput() const + { + return true; + } + /** * Get the currently active instance of the sound driver. */ -- cgit v1.2.3-54-g00ecf