Skip to content

Commit

Permalink
HTE lint-ignore message cleanup at fbcode (#8799)
Browse files Browse the repository at this point in the history
Summary:
Pull Request resolved: facebook/hhvm#8799

Pull Request resolved: facebook/sapling#72

Pull Request resolved: facebook/openr#82

Pull Request resolved: facebook/fbthrift#410

The diff removes HOWTOEVEN from all lint-ignore messages at fbcode

Reviewed By: dkgi

Differential Revision: D26278830

fbshipit-source-id: ebb4ea405f6249b28333f5594fa0770a2667267b
  • Loading branch information
ivanmurashko authored and facebook-github-bot committed Feb 7, 2021
1 parent ccf61c6 commit 82fbf2d
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 9 deletions.
4 changes: 2 additions & 2 deletions fboss/agent/hw/bcm/BcmEcmpUtils.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ getEcmpGroupInHw(const BcmSwitch* hw, bcm_if_t ecmp, int sizeInSw) {
existing.ecmp_intf = ecmp;
int pathsInHwCount;
if (hw->getPlatform()->getAsic()->isSupported(HwAsic::Feature::HSDK)) {
// @lint-ignore HOWTOEVEN CLANGTIDY CArray
// @lint-ignore CLANGTIDY
bcm_l3_ecmp_member_t pathsInHw[sizeInSw];
bcm_l3_ecmp_get(
hw->getUnit(), &existing, sizeInSw, pathsInHw, &pathsInHwCount);
Expand All @@ -39,7 +39,7 @@ getEcmpGroupInHw(const BcmSwitch* hw, bcm_if_t ecmp, int sizeInSw) {
}
}
} else {
// @lint-ignore HOWTOEVEN CLANGTIDY CArray
// @lint-ignore CLANGTIDY
bcm_if_t pathsInHw[sizeInSw];
bcm_l3_egress_ecmp_get(
hw->getUnit(), &existing, sizeInSw, pathsInHw, &pathsInHwCount);
Expand Down
14 changes: 7 additions & 7 deletions fboss/agent/hw/bcm/BcmEgress.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -328,7 +328,7 @@ void BcmEcmpEgress::program() {
if (ucmpSupported_) {
obj.ecmp_group_flags = BCM_L3_ECMP_MEMBER_WEIGHTED;
}
// @lint-ignore HOWTOEVEN CLANGTIDY CArray
// @lint-ignore CLANGTIDY
bcm_l3_ecmp_member_t ecmpMemberArray[numPaths];
auto idx = 0;
for (const auto& path : egressId2Weight_) {
Expand Down Expand Up @@ -364,7 +364,7 @@ void BcmEcmpEgress::program() {
createWideEcmpEntry(numPaths);
return;
} else {
// @lint-ignore HOWTOEVEN CLANGTIDY
// @lint-ignore CLANGTIDY
bcm_if_t pathsArray[numPaths];
auto index = 0;
for (const auto& path : egressId2Weight_) {
Expand Down Expand Up @@ -576,7 +576,7 @@ bool BcmEcmpEgress::removeEgressIdHwNotLocked(
ret = bcm_l3_ecmp_get(unit, &existing, 0, nullptr, &totalMembersInHw);
bcmCheckError(ret, "Unable to get ecmp entry ", ecmpId);
if (totalMembersInHw > 0) {
// @lint-ignore HOWTOEVEN CLANGTIDY CArray
// @lint-ignore CLANGTIDY
bcm_l3_ecmp_member_t membersInHw[totalMembersInHw];
ret = bcm_l3_ecmp_get(
unit, &existing, totalMembersInHw, membersInHw, &totalMembersInHw);
Expand Down Expand Up @@ -705,7 +705,7 @@ bool BcmEcmpEgress::addEgressIdHwLocked(
existing.ecmp_intf = ecmpId;
int countInHw = 0;
int ret;
// @lint-ignore HOWTOEVEN CLANGTIDY CArray
// @lint-ignore CLANGTIDY
bcm_l3_ecmp_member_t membersInHw[numPaths];
int totalMembersInHw = -1;
int memberIndex = -1;
Expand All @@ -727,7 +727,7 @@ bool BcmEcmpEgress::addEgressIdHwLocked(
}
}
} else {
// @lint-ignore HOWTOEVEN CLANGTIDY CArray
// @lint-ignore CLANGTIDY
// In WideECMP case, HW will have kMaxWeightedEcmpPaths entries
bcm_if_t pathsInHw[kMaxWeightedEcmpPaths];
int totalPathsInHw;
Expand Down Expand Up @@ -930,7 +930,7 @@ void BcmEcmpEgress::programWideEcmp(
bcm_l3_egress_ecmp_t obj;
bcm_l3_egress_ecmp_t_init(&obj);
// weighted path size needs to be power of 2
// @lint-ignore HOWTOEVEN CLANGTIDY
// @lint-ignore CLANGTIDY
bcm_if_t pathsArray[kMaxWeightedEcmpPaths];
auto index = 0;
if (id != INVALID) {
Expand Down Expand Up @@ -974,7 +974,7 @@ bool BcmEcmpEgress::rebalanceWideEcmpEntry(
int unit,
EgressId ecmpId,
std::pair<EgressId, int> toRemove) {
// @lint-ignore HOWTOEVEN CLANGTIDY
// @lint-ignore CLANGTIDY
bcm_if_t pathsInHw[kMaxWeightedEcmpPaths];
int totalPathsInHw;
bcm_l3_egress_ecmp_t obj;
Expand Down

0 comments on commit 82fbf2d

Please sign in to comment.