summaryrefslogtreecommitdiff
path: root/src/filter
diff options
context:
space:
mode:
authorMax Kellermann <max@musicpd.org>2018-07-17 21:08:41 +0200
committerMax Kellermann <max@musicpd.org>2018-07-17 21:08:41 +0200
commit5b192beaa5cb8f007b04bb4c52fc2076c12107c6 (patch)
treeea316e7beb98cbf6b6870d0d545d89f77425d0c7 /src/filter
parentef38330d748742945a781ef520ffd4de2d1f9301 (diff)
config/Global: remove ConfigBlock::SetUsed() call, let caller do that
This fixes an old bug which caused the "unused" warnings to be unreliable; only the first block in the list was marked as being "used", no matter if it was really used, and the rest was never marked as "used", suppressing all warnings for them.
Diffstat (limited to 'src/filter')
-rw-r--r--src/filter/LoadChain.cxx2
1 files changed, 2 insertions, 0 deletions
diff --git a/src/filter/LoadChain.cxx b/src/filter/LoadChain.cxx
index 558134b92..74c94a0a9 100644
--- a/src/filter/LoadChain.cxx
+++ b/src/filter/LoadChain.cxx
@@ -42,6 +42,8 @@ filter_chain_append_new(PreparedFilter &chain, const char *template_name)
throw FormatRuntimeError("Filter template not found: %s",
template_name);
+ cfg->SetUsed();
+
// Instantiate one of those filter plugins with the template name as a hint
auto f = filter_configured_new(*cfg);