Speed up SetTimestamp benchmark. (#402)

g-easy · web-flow · commit 5ff1563ce08b · 2019-10-30T15:33:44.000+11:00
Move the proto constructor out of the loop.
diff --git a/opencensus/common/internal/timestamp_benchmark.cc b/opencensus/common/internal/timestamp_benchmark.cc
@@ -22,8 +22,8 @@ namespace {
 
 void BM_SetTimestamp(benchmark::State& state) {
   absl::Time t = absl::Now();
+  google::protobuf::Timestamp proto;
   for (auto _ : state) {
-    google::protobuf::Timestamp proto;
     ::opencensus::common::SetTimestamp(t, &proto);
   }
 }

Original file line number	Diff line number	Diff line change
`@@ -22,8 +22,8 @@ namespace {`
`22`	`22`
`23`	`23`	`void BM_SetTimestamp(benchmark::State& state) {`
`24`	`24`	`absl::Time t = absl::Now();`
	`25`	`+ google::protobuf::Timestamp proto;`
`25`	`26`	`for (auto _ : state) {`
`26`		`- google::protobuf::Timestamp proto;`
`27`	`27`	`::opencensus::common::SetTimestamp(t, &proto);`
`28`	`28`	`}`
`29`	`29`	`}`