pragma once (#15274)

Commit Message: Add pragma once almost every header. Risk Level: LOW Testing: self-contained Docs Changes: Release Notes: Platform Specific Features: [Optional Runtime guard:] Part of #15264 [Optional Deprecated:] [Optional API Considerations:] Additional Description: Signed-off-by: Yuchen Dai <[email protected]>
envoyproxy · Mar 3, 2021 · e9f025a · e9f025a
1 parent 5002661
commit e9f025a
Show file tree

Hide file tree

Showing 44 changed files with 88 additions and 0 deletions.
diff --git a/source/common/common/documentation_url.h b/source/common/common/documentation_url.h
@@ -1,3 +1,5 @@
+#pragma once
+
 namespace Envoy {
 
 // TODO(ggreenway): replace 'latest' with the current version, pulled from the VERSION file at

diff --git a/source/common/grpc/google_async_client_cache.h b/source/common/grpc/google_async_client_cache.h
@@ -1,3 +1,5 @@
+#pragma once
+
 #include <memory>
 
 #include "envoy/grpc/async_client_manager.h"

diff --git a/source/common/stats/scope_prefixer.h b/source/common/stats/scope_prefixer.h
@@ -1,3 +1,5 @@
+#pragma once
+
 #include "envoy/stats/scope.h"
 
 #include "common/stats/symbol_table_impl.h"

diff --git a/source/common/upstream/load_stats_reporter.h b/source/common/upstream/load_stats_reporter.h
@@ -1,3 +1,5 @@
+#pragma once
+
 #include "envoy/event/dispatcher.h"
 #include "envoy/service/load_stats/v3/lrs.pb.h"
 #include "envoy/stats/scope.h"

diff --git a/source/extensions/common/sqlutils/sqlutils.h b/source/extensions/common/sqlutils/sqlutils.h
@@ -1,3 +1,5 @@
+#pragma once
+
 #include "common/protobuf/utility.h"
 
 #include "include/sqlparser/SQLParser.h"

diff --git a/source/extensions/filters/network/http_connection_manager/dependency_manager.h b/source/extensions/filters/network/http_connection_manager/dependency_manager.h
@@ -1,3 +1,5 @@
+#pragma once
+
 #include "envoy/extensions/filters/common/dependency/v3/dependency.pb.h"
 
 namespace Envoy {

diff --git a/source/extensions/filters/network/postgres_proxy/postgres_message.h b/source/extensions/filters/network/postgres_proxy/postgres_message.h
@@ -1,3 +1,5 @@
+#pragma once
+
 #include "common/buffer/buffer_impl.h"
 
 #include "absl/strings/str_cat.h"

diff --git a/source/extensions/io_socket/user_space/config.h b/source/extensions/io_socket/user_space/config.h
@@ -1,3 +1,5 @@
+#pragma once
+
 namespace Envoy {
 namespace Extensions {
 namespace IoSocket {

diff --git a/source/extensions/quic_listeners/quiche/codec_impl.h b/source/extensions/quic_listeners/quiche/codec_impl.h
@@ -1,3 +1,5 @@
+#pragma once
+
 #include "envoy/http/codec.h"
 #include "envoy/registry/registry.h"
 

diff --git a/source/extensions/quic_listeners/quiche/envoy_quic_server_connection.h b/source/extensions/quic_listeners/quiche/envoy_quic_server_connection.h
@@ -1,3 +1,5 @@
+#pragma once
+
 #include "envoy/network/listener.h"
 
 #include "server/connection_handler_impl.h"

diff --git a/source/extensions/quic_listeners/quiche/platform/string_utils.h b/source/extensions/quic_listeners/quiche/platform/string_utils.h
@@ -1,3 +1,5 @@
+#pragma once
+
 // NOLINT(namespace-envoy)
 //
 // This file is part of the QUICHE platform implementation, and is not to be

diff --git a/source/extensions/quic_listeners/quiche/quic_io_handle_wrapper.h b/source/extensions/quic_listeners/quiche/quic_io_handle_wrapper.h
@@ -1,3 +1,5 @@
+#pragma once
+
 #include <chrono>
 #include <iostream>
 

diff --git a/source/extensions/retry/host/omit_canary_hosts/config.h b/source/extensions/retry/host/omit_canary_hosts/config.h
@@ -1,3 +1,5 @@
+#pragma once
+
 #include "envoy/extensions/retry/host/omit_canary_hosts/v3/omit_canary_hosts.pb.validate.h"
 #include "envoy/upstream/retry.h"
 

diff --git a/source/extensions/tracers/skywalking/skywalking_stats.h b/source/extensions/tracers/skywalking/skywalking_stats.h
@@ -1,3 +1,5 @@
+#pragma once
+
 #include "envoy/stats/stats_macros.h"
 
 namespace Envoy {

diff --git a/source/extensions/transport_sockets/starttls/starttls_socket.h b/source/extensions/transport_sockets/starttls/starttls_socket.h
@@ -1,3 +1,5 @@
+#pragma once
+
 #include "envoy/extensions/transport_sockets/starttls/v3/starttls.pb.h"
 #include "envoy/extensions/transport_sockets/starttls/v3/starttls.pb.validate.h"
 #include "envoy/network/transport_socket.h"

diff --git a/test/common/formatter/command_extension.h b/test/common/formatter/command_extension.h
@@ -1,3 +1,5 @@
+#pragma once
+
 #include <string>
 
 #include "envoy/config/typed_config.h"

diff --git a/test/common/http/http2/frame_replay.h b/test/common/http/http2/frame_replay.h
@@ -1,3 +1,5 @@
+#pragma once
+
 #include <cstdint>
 #include <memory>
 #include <vector>

diff --git a/test/common/network/udp_listener_impl_test_base.h b/test/common/network/udp_listener_impl_test_base.h
@@ -1,3 +1,5 @@
+#pragma once
+
 #include <cstddef>
 #include <memory>
 #include <string>

diff --git a/test/common/upstream/health_check_fuzz_test_utils.h b/test/common/upstream/health_check_fuzz_test_utils.h
@@ -1,3 +1,5 @@
+#pragma once
+
 #include <vector>
 
 #include "common/upstream/health_checker_impl.h"

diff --git a/test/common/upstream/load_balancer_fuzz_base.h b/test/common/upstream/load_balancer_fuzz_base.h
@@ -1,3 +1,5 @@
+#pragma once
+
 #include "envoy/config/cluster/v3/cluster.pb.h"
 
 #include "common/upstream/load_balancer_impl.h"

diff --git a/test/common/upstream/zone_aware_load_balancer_fuzz_base.h b/test/common/upstream/zone_aware_load_balancer_fuzz_base.h
@@ -1,3 +1,5 @@
+#pragma once
+
 #include "test/mocks/upstream/priority_set.h"
 
 #include "load_balancer_fuzz_base.h"

diff --git a/test/extensions/clusters/redis/mocks.h b/test/extensions/clusters/redis/mocks.h
@@ -1,3 +1,5 @@
+#pragma once
+
 #include "envoy/extensions/clusters/redis/v3/redis_cluster.pb.h"
 #include "envoy/extensions/clusters/redis/v3/redis_cluster.pb.validate.h"
 #include "envoy/extensions/filters/network/redis_proxy/v3/redis_proxy.pb.h"

diff --git a/test/extensions/common/tap/common.h b/test/extensions/common/tap/common.h
@@ -1,3 +1,5 @@
+#pragma once
+
 #include "envoy/data/tap/v3/wrapper.pb.h"
 
 #include "common/protobuf/utility.h"

diff --git a/...tensions/filters/http/adaptive_concurrency/adaptive_concurrency_filter_integration_test.h b/...tensions/filters/http/adaptive_concurrency/adaptive_concurrency_filter_integration_test.h
@@ -1,3 +1,5 @@
+#pragma once
+
 #include "test/integration/http_integration.h"
 #include "test/test_common/simulated_time_system.h"
 #include "test/test_common/utility.h"

diff --git a/test/extensions/filters/http/common/fuzz/uber_filter.h b/test/extensions/filters/http/common/fuzz/uber_filter.h
@@ -1,3 +1,5 @@
+#pragma once
+
 #include "test/extensions/filters/http/common/fuzz/http_filter_fuzzer.h"
 #include "test/fuzz/utility.h"
 #include "test/mocks/buffer/mocks.h"

diff --git a/test/extensions/filters/http/tap/common.h b/test/extensions/filters/http/tap/common.h
@@ -1,3 +1,5 @@
+#pragma once
+
 #include "extensions/filters/http/tap/tap_config.h"
 
 #include "gmock/gmock.h"

diff --git a/test/extensions/filters/listener/common/fuzz/listener_filter_fakes.h b/test/extensions/filters/listener/common/fuzz/listener_filter_fakes.h
@@ -1,3 +1,5 @@
+#pragma once
+
 #include "common/api/os_sys_calls_impl.h"
 #include "common/network/io_socket_handle_impl.h"
 

diff --git a/test/extensions/filters/listener/common/fuzz/listener_filter_fuzzer.h b/test/extensions/filters/listener/common/fuzz/listener_filter_fuzzer.h
@@ -1,3 +1,5 @@
+#pragma once
+
 #include "envoy/network/filter.h"
 
 #include "test/extensions/filters/listener/common/fuzz/listener_filter_fakes.h"

diff --git a/test/extensions/filters/network/common/fuzz/uber_readfilter.h b/test/extensions/filters/network/common/fuzz/uber_readfilter.h
@@ -1,3 +1,5 @@
+#pragma once
+
 #include "envoy/network/filter.h"
 
 #include "common/protobuf/protobuf.h"

diff --git a/test/extensions/filters/network/common/fuzz/uber_writefilter.h b/test/extensions/filters/network/common/fuzz/uber_writefilter.h
@@ -1,3 +1,5 @@
+#pragma once
+
 #include "envoy/network/filter.h"
 
 #include "common/protobuf/protobuf.h"

diff --git a/test/extensions/filters/network/common/fuzz/utils/fakes.h b/test/extensions/filters/network/common/fuzz/utils/fakes.h
@@ -1,3 +1,5 @@
+#pragma once
+
 #include "test/mocks/server/factory_context.h"
 
 namespace Envoy {

diff --git a/test/extensions/filters/network/http_connection_manager/config_test_base.h b/test/extensions/filters/network/http_connection_manager/config_test_base.h
@@ -1,3 +1,5 @@
+#pragma once
+
 #include "common/filter/http/filter_config_discovery_impl.h"
 #include "common/http/date_provider_impl.h"
 #include "common/network/address_impl.h"

diff --git a/test/extensions/filters/network/postgres_proxy/postgres_test_utils.h b/test/extensions/filters/network/postgres_proxy/postgres_test_utils.h
@@ -1,3 +1,5 @@
+#pragma once
+
 #include "common/buffer/buffer_impl.h"
 
 namespace Envoy {

diff --git a/test/extensions/quic_listeners/quiche/test_proof_source.h b/test/extensions/quic_listeners/quiche/test_proof_source.h
@@ -1,3 +1,5 @@
+#pragma once
+
 #if defined(__GNUC__)
 #pragma GCC diagnostic push
 #pragma GCC diagnostic ignored "-Wunused-parameter"

diff --git a/test/extensions/quic_listeners/quiche/test_proof_verifier.h b/test/extensions/quic_listeners/quiche/test_proof_verifier.h
@@ -1,3 +1,5 @@
+#pragma once
+
 #include "extensions/quic_listeners/quiche/envoy_quic_proof_verifier_base.h"
 
 namespace Envoy {

diff --git a/test/extensions/quic_listeners/quiche/test_utils.h b/test/extensions/quic_listeners/quiche/test_utils.h
@@ -1,3 +1,5 @@
+#pragma once
+
 #include "extensions/quic_listeners/quiche/quic_filter_manager_connection_impl.h"
 
 #if defined(__GNUC__)

diff --git a/test/fuzz/random.h b/test/fuzz/random.h
@@ -1,3 +1,5 @@
+#pragma once
+
 #include <memory>
 #include <random>
 #include <utility>

diff --git a/test/integration/clusters/custom_static_cluster.h b/test/integration/clusters/custom_static_cluster.h
@@ -1,3 +1,5 @@
+#pragma once
+
 #include <chrono>
 #include <list>
 #include <string>

diff --git a/test/mocks/local_reply/mocks.h b/test/mocks/local_reply/mocks.h
@@ -1,3 +1,5 @@
+#pragma once
+
 #include "common/local_reply/local_reply.h"
 
 #include "gmock/gmock.h"

diff --git a/test/server/config_validation/xds_verifier.h b/test/server/config_validation/xds_verifier.h
@@ -1,3 +1,5 @@
+#pragma once
+
 #include "envoy/admin/v3/config_dump.pb.h"
 #include "envoy/common/exception.h"
 #include "envoy/config/cluster/v3/cluster.pb.h"

diff --git a/test/server/listener_manager_impl_test.h b/test/server/listener_manager_impl_test.h
@@ -1,3 +1,5 @@
+#pragma once
+
 #include <memory>
 
 #include "envoy/admin/v3/config_dump.pb.h"

diff --git a/test/test_common/file_system_for_test.h b/test/test_common/file_system_for_test.h
@@ -1,3 +1,5 @@
+#pragma once
+
 #include "envoy/filesystem/filesystem.h"
 
 namespace Envoy {

diff --git a/test/test_common/thread_factory_for_test.h b/test/test_common/thread_factory_for_test.h
@@ -1,3 +1,5 @@
+#pragma once
+
 #include "envoy/thread/thread.h"
 
 namespace Envoy {

diff --git a/test/test_common/wasm_base.h b/test/test_common/wasm_base.h
@@ -1,3 +1,5 @@
+#pragma once
+
 #include <cstdio>
 
 #include "envoy/extensions/wasm/v3/wasm.pb.validate.h"