diff options
author | Henning Baldersheim <balder@yahoo-inc.com> | 2022-05-14 21:03:18 +0000 |
---|---|---|
committer | Henning Baldersheim <balder@yahoo-inc.com> | 2022-05-14 21:23:41 +0000 |
commit | e1f46d816bbd79b43f9ab1f5b411edee4e1d888b (patch) | |
tree | 83bed6f3964be310d7057dc589a3684a61429959 /searchsummary | |
parent | 9cd4da2c34cb9b7b60cd90fb8a5ed3669349c5d1 (diff) |
GC unused code and dependencies
Diffstat (limited to 'searchsummary')
3 files changed, 2 insertions, 28 deletions
diff --git a/searchsummary/CMakeLists.txt b/searchsummary/CMakeLists.txt index ac392c6582c..0a5c7ed4c99 100644 --- a/searchsummary/CMakeLists.txt +++ b/searchsummary/CMakeLists.txt @@ -1,18 +1,12 @@ # Copyright Yahoo. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root. vespa_define_module( DEPENDS - fastos vespalog vespalib - staging_vespalib - fnet configdefinitions fastlib_fast document - config_cloudconfig searchcommon - persistencetypes - metrics searchlib juniper diff --git a/searchsummary/src/vespa/searchsummary/docsummary/juniperproperties.cpp b/searchsummary/src/vespa/searchsummary/docsummary/juniperproperties.cpp index cef0ba5793b..cf3def60f4e 100644 --- a/searchsummary/src/vespa/searchsummary/docsummary/juniperproperties.cpp +++ b/searchsummary/src/vespa/searchsummary/docsummary/juniperproperties.cpp @@ -1,7 +1,6 @@ // Copyright Yahoo. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root. #include "juniperproperties.h" -#include <vespa/searchcommon/config/subscriptionproxyng.h> #include <vespa/vespalib/util/stringfmt.h> using vespa::config::search::summary::JuniperrcConfig; @@ -22,9 +21,7 @@ JuniperProperties::JuniperProperties(const JuniperrcConfig &cfg) : configure(cfg); } -JuniperProperties::~JuniperProperties() { - // empty -} +JuniperProperties::~JuniperProperties() = default; void JuniperProperties::reset() @@ -87,13 +84,6 @@ JuniperProperties::configure(const JuniperrcConfig &cfg) } } -void -JuniperProperties::subscribe(const char *configId) -{ - SubscriptionProxyNg<JuniperProperties, JuniperrcConfig> subscriber(*this, &JuniperProperties::configure); - subscriber.subscribe(configId); -} - const char * JuniperProperties::GetProperty(const char *name, const char *def) { diff --git a/searchsummary/src/vespa/searchsummary/docsummary/juniperproperties.h b/searchsummary/src/vespa/searchsummary/docsummary/juniperproperties.h index 856d564acc6..6b1613b6c07 100644 --- a/searchsummary/src/vespa/searchsummary/docsummary/juniperproperties.h +++ b/searchsummary/src/vespa/searchsummary/docsummary/juniperproperties.h @@ -34,16 +34,6 @@ public: virtual ~JuniperProperties(); /** - * This method subscribes to config from the given configuration id. This does the necessary mapping from - * user-friendly configuration parameters to juniper specific properties. Note that no exceptions thrown by the - * configuration framework are caught in this method. Please refer to the config framework for details on what to - * expect. - * - * @param configId The config id to subscribe to. - */ - void subscribe(const char *configId); - - /** * Implements configure callback for config subscription. * * @param cfg The configuration object. @@ -51,7 +41,7 @@ public: void configure(const vespa::config::search::summary::JuniperrcConfig &cfg); // Inherit doc from IJuniperProperties. - const char *GetProperty(const char *name, const char *def = NULL) override; + const char *GetProperty(const char *name, const char *def) override; }; } |