Skip to content

Commit b1ef59a

Browse files
r-barnesfacebook-github-bot
authored andcommitted
Remove unused exception parameter from gloo/rendezvous/context.cc
Summary: `-Wunused-exception-parameter` has identified an unused exception parameter. This diff removes it. This: ``` try { ... } catch (exception& e) { // no use of e } ``` should instead be written as ``` } catch (exception&) { ``` If the code compiles, this is safe to land. Reviewed By: dmm-fb Differential Revision: D52957891 fbshipit-source-id: 1b107cbfea042a6dc3a9195d550110b0a9c72841
1 parent d1c74a7 commit b1ef59a

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

gloo/rendezvous/context.cc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ ContextFactory::ContextFactory(std::shared_ptr<::gloo::Context> backingContext)
4646
GLOO_ENFORCE(
4747
backingContext_->getPair(i) != nullptr,
4848
"Missing pair in backing context");
49-
} catch(std::out_of_range& e) {
49+
} catch(std::out_of_range&) {
5050
GLOO_THROW("Backing context not fully connected");
5151
}
5252
}

0 commit comments

Comments
 (0)