From 02ca9b0b4f554868b533d2f6526a480ea1e7d035 Mon Sep 17 00:00:00 2001 From: Richard Barnes Date: Sat, 9 Mar 2024 06:15:07 -0800 Subject: [PATCH] Remove "using namespace" dep in ubs/admin_api/policy_engine/internal/auth/UBSPolicyAclCheckerSet.h + 5 Summary: The files modified in this diff rely on `using namespace` in the global namespace. In this diff we break that reliance by adding appropriate namespace qualifiers. Landing this diff helps us onboard our code to the `-Wheader-hygiene` warning flag. D54681732 aggregates many of these files and D54681798 removes the associated headers; however, I have broken D54681732 up to make landing and rollbacks safer. Reviewed By: palmje Differential Revision: D54692601 fbshipit-source-id: dbd641443317e31e11902fd45d9bddcfa7c086d5 --- velox/common/memory/SharedArbitrator.cpp | 2 ++ 1 file changed, 2 insertions(+) diff --git a/velox/common/memory/SharedArbitrator.cpp b/velox/common/memory/SharedArbitrator.cpp index fe0938802cc7..a3d22a76a92f 100644 --- a/velox/common/memory/SharedArbitrator.cpp +++ b/velox/common/memory/SharedArbitrator.cpp @@ -28,6 +28,8 @@ using facebook::velox::common::testutil::TestValue; namespace facebook::velox::memory { +using namespace facebook::velox::memory; + namespace { // Returns the max capacity to grow of memory 'pool'. The calculation is based