diff options
author | Aqua-sama <aqua@iserlohn-fortress.net> | 2021-03-26 10:26:10 +0200 |
---|---|---|
committer | Aqua-sama <aqua@iserlohn-fortress.net> | 2021-03-27 07:44:40 +0200 |
commit | 97575a239b8f4d87a8f7ad5bb324c8e71c02d4d9 (patch) | |
tree | c2b5e5de748c9065ef4243e03761ad4afbe9a36b /src/cpu | |
parent | makefile: make makefile recursive (diff) | |
download | kernel.cpp-97575a239b8f4d87a8f7ad5bb324c8e71c02d4d9.tar.xz |
clang-tidy fixes
Diffstat (limited to 'src/cpu')
-rw-r--r-- | src/cpu/cpu.h | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/src/cpu/cpu.h b/src/cpu/cpu.h index 2dc9cbe..5b4763f 100644 --- a/src/cpu/cpu.h +++ b/src/cpu/cpu.h @@ -75,14 +75,14 @@ public: CPU(); ~CPU() = default; - const char* manufacturer() const { return m_manufacturer; } - uint8_t family() const { return m_info.family == 15 ? m_info.family + m_info.family_ex : m_info.family; } - uint8_t model() const { + [[nodiscard]] const char* manufacturer() const { return m_manufacturer; } + [[nodiscard]] uint8_t family() const { return m_info.family == 15 ? m_info.family + m_info.family_ex : m_info.family; } + [[nodiscard]] uint8_t model() const { return (m_info.model == 6 || m_info.model == 15) ? (static_cast<uint8_t>(m_info.model_ex << 4) + m_info.model) : m_info.model; } - uint8_t stepping() const { return m_info.stepping; } - auto features() const { return m_features; } + [[nodiscard]] uint8_t stepping() const { return m_info.stepping; } + [[nodiscard]] auto features() const { return m_features; } private: char m_manufacturer[13] = {'\0'}; @@ -91,10 +91,10 @@ private: uint8_t model : 4 = 0; uint8_t family : 4 = 0; uint8_t type : 2 = 0; - uint8_t reserved_1 : 2 = 0; + [[maybe_unused]] uint8_t unused_1 : 2 = 0; uint8_t model_ex : 4 = 0; uint8_t family_ex : 8 = 0; - uint8_t reserved_2 : 4 = 0; + [[maybe_unused]] uint8_t unused_2 : 4 = 0; } __attribute__((packed)) m_info; features_t m_features; |