s/setCallback_/setCallback/ for detail::State
authorHans Fugal <fugalh@fb.com>
Mon, 30 Jun 2014 20:34:42 +0000 (13:34 -0700)
committerTudor Bosman <tudorb@fb.com>
Mon, 7 Jul 2014 15:42:09 +0000 (08:42 -0700)
Summary: This isn't really an internal-to-this-class method. :-/

Test Plan: builds

Reviewed By: hannesr@fb.com

Subscribers: jsedgwick, net-systems@, fugalh, exa

FB internal diff: D1412008

folly/wangle/Future-inl.h
folly/wangle/detail/State.h

index 27af013e757e3b5958390b63b409dcd18f43bb96..c3002f48e15eea54bc55f8dbbba3f56d6e1dab15 100644 (file)
@@ -59,7 +59,7 @@ template <class T>
 template <class F>
 void Future<T>::setCallback_(F&& func) {
   throwIfInvalid();
-  state_->setCallback_(std::move(func));
+  state_->setCallback(std::move(func));
   state_ = nullptr;
 }
 
index b971a18897afbbe7d94a2ec3c5751503753a8faf..bf2a34347afdae3cd2fe031f2c54398f19ca2bfe 100644 (file)
@@ -47,9 +47,9 @@ class State {
   }
 
   template <typename F>
-  void setCallback_(F func) {
+  void setCallback(F func) {
     if (callback_) {
-      throw std::logic_error("setCallback_ called twice");
+      throw std::logic_error("setCallback called twice");
     }
 
     callback_ = std::move(func);