Skip to content

Commit

Permalink
Remove redundant flag declarations in HwRouteScaleBenchmarkHelpers.h
Browse files Browse the repository at this point in the history
Summary:
Remove flags and include from header instead:

```
DECLARE_bool(json);
DECLARE_bool(dsf_subscribe);
```

Reviewed By: srikrishnagopu

Differential Revision: D61942628

fbshipit-source-id: 824129957069c83bff2c5eced7e4d1a5ceef546b
  • Loading branch information
maxwindiff authored and facebook-github-bot committed Aug 30, 2024
1 parent 40419dc commit 9ec7922
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 3 deletions.
2 changes: 2 additions & 0 deletions fboss/agent/hw/benchmarks/BUCK
Original file line number Diff line number Diff line change
Expand Up @@ -34,9 +34,11 @@ cpp_library(
"HwRouteScaleBenchmarkHelpers.h",
],
exported_deps = [
"//fboss/agent:agent_features",
"//fboss/agent:core",
"//fboss/agent:hw_switch",
"//fboss/agent:utils",
"//fboss/agent/benchmarks:mono_agent_benchmarks",
"//fboss/agent/hw/test:config_factory",
"//fboss/agent/hw/test:hw_switch_ensemble_factory",
"//fboss/agent/test:agent_ensemble",
Expand Down
5 changes: 2 additions & 3 deletions fboss/agent/hw/benchmarks/HwRouteScaleBenchmarkHelpers.h
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,9 @@
*
*/

#include "fboss/agent/AgentFeatures.h"
#include "fboss/agent/HwSwitch.h"
#include "fboss/agent/benchmarks/AgentBenchmarks.h"
#include "fboss/agent/hw/test/ConfigFactory.h"
#include "fboss/agent/hw/test/HwSwitchEnsemble.h"
#include "fboss/agent/hw/test/HwSwitchEnsembleFactory.h"
Expand All @@ -26,9 +28,6 @@

#include "fboss/agent/test/AgentEnsemble.h"

DECLARE_bool(json);
DECLARE_bool(dsf_subscribe);

namespace {
const auto kEcmpWidth = 512;
const auto kEcmpGroup = 8;
Expand Down

0 comments on commit 9ec7922

Please sign in to comment.