Skip to content

Commit ddab9e5

Browse files
committed
clang-tidy fixes
1 parent feba24f commit ddab9e5

2 files changed

Lines changed: 10 additions & 3 deletions

File tree

.github/workflows/clang-tidy.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ jobs:
3333
run: |
3434
sudo apt-get update -q
3535
sudo apt-get install -y --no-install-recommends \
36-
cmake clang clang-tidy python3-clang-tidy build-essential
36+
cmake clang clang-tidy build-essential
3737
3838
- name: Configure CMake
3939
run: |

src/vmaware.hpp

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -459,7 +459,7 @@
459459
namespace brands { // TODO, remove this in the 2.8.0 or any release after the 2.7.0
460460
#define LEGACY(name, full_name) \
461461
[[deprecated("Use VM::brands::" #name " instead")]] \
462-
static constexpr const char* (name) = full_name
462+
static constexpr const char* name = full_name /* NOLINT(bugprone-macro-parentheses) */
463463

464464
LEGACY(NULL_BRAND, "Unknown");
465465
LEGACY(VBOX, "VirtualBox");
@@ -835,8 +835,15 @@ struct VM {
835835
*b = static_cast<unsigned int>(regs[1]);
836836
*c = static_cast<unsigned int>(regs[2]);
837837
*d = static_cast<unsigned int>(regs[3]);
838-
#else
838+
#elif (x86)
839839
__get_cpuid_count(leaf, subleaf, a, b, c, d);
840+
#else
841+
VMAWARE_UNUSED(leaf);
842+
VMAWARE_UNUSED(subleaf);
843+
VMAWARE_UNUSED(a);
844+
VMAWARE_UNUSED(b);
845+
VMAWARE_UNUSED(c);
846+
VMAWARE_UNUSED(d);
840847
#endif
841848
}
842849

0 commit comments

Comments
 (0)