From cb3f8bf36d9e8fbb0c8d9f30fd6dbc64ac7d4a61 Mon Sep 17 00:00:00 2001 From: Hans Fugal Date: Fri, 5 Dec 2014 09:46:56 -0800 Subject: [PATCH] More logging in HHWheelTimer::scheduleTimeoutFn Summary: This was supposed to be an update to D1720014 but I screwed up with arc. Test Plan: run the test by hand and see the output Reviewed By: jsedgwick@fb.com Subscribers: exa, njormrod, folly-diffs@ FB internal diff: D1721951 Signature: t1:1721951:1417801009:72a27ec2ca473a996785ff9cae48bd51a43d9045 --- folly/io/async/HHWheelTimer.h | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/folly/io/async/HHWheelTimer.h b/folly/io/async/HHWheelTimer.h index eff837e5..062544b5 100644 --- a/folly/io/async/HHWheelTimer.h +++ b/folly/io/async/HHWheelTimer.h @@ -185,8 +185,11 @@ class HHWheelTimer : private folly::AsyncTimeout, try { fn_(); } catch (std::exception const& e) { - LOG(ERROR) << e.what(); - } catch (...) { } + LOG(ERROR) << "HHWheelTimer timeout callback threw an exception: " + << e.what(); + } catch (...) { + LOG(ERROR) << "HHWheelTimer timeout callback threw a non-exception."; + } delete this; } F fn_; -- 2.34.1