telemetry: Make operator== and operator!= const member functions of Field
These operators don't modify internal class state, so they can be made const member functions. While we're at it, drop the unnecessary inline keywords. Member functions that are defined in the class declaration are already inline by default.
This commit is contained in:
parent
faa62b0d1e
commit
3244042c4c
|
@ -84,11 +84,11 @@ public:
|
||||||
return value;
|
return value;
|
||||||
}
|
}
|
||||||
|
|
||||||
inline bool operator==(const Field<T>& other) {
|
bool operator==(const Field& other) const {
|
||||||
return (type == other.type) && (name == other.name) && (value == other.value);
|
return (type == other.type) && (name == other.name) && (value == other.value);
|
||||||
}
|
}
|
||||||
|
|
||||||
inline bool operator!=(const Field<T>& other) {
|
bool operator!=(const Field& other) const {
|
||||||
return !(*this == other);
|
return !(*this == other);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
Reference in New Issue