Common/Analytics: std::move std::string constructor parameter

Allows calling code to move into the constructor, avoiding the creation
of another string copy.
This commit is contained in:
Lioncash 2019-06-03 18:33:02 -04:00
parent f813c4951a
commit 58e2cd5486
2 changed files with 2 additions and 2 deletions

View File

@ -198,7 +198,7 @@ void StdoutAnalyticsBackend::Send(std::string report)
HexDump(reinterpret_cast<const u8*>(report.data()), report.size()).c_str()); HexDump(reinterpret_cast<const u8*>(report.data()), report.size()).c_str());
} }
HttpAnalyticsBackend::HttpAnalyticsBackend(const std::string& endpoint) : m_endpoint(endpoint) HttpAnalyticsBackend::HttpAnalyticsBackend(std::string endpoint) : m_endpoint(std::move(endpoint))
{ {
} }

View File

@ -184,7 +184,7 @@ public:
class HttpAnalyticsBackend : public AnalyticsReportingBackend class HttpAnalyticsBackend : public AnalyticsReportingBackend
{ {
public: public:
HttpAnalyticsBackend(const std::string& endpoint); explicit HttpAnalyticsBackend(std::string endpoint);
~HttpAnalyticsBackend() override; ~HttpAnalyticsBackend() override;
void Send(std::string report) override; void Send(std::string report) override;