diff options
| author |  Lioncash | 2018-07-18 00:27:34 -0400 |
|---|---|---|
| committer | Lioncash | 2018-07-18 00:28:47 -0400 |
| commit | 0aebe6b3d52403e042522720cb5646ddbcb306d6 (patch) | |
| tree | 5a5d3665b7978dffe64830886174a6d1c8a61a92 | |
| parent | telemetry: Default copy/move constructors and assignment operators (diff) | |
| download | yuzu-0aebe6b3d52403e042522720cb5646ddbcb306d6.tar.gz yuzu-0aebe6b3d52403e042522720cb5646ddbcb306d6.tar.xz yuzu-0aebe6b3d52403e042522720cb5646ddbcb306d6.zip | |
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.
Diffstat (limited to '')
| -rw-r--r-- | src/common/telemetry.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/common/telemetry.h b/src/common/telemetry.h index 2c945443b..155cf59ff 100644 --- a/src/common/telemetry.h +++ b/src/common/telemetry.h | |||
| @@ -84,11 +84,11 @@ public: | |||
| 84 | return value; | 84 | return value; |
| 85 | } | 85 | } |
| 86 | 86 | ||
| 87 | inline bool operator==(const Field<T>& other) { | 87 | bool operator==(const Field& other) const { |
| 88 | return (type == other.type) && (name == other.name) && (value == other.value); | 88 | return (type == other.type) && (name == other.name) && (value == other.value); |
| 89 | } | 89 | } |
| 90 | 90 | ||
| 91 | inline bool operator!=(const Field<T>& other) { | 91 | bool operator!=(const Field& other) const { |
| 92 | return !(*this == other); | 92 | return !(*this == other); |
| 93 | } | 93 | } |
| 94 | 94 | ||