From fff265a9a99d255439de538acf63ac053854e55c Mon Sep 17 00:00:00 2001 From: Andrew Kaster Date: Sun, 31 Oct 2021 16:43:03 -0600 Subject: [PATCH] Kernel: Suppress clang-tidy warning on declaration of s_mm_lock Seems we are declaring this guy as extern RecursiveSpinLock s_mm_lock; in both Thread.h and MemoryManager.h. Smells funny for sure. --- Kernel/Memory/MemoryManager.h | 1 + 1 file changed, 1 insertion(+) diff --git a/Kernel/Memory/MemoryManager.h b/Kernel/Memory/MemoryManager.h index 13380820b56..e1229d1bb53 100644 --- a/Kernel/Memory/MemoryManager.h +++ b/Kernel/Memory/MemoryManager.h @@ -100,6 +100,7 @@ struct MemoryManagerData { PhysicalAddress m_last_quickmap_pt; }; +// NOLINTNEXTLINE(readability-redundant-declaration) FIXME: Why do we declare this here *and* in Thread.h? extern RecursiveSpinlock s_mm_lock; // This class represents a set of committed physical pages.