Skip to content

Commit 28d85a4

Browse files
r-barnesfacebook-github-bot
authored andcommitted
Remove unused variables in gloo/transport/ibverbs/device.cc
Summary: LLVM-15 has a warning `-Wunused-but-set-variable` which we treat as an error because it's so often diagnostic of a code issue. Unused variables can compromise readability or, worse, performance. This diff either (a) removes an unused variable and, possibly, it's associated code, or (b) qualifies the variable with `[[maybe_unused]]`, mostly in cases where the variable _is_ used, but, eg, in an `assert` statement that isn't present in production code. - If you approve of this diff, please use the "Accept & Ship" button :-) Reviewed By: danzimm, meyering Differential Revision: D52847976 fbshipit-source-id: 6c797d181b3682613c0ab44689461b0515727306
1 parent 6a4f7ed commit 28d85a4

File tree

1 file changed

+0
-2
lines changed

1 file changed

+0
-2
lines changed

gloo/transport/ibverbs/device.cc

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,8 +25,6 @@ namespace gloo {
2525
namespace transport {
2626
namespace ibverbs {
2727

28-
static const std::chrono::seconds kTimeoutDefault = std::chrono::seconds(30);
29-
3028
// Scope guard for ibverbs device list.
3129
class IbvDevices {
3230
public:

0 commit comments

Comments
 (0)