diff options
author | Arne H Juul <arnej27959@users.noreply.github.com> | 2017-04-24 12:10:42 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2017-04-24 12:10:42 +0200 |
commit | 9ff4bdb407ed8d855a3f86a17c99906ff738177b (patch) | |
tree | fc2b050224d7dde92d57e1f9cac12c1e5aaf6b90 /slobrok/src/tests | |
parent | 32ae190acc9ac5081049e1c7008d1602c68cf821 (diff) |
Revert "Balder/enforce override 2"
Diffstat (limited to 'slobrok/src/tests')
-rw-r--r-- | slobrok/src/tests/mirrorapi/match_test.cpp | 7 | ||||
-rw-r--r-- | slobrok/src/tests/oldapi/mirror.h | 11 | ||||
-rw-r--r-- | slobrok/src/tests/startsome/rpc_info.cpp | 5 | ||||
-rw-r--r-- | slobrok/src/tests/startsome/tstdst.cpp | 10 |
4 files changed, 24 insertions, 9 deletions
diff --git a/slobrok/src/tests/mirrorapi/match_test.cpp b/slobrok/src/tests/mirrorapi/match_test.cpp index 94900daa202..d9ebcfd6f29 100644 --- a/slobrok/src/tests/mirrorapi/match_test.cpp +++ b/slobrok/src/tests/mirrorapi/match_test.cpp @@ -4,11 +4,12 @@ class MatchTester : public slobrok::api::IMirrorAPI { - SpecList lookup(const std::string &) const override { + virtual SpecList lookup(const std::string &) const override { return SpecList(); } - uint32_t updates() const override { return 0; } - bool ready() const override { return true; } + virtual uint32_t updates() const override { return 0; } + + virtual bool ready() const override { return true; } const std::string name; diff --git a/slobrok/src/tests/oldapi/mirror.h b/slobrok/src/tests/oldapi/mirror.h index b83392dec5e..1f0db564cae 100644 --- a/slobrok/src/tests/oldapi/mirror.h +++ b/slobrok/src/tests/oldapi/mirror.h @@ -1,5 +1,6 @@ // Copyright 2016 Yahoo Inc. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root. -#pragma once +#ifndef Old_MIRROR_H +#define Old_MIRROR_H #include <vespa/fnet/frt/frt.h> #include <vespa/vespalib/util/gencnt.h> @@ -85,7 +86,10 @@ public: **/ ~MirrorOld(); + // Inherit doc from IMirrorOld. SpecList lookup(const std::string & pattern) const override; + + // Inherit doc from IMirrorOld. uint32_t updates() const override { return _updates.getAsInt(); } /** @@ -106,7 +110,10 @@ private: MirrorOld(const MirrorOld &); MirrorOld &operator=(const MirrorOld &); + /** from FNET_Task, polls slobrok **/ void PerformTask() override; + + /** from FRT_IRequestWait **/ void RequestDone(FRT_RPCRequest *req) override; FRT_Supervisor &_orb; @@ -124,3 +131,5 @@ private: } // namespace api } // namespace slobrok + +#endif diff --git a/slobrok/src/tests/startsome/rpc_info.cpp b/slobrok/src/tests/startsome/rpc_info.cpp index be5418ff1a8..76b578d4f88 100644 --- a/slobrok/src/tests/startsome/rpc_info.cpp +++ b/slobrok/src/tests/startsome/rpc_info.cpp @@ -1,7 +1,8 @@ // Copyright 2016 Yahoo Inc. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root. - +#include <vespa/fastos/fastos.h> +#include <vespa/log/log.h> +LOG_SETUP("rpc_info"); #include <vespa/fnet/frt/frt.h> -#include <vespa/fastos/app.h> class RPCInfo : public FastOS_Application { diff --git a/slobrok/src/tests/startsome/tstdst.cpp b/slobrok/src/tests/startsome/tstdst.cpp index 7d3fcef8695..e2f6ae070b3 100644 --- a/slobrok/src/tests/startsome/tstdst.cpp +++ b/slobrok/src/tests/startsome/tstdst.cpp @@ -1,11 +1,15 @@ // Copyright 2016 Yahoo Inc. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root. +#include <vespa/fastos/fastos.h> +#include <vespa/log/log.h> +LOG_SETUP("testrpcserver"); #include <vespa/vespalib/util/host_name.h> +#include <vespa/fnet/fnet.h> #include <vespa/fnet/frt/frt.h> -#include <vespa/fastos/app.h> -#include <vespa/log/log.h> -LOG_SETUP("testrpcserver"); +#include <vector> +#include <string> +#include <sstream> class FNET_Transport; class FNET_Scheduler; |