fix local variable shadow
authorShijin Kong <shikong@fb.com>
Mon, 8 Dec 2014 18:03:30 +0000 (10:03 -0800)
committerDave Watson <davejwatson@fb.com>
Thu, 11 Dec 2014 16:01:10 +0000 (08:01 -0800)
Summary:
D1720014 line 183 in HHWheelTimer.h broke proxygen/facebook/httpclient clang build:

./folly/io/async/HHWheelTimer.h:183:17: error: declaration shadows a local variable [-Werror,-Wshadow]
Wrapper(F fn) : fn_(std::move(fn)) {}
^
./folly/io/async/HHWheelTimer.h:181:28: note: previous declaration is here
void scheduleTimeoutFn(F fn, std::chrono::milliseconds timeout) {

I renamed the `fn` in `Wrapper` to `f`.

Test Plan: pass

Reviewed By: seanc@fb.com

Subscribers: njormrod, folly-diffs@, subodh, seanc, kmdent, fma

FB internal diff: D1724835

Signature: t1:1724835:1418052896:d9a72230d4fd1a4a734a33142966fe796f4fd362

folly/io/async/HHWheelTimer.h

index 062544b5ef16738958796ca334095f93173e0ce5..7c6db8a0b428073e0e30ef4ad9a449758d380a01 100644 (file)
@@ -180,7 +180,7 @@ class HHWheelTimer : private folly::AsyncTimeout,
   template <class F>
   void scheduleTimeoutFn(F fn, std::chrono::milliseconds timeout) {
     struct Wrapper : Callback {
-      Wrapper(F fn) : fn_(std::move(fn)) {}
+      Wrapper(F f) : fn_(std::move(f)) {}
       void timeoutExpired() noexcept override {
         try {
           fn_();