commit 5c7a2a007c2553354514a9e719245295dcebd34f
author Bob Beck <bbe@google.com> Mon Nov 20 17:28:21 2023 -0700
committer Boringssl LUCI CQ <boringssl-scoped@luci-project-accounts.iam.gserviceaccount.com> Tue Nov 21 19:09:26 2023 +0000
tree c49f515a7795ab9f97fb073967c9de3af349a9d8
parent fcd464ce97d96fd0278ad3082a8429022ae2c4d7

Clang-format all of pki.

Before making further changes. This pass is without
InsertBraces, will follow up with InsertBraces
for separate review

Bug: 659
Change-Id: Ie311dcd932aec5f98c16573f9326b1918a639adf
Reviewed-on: https://boringssl-review.googlesource.com/c/boringssl/+/64067
Commit-Queue: David Benjamin <davidben@google.com>
Auto-Submit: Bob Beck <bbe@google.com>
Reviewed-by: David Benjamin <davidben@google.com>

pki/crl_unittest.cc

diff --git a/pki/crl_unittest.cc b/pki/crl_unittest.cc
index a2479d4..b9be70f 100644
--- a/pki/crl_unittest.cc
+++ b/pki/crl_unittest.cc
@@ -6,12 +6,12 @@
 
 #include <string_view>
 
+#include <gtest/gtest.h>
+#include <openssl/pool.h>
 #include "cert_errors.h"
 #include "parsed_certificate.h"
 #include "string_util.h"
 #include "test_helpers.h"
-#include <gtest/gtest.h>
-#include <openssl/pool.h>
 
 namespace bssl {
 
@@ -27,12 +27,13 @@
     std::string_view data) {
   CertErrors errors;
   return ParsedCertificate::Create(
-      bssl::UniquePtr<CRYPTO_BUFFER>(CRYPTO_BUFFER_new(
-          reinterpret_cast<const uint8_t*>(data.data()), data.size(), nullptr)),
+      bssl::UniquePtr<CRYPTO_BUFFER>(
+          CRYPTO_BUFFER_new(reinterpret_cast<const uint8_t *>(data.data()),
+                            data.size(), nullptr)),
       {}, &errors);
 }
 
-class CheckCRLTest : public ::testing::TestWithParam<const char*> {};
+class CheckCRLTest : public ::testing::TestWithParam<const char *> {};
 
 // Test prefix naming scheme:
 //   good = valid CRL, cert affirmatively not revoked
@@ -40,7 +41,7 @@
 //   bad = valid CRL, but cert status is unknown (cases like unhandled features,
 //           mismatching issuer or signature, etc)
 //   invalid = corrupt or violates some spec requirement
-constexpr char const* kTestParams[] = {
+constexpr char const *kTestParams[] = {
     "good.pem",
     "good_issuer_name_normalization.pem",
     "good_issuer_no_keyusage.pem",
@@ -111,7 +112,7 @@
 
 struct PrintTestName {
   std::string operator()(
-      const testing::TestParamInfo<const char*>& info) const {
+      const testing::TestParamInfo<const char *> &info) const {
     std::string_view name(info.param);
     // Strip ".pem" from the end as GTest names cannot contain period.
     name.remove_suffix(4);
@@ -119,9 +120,7 @@
   }
 };
 
-INSTANTIATE_TEST_SUITE_P(All,
-                         CheckCRLTest,
-                         ::testing::ValuesIn(kTestParams),
+INSTANTIATE_TEST_SUITE_P(All, CheckCRLTest, ::testing::ValuesIn(kTestParams),
                          PrintTestName());
 
 TEST_P(CheckCRLTest, FromFile) {
@@ -162,7 +161,7 @@
   // TODO(https://crbug.com/749276): This seems slightly hacky. Maybe the
   // distribution point to use should be specified separately in the test PEM?
   ParsedDistributionPoint fake_cert_dp;
-  ParsedDistributionPoint* cert_dp = &fake_cert_dp;
+  ParsedDistributionPoint *cert_dp = &fake_cert_dp;
   std::vector<ParsedDistributionPoint> distribution_points;
   ParsedExtension crl_dp_extension;
   if (cert->GetExtension(der::Input(kCrlDistributionPointsOid),
@@ -206,4 +205,4 @@
 
 }  // namespace
 
-}  // namespace net
+}  // namespace bssl