Commit 816b97a7 authored by Alexey Spiridonov's avatar Alexey Spiridonov Committed by Noam Lerner

Fix logFormat

Summary: Ooops during port ;)

Test Plan: unit test

Reviewed By: agoder@fb.com

Subscribers: trunkagent, folly-diffs@, yfeldblum

FB internal diff: D1905639

Signature: t1:1905639:1426171227:a6e415046a85f098ac090f069476204fb0367c42
parent 6001c511
...@@ -94,7 +94,7 @@ public: ...@@ -94,7 +94,7 @@ public:
template<typename... Args> template<typename... Args>
double logFormat(Args&&... args) { double logFormat(Args&&... args) {
auto now = Clock::now(); auto now = Clock::now();
return logImpl(now, format(std::forward<Args>(args)...)); return logImpl(now, format(std::forward<Args>(args)...).str());
} }
private: private:
......
...@@ -50,6 +50,9 @@ TEST(TestAutoTimer, HandleBasic) { ...@@ -50,6 +50,9 @@ TEST(TestAutoTimer, HandleBasic) {
timer.log("foo"); timer.log("foo");
ASSERT_EQ("foo", StubLogger::m); ASSERT_EQ("foo", StubLogger::m);
ASSERT_EQ(2, StubLogger::t); ASSERT_EQ(2, StubLogger::t);
timer.logFormat("bar {}", 5e-2);
ASSERT_EQ("bar 0.05", StubLogger::m);
ASSERT_EQ(0, StubLogger::t);
} }
TEST(TestAutoTimer, HandleLogOnDestruct) { TEST(TestAutoTimer, HandleLogOnDestruct) {
......
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