Fix a bug in ObserverManager destructor
authorAndrii Grynenko <andrii@fb.com>
Tue, 13 Sep 2016 00:53:52 +0000 (17:53 -0700)
committerFacebook Github Bot 3 <facebook-github-bot-3-bot@fb.com>
Tue, 13 Sep 2016 01:08:31 +0000 (18:08 -0700)
Summary: facepaw

Reviewed By: ckwalsh

Differential Revision: D3853363

fbshipit-source-id: f863a3e3ae830256f5cff6c56f23254ff6fdee2a

folly/experimental/observer/detail/ObserverManager.cpp

index e29522ab92f679f1c7a9e769e9f6d64b0f5e4bc2..9f8b7b7f1afa83fcdacf0f3e0cca41464667941f 100644 (file)
@@ -147,8 +147,8 @@ ObserverManager::ObserverManager() {
 ObserverManager::~ObserverManager() {
   // Destroy NextQueue, before the rest of this object, since it expects
   // ObserverManager to be alive.
-  nextQueue_.release();
-  currentQueue_.release();
+  nextQueue_.reset();
+  currentQueue_.reset();
 }
 
 void ObserverManager::scheduleCurrent(Function<void()> task) {