Style

Author: tom-andersen

Firestore/core/src/local/globals_cache.h

@@ -29,10 +29,11 @@ using nanopb::ByteString;
 /**
  * General purpose cache for global values.
  *
- * Global state that cuts across components should be saved here. Following are contained herein:
+ * Global state that cuts across components should be saved here. Following are
+ * contained herein:
  *
- * `sessionToken` tracks server interaction across Listen and Write streams. This facilitates cache
- * synchronization and invalidation.
+ * `sessionToken` tracks server interaction across Listen and Write streams.
+ * This facilitates cache synchronization and invalidation.
  */
 class GlobalsCache {
  public:
@@ -47,7 +48,6 @@ class GlobalsCache {
    * Sets session token.
    */
   virtual void SetSessionToken(const ByteString& session_token) = 0;
-
 };
 
 }  // namespace local

Firestore/core/src/local/leveldb_globals_cache.h

@@ -1,18 +1,18 @@
 /*
-* Copyright 2024 Google LLC
-*
-* Licensed under the Apache License, Version 2.0 (the "License");
-* you may not use this file except in compliance with the License.
-* You may obtain a copy of the License at
-*
-*      http://www.apache.org/licenses/LICENSE-2.0
-*
-* Unless required by applicable law or agreed to in writing, software
-* distributed under the License is distributed on an "AS IS" BASIS,
-* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-* See the License for the specific language governing permissions and
-* limitations under the License.
-*/
+ * Copyright 2024 Google LLC
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
 
 #ifndef FIRESTORE_CORE_SRC_LOCAL_LEVELDB_GLOBALS_CACHE_H
 #define FIRESTORE_CORE_SRC_LOCAL_LEVELDB_GLOBALS_CACHE_H
@@ -40,11 +40,9 @@ class LevelDbGlobalsCache : public GlobalsCache {
    */
   void SetSessionToken(const ByteString& session_token) override;
 
-
  private:
   // The LevelDbGlobalsCache is owned by LevelDbPersistence.
   LevelDbPersistence* db_ = nullptr;
-
 };
 
 }  // namespace local

Firestore/core/src/local/leveldb_persistence.h

@@ -23,7 +23,6 @@
 #include <unordered_map>
 
 #include "Firestore/core/src/credentials/user.h"
-#include "Firestore/core/src/local/leveldb_globals_cache.h"
 #include "Firestore/core/src/local/leveldb_bundle_cache.h"
 #include "Firestore/core/src/local/leveldb_document_overlay_cache.h"
 #include "Firestore/core/src/local/leveldb_globals_cache.h"

Firestore/core/src/local/memory_globals_cache.h

@@ -28,7 +28,6 @@ namespace local {
 
 class MemoryGlobalsCache : public GlobalsCache {
  public:
-
   /**
    * Gets session token.
    */

Firestore/core/test/unit/local/globals_cache_test.cc

@@ -36,10 +36,11 @@ GlobalsCacheTest::GlobalsCacheTest() : GlobalsCacheTest(GetParam()()) {
 namespace {
 
 TEST_P(GlobalsCacheTest, ReturnsEmptyBytestringWhenSessionTokenNotFound) {
-  persistence_->Run("test_returns_empty_bytestring_when_session_token_not_found", [&] {
-    auto expected = ByteString();
-    EXPECT_EQ(cache_->GetSessionToken(), expected);
-  });
+  persistence_->Run(
+      "test_returns_empty_bytestring_when_session_token_not_found", [&] {
+        auto expected = ByteString();
+        EXPECT_EQ(cache_->GetSessionToken(), expected);
+      });
 }
 
 TEST_P(GlobalsCacheTest, ReturnsSavedSessionToken) {
@@ -59,7 +60,7 @@ TEST_P(GlobalsCacheTest, ReturnsSavedSessionToken) {
   });
 }
 
-}
+}  // namespace
 
 }  // namespace local
 }  // namespace firestore

Firestore/core/test/unit/local/globals_cache_test.h

@@ -14,7 +14,6 @@
  * limitations under the License.
  */
 
-
 #ifndef FIRESTORE_CORE_TEST_UNIT_LOCAL_GLOBALS_CACHE_TEST_H
 #define FIRESTORE_CORE_TEST_UNIT_LOCAL_GLOBALS_CACHE_TEST_H