commit 1b235ece3a057851da549b1db65e277ebf7632a8
author Vitaly Buka <vitalybuka@google.com> Sat May 01 01:55:21 2021 -0700
committer Copybara-Service <copybara-worker@google.com> Sun May 09 19:09:51 2021 -0700
tree 3c33bef5acdf150ee45a09a7285fdb0497de155e
parent 9cbe475cffd3e6aa873c55710882a11abbf359c0

[scudo][NFC] Fix clang-tidy warnings

GitOrigin-RevId: 95aa116d0cf3fcc67bfab050fda132b14960c4df

secondary.h

diff --git a/secondary.h b/secondary.h
index 540f6b6..ea5d680 100644
--- a/secondary.h
+++ b/secondary.h
@@ -9,9 +9,12 @@
 #ifndef SCUDO_SECONDARY_H_
 #define SCUDO_SECONDARY_H_
 
+#include "chunk.h"
 #include "common.h"
 #include "list.h"
+#include "memtag.h"
 #include "mutex.h"
+#include "options.h"
 #include "stats.h"
 #include "string_utils.h"
 
@@ -279,13 +282,15 @@
               Config::SecondaryCacheMinReleaseToOsIntervalMs);
       atomic_store_relaxed(&ReleaseToOsIntervalMs, Interval);
       return true;
-    } else if (O == Option::MaxCacheEntriesCount) {
+    }
+    if (O == Option::MaxCacheEntriesCount) {
       const u32 MaxCount = static_cast<u32>(Value);
       if (MaxCount > Config::SecondaryCacheEntriesArraySize)
         return false;
       atomic_store_relaxed(&MaxEntriesCount, MaxCount);
       return true;
-    } else if (O == Option::MaxCacheEntrySize) {
+    }
+    if (O == Option::MaxCacheEntrySize) {
       atomic_store_relaxed(&MaxEntrySize, static_cast<uptr>(Value));
       return true;
     }