diff options
author | Henning Baldersheim <balder@oath.com> | 2018-08-10 23:35:13 +0200 |
---|---|---|
committer | Henning Baldersheim <balder@oath.com> | 2018-08-10 23:35:13 +0200 |
commit | 4be58f582e0cb3313eedc0abedc86170ab41580c (patch) | |
tree | 8a4a10d78e98cd4b0aa4f1eff364cfe81ed8bd7b /storage | |
parent | 6ecf0ff7e093eae86292c283bf586c405a557659 (diff) |
Pass stringref by value
Diffstat (limited to 'storage')
7 files changed, 10 insertions, 10 deletions
diff --git a/storage/src/vespa/storage/common/hostreporter/kernelmetrictool.cpp b/storage/src/vespa/storage/common/hostreporter/kernelmetrictool.cpp index c8d73737d7c..ea6e266f0b2 100644 --- a/storage/src/vespa/storage/common/hostreporter/kernelmetrictool.cpp +++ b/storage/src/vespa/storage/common/hostreporter/kernelmetrictool.cpp @@ -30,7 +30,7 @@ vespalib::string stripWhitespace(const vespalib::string& s) { return s.substr(start, stop - start + 1); } -vespalib::string getLine(const vespalib::stringref& key, +vespalib::string getLine(vespalib::stringref key, const vespalib::stringref& content) { vespalib::string::size_type start(0); @@ -63,7 +63,7 @@ uint32_t getTokenCount(const vespalib::string& line) { return st.size(); } -uint64_t toLong(const vespalib::stringref& s, int base) { +uint64_t toLong(vespalib::stringref s, int base) { char* endptr; // FIXME C++17 range-safe from_chars() instead of strtoull() uint64_t result(strtoull(s.data(), &endptr, base)); diff --git a/storage/src/vespa/storage/common/hostreporter/kernelmetrictool.h b/storage/src/vespa/storage/common/hostreporter/kernelmetrictool.h index dd41cb1fcd2..0d07f045de2 100644 --- a/storage/src/vespa/storage/common/hostreporter/kernelmetrictool.h +++ b/storage/src/vespa/storage/common/hostreporter/kernelmetrictool.h @@ -15,14 +15,14 @@ vespalib::string readFile(const char* fileName); vespalib::string stripWhitespace(const vespalib::string& s); -vespalib::string getLine(const vespalib::stringref& key, +vespalib::string getLine(vespalib::stringref key, const vespalib::stringref& content); vespalib::string getToken(uint32_t index, const vespalib::string& line); uint32_t getTokenCount(const vespalib::string& line); -uint64_t toLong(const vespalib::stringref& s, int base = 0) ; +uint64_t toLong(vespalib::stringref s, int base = 0) ; } /* namespace kernelmetrictool */ } /* namespace storage */ diff --git a/storage/src/vespa/storage/config/distributorconfiguration.h b/storage/src/vespa/storage/config/distributorconfiguration.h index d3e589f9d00..8c2e0568796 100644 --- a/storage/src/vespa/storage/config/distributorconfiguration.h +++ b/storage/src/vespa/storage/config/distributorconfiguration.h @@ -70,11 +70,11 @@ public: _lastGarbageCollectionChange = lastChangeTime; } - bool stateCheckerIsActive(const vespalib::stringref & stateCheckerName) const { + bool stateCheckerIsActive(vespalib::stringref stateCheckerName) const { return _blockedStateCheckers.find(stateCheckerName) == _blockedStateCheckers.end(); } - void disableStateChecker(const vespalib::stringref & stateCheckerName) { + void disableStateChecker(vespalib::stringref stateCheckerName) { _blockedStateCheckers.insert(stateCheckerName); } diff --git a/storage/src/vespa/storage/storageserver/fnetlistener.cpp b/storage/src/vespa/storage/storageserver/fnetlistener.cpp index 32ea8982ffb..d92b672c52a 100644 --- a/storage/src/vespa/storage/storageserver/fnetlistener.cpp +++ b/storage/src/vespa/storage/storageserver/fnetlistener.cpp @@ -43,7 +43,7 @@ FNetListener::getListenPort() const } void -FNetListener::registerHandle(const vespalib::stringref & handle) { +FNetListener::registerHandle(vespalib::stringref handle) { _slobrokRegister.registerName(handle); while (_slobrokRegister.busy()) { LOG(debug, "Waiting to register in slobrok"); diff --git a/storage/src/vespa/storage/storageserver/fnetlistener.h b/storage/src/vespa/storage/storageserver/fnetlistener.h index bf004a3eb2b..2ffdae24b29 100644 --- a/storage/src/vespa/storage/storageserver/fnetlistener.h +++ b/storage/src/vespa/storage/storageserver/fnetlistener.h @@ -27,7 +27,7 @@ public: void RPC_getCurrentTime(FRT_RPCRequest *req); void RPC_setDistributionStates(FRT_RPCRequest* req); - void registerHandle(const vespalib::stringref & handle); + void registerHandle(vespalib::stringref handle); void close(); int getListenPort() const; diff --git a/storage/src/vespa/storage/visiting/visitorthread.cpp b/storage/src/vespa/storage/visiting/visitorthread.cpp index 6bf28b08540..f0374e07ef7 100644 --- a/storage/src/vespa/storage/visiting/visitorthread.cpp +++ b/storage/src/vespa/storage/visiting/visitorthread.cpp @@ -365,7 +365,7 @@ VisitorThread::handleNonExistingVisitorCall(const Event& entry, * Utility function to get a visitor instance from a given library. */ std::shared_ptr<Visitor> -VisitorThread::createVisitor(const vespalib::stringref & libName, +VisitorThread::createVisitor(vespalib::stringref libName, const vdslib::Parameters& params, vespalib::asciistream & error) { diff --git a/storage/src/vespa/storage/visiting/visitorthread.h b/storage/src/vespa/storage/visiting/visitorthread.h index 97ca70f3761..483837ed741 100644 --- a/storage/src/vespa/storage/visiting/visitorthread.h +++ b/storage/src/vespa/storage/visiting/visitorthread.h @@ -125,7 +125,7 @@ private: void trimRecentlyCompletedList(framework::SecondTime currentTime); void handleNonExistingVisitorCall(const Event& entry, api::ReturnCode& code); - std::shared_ptr<Visitor> createVisitor(const vespalib::stringref & libName, + std::shared_ptr<Visitor> createVisitor(vespalib::stringref libName, const vdslib::Parameters& params, vespalib::asciistream & error); |