Skip to content

Commit

Permalink
Remove "using namespace" dep in ubs/admin_api/policy_engine/internal/…
Browse files Browse the repository at this point in the history
…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
  • Loading branch information
r-barnes authored and facebook-github-bot committed Mar 9, 2024
1 parent 08d4982 commit 02ca9b0
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions velox/common/memory/SharedArbitrator.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down

0 comments on commit 02ca9b0

Please sign in to comment.