Avoid passing temporary to get_ref_default()

Summary: `get_ref_default()` won't accept temporary default values after D4768769.

Reviewed By: yfeldblum

Differential Revision: D4769043

fbshipit-source-id: 46b039e13b80721922f69036c6cc02f5642a26f9

diff --git a/folly/io/async/Request.cpp b/folly/io/async/Request.cpp
index 0c56d200aaec9cb0c333f12a14f1488dd9011122..e82b16fdf5df3b97f2221957eaff15516c24a8d1 100644 (file)
--- a/folly/io/async/Request.cpp
+++ b/folly/io/async/Request.cpp
@@ -61,12 +61,14 @@ bool RequestContext::hasContextData(const std::string& val) const {
 }
 
 RequestData* RequestContext::getContextData(const std::string& val) {
-  return get_ref_default(*data_.rlock(), val, nullptr).get();
+  const std::unique_ptr<RequestData> dflt{nullptr};
+  return get_ref_default(*data_.rlock(), val, dflt).get();
 }
 
 const RequestData* RequestContext::getContextData(
     const std::string& val) const {
-  return get_ref_default(*data_.rlock(), val, nullptr).get();
+  const std::unique_ptr<RequestData> dflt{nullptr};
+  return get_ref_default(*data_.rlock(), val, dflt).get();
 }
 
 void RequestContext::onSet() {