telemetry: Remove unnecessary Field constructor
We can just take the value parameter by value which allows both moving into it, and copies at the same time, depending on the calling code.
This commit is contained in:
parent
0aebe6b3d5
commit
c65a8fafa0
|
@ -52,10 +52,7 @@ public:
|
|||
template <typename T>
|
||||
class Field : public FieldInterface {
|
||||
public:
|
||||
Field(FieldType type, std::string name, const T& value)
|
||||
: name(std::move(name)), type(type), value(value) {}
|
||||
|
||||
Field(FieldType type, std::string name, T&& value)
|
||||
Field(FieldType type, std::string name, T value)
|
||||
: name(std::move(name)), type(type), value(std::move(value)) {}
|
||||
|
||||
Field(const Field&) = default;
|
||||
|
|
Reference in New Issue