Update for comments.

Author: cferris1000

compiler-rt/lib/scudo/standalone/combined.h

@@ -184,10 +184,11 @@ class Allocator {
     const s32 ReleaseToOsIntervalMs = getFlags()->release_to_os_interval_ms;
     Primary.init(ReleaseToOsIntervalMs);
     Secondary.init(&Stats, ReleaseToOsIntervalMs);
-    if (!AllocatorConfig::getQuarantineDisabled())
+    if (!AllocatorConfig::getQuarantineDisabled()) {
       Quarantine.init(
           static_cast<uptr>(getFlags()->quarantine_size_kb << 10),
           static_cast<uptr>(getFlags()->thread_local_quarantine_size_kb << 10));
+    }
   }
 
   void enableRingBuffer() NO_THREAD_SAFETY_ANALYSIS {
@@ -277,18 +278,20 @@ class Allocator {
   //   the last two items).
   void commitBack(TSD<ThisT> *TSD) {
     TSD->assertLocked(/*BypassCheck=*/true);
-    if (!AllocatorConfig::getQuarantineDisabled())
+    if (!AllocatorConfig::getQuarantineDisabled()) {
       Quarantine.drain(&TSD->getQuarantineCache(),
                        QuarantineCallback(*this, TSD->getSizeClassAllocator()));
+    }
     TSD->getSizeClassAllocator().destroy(&Stats);
   }
 
   void drainCache(TSD<ThisT> *TSD) {
     TSD->assertLocked(/*BypassCheck=*/true);
-    if (!AllocatorConfig::getQuarantineDisabled())
+    if (!AllocatorConfig::getQuarantineDisabled()) {
       Quarantine.drainAndRecycle(
           &TSD->getQuarantineCache(),
           QuarantineCallback(*this, TSD->getSizeClassAllocator()));
+    }
     TSD->getSizeClassAllocator().drain();
   }
   void drainCaches() { TSDRegistry.drainCaches(this); }

compiler-rt/lib/scudo/standalone/tests/combined_test.cpp

@@ -1133,7 +1133,7 @@ TEST(ScudoCombinedTest, QuarantineEnabled) {
   }
   EXPECT_GE(Stats.size(), ActualSize);
 
-  // No quarantine stats should exist.
+  // Quarantine stats should be present.
   EXPECT_NE(Stats.find("Stats: Quarantine"), std::string::npos);
 }
 
@@ -1158,6 +1158,6 @@ TEST(ScudoCombinedTest, QuarantineDisabled) {
   }
   EXPECT_GE(Stats.size(), ActualSize);
 
-  // No quarantine stats should exist.
+  // No quarantine stats should not be present.
   EXPECT_EQ(Stats.find("Stats: Quarantine"), std::string::npos);
 }