Commit f5e28344 authored by Jian He's avatar Jian He Committed by Facebook GitHub Bot

fix some -Werror,-Wsuggest-override violations

Reviewed By: yfeldblum

Differential Revision: D32187345

fbshipit-source-id: d384aac90c5c0c10580824305ce108e7c0b0e577
parent bf2a54fe
...@@ -30,7 +30,7 @@ namespace folly { ...@@ -30,7 +30,7 @@ namespace folly {
/// threadsafe. /// threadsafe.
class DefaultKeepAliveExecutor : public virtual Executor { class DefaultKeepAliveExecutor : public virtual Executor {
public: public:
virtual ~DefaultKeepAliveExecutor() { DCHECK(!keepAlive_); } virtual ~DefaultKeepAliveExecutor() override { DCHECK(!keepAlive_); }
template <typename ExecutorT> template <typename ExecutorT>
static auto getWeakRef(ExecutorT& executor) { static auto getWeakRef(ExecutorT& executor) {
......
...@@ -166,7 +166,7 @@ class AsyncSSLSocket : public AsyncSocket { ...@@ -166,7 +166,7 @@ class AsyncSSLSocket : public AsyncSocket {
sslSocket_(sslSocket), sslSocket_(sslSocket),
dg_(std::move(dg)) {} dg_(std::move(dg)) {}
~DefaultOpenSSLAsyncFinishCallback() { ~DefaultOpenSSLAsyncFinishCallback() override {
pipeReader_->setReadCB(nullptr); pipeReader_->setReadCB(nullptr);
sslSocket_->setAsyncOperationFinishCallback(nullptr); sslSocket_->setAsyncOperationFinishCallback(nullptr);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment