Remove some not-resolving make_pair() invocations.

There's nothing wrong with make_pair(), but there is some type
resolution issue in these instances, at least with GCC 4.8.

Signed-off-by: Matt Benjamin <matt@linuxbox.com>

diff --git a/src/rgw/rgw_main.cc b/src/rgw/rgw_main.cc
index 0290a9dd3e2a3b672a188f9df8ec0239c2c4e038..c2057cde7b1a0f6ee9adf8db13d9208b936d9d18 100644 (file)
--- a/src/rgw/rgw_main.cc
+++ b/src/rgw/rgw_main.cc
@@ -1104,7 +1104,7 @@ int main(int argc, const char **argv)
     configs.push_back(config);
 
     string framework = config->get_framework();
-    fe_map.insert(make_pair<string, RGWFrontendConfig *>(framework, config));
+    fe_map.insert(pair<string, RGWFrontendConfig*>(framework, config));
   }
 
   list<RGWFrontend *> fes;

diff --git a/src/rgw/rgw_rest_client.cc b/src/rgw/rgw_rest_client.cc
index 256db77aba77148d01c598f09fb80d9b266b9818..3b8a58d49bb6929c62d67487dfbdfeb0bf86ee6e 100644 (file)
--- a/src/rgw/rgw_rest_client.cc
+++ b/src/rgw/rgw_rest_client.cc
@@ -89,7 +89,7 @@ int RGWRESTSimpleRequest::execute(RGWAccessKey& key, const char *method, const c
 
   string date_str;
   get_new_date_str(cct, date_str);
-  headers.push_back(make_pair<string, string>("HTTP_DATE", date_str));
+  headers.push_back(pair<string, string>("HTTP_DATE", date_str));
 
   string canonical_header;
   map<string, string> meta_map;
@@ -108,7 +108,7 @@ int RGWRESTSimpleRequest::execute(RGWAccessKey& key, const char *method, const c
 
   ldout(cct, 15) << "generated auth header: " << auth_hdr << dendl;
 
-  headers.push_back(make_pair<string, string>("AUTHORIZATION", auth_hdr));
+  headers.push_back(pair<string, string>("AUTHORIZATION", auth_hdr));
   int r = process(method, new_url.c_str());
   if (r < 0)
     return r;
@@ -221,12 +221,12 @@ int RGWRESTSimpleRequest::forward_request(RGWAccessKey& key, req_info& info, siz
   map<string, string, ltstr_nocase>& m = new_env.get_map();
   map<string, string>::iterator iter;
   for (iter = m.begin(); iter != m.end(); ++iter) {
-    headers.push_back(make_pair<string, string>(iter->first, iter->second));
+    headers.push_back(pair<string, string>(iter->first, iter->second));
   }
 
   map<string, string>& meta_map = new_info.x_meta_map;
   for (iter = meta_map.begin(); iter != meta_map.end(); ++iter) {
-    headers.push_back(make_pair<string, string>(iter->first, iter->second));
+    headers.push_back(pair<string, string>(iter->first, iter->second));
   }
 
   string params_str;
@@ -446,7 +446,7 @@ int RGWRESTStreamWriteRequest::put_obj_init(RGWAccessKey& key, rgw_obj& obj, uin
 
   map<string, string>::iterator iter;
   for (iter = m.begin(); iter != m.end(); ++iter) {
-    headers.push_back(make_pair<string, string>(iter->first, iter->second));
+    headers.push_back(pair<string, string>(iter->first, iter->second));
   }
 
   cb = new RGWRESTStreamOutCB(this);
@@ -585,7 +585,7 @@ int RGWRESTStreamReadRequest::get_obj(RGWAccessKey& key, map<string, string>& ex
   map<string, string, ltstr_nocase>& m = new_env.get_map();
   map<string, string>::iterator iter;
   for (iter = m.begin(); iter != m.end(); ++iter) {
-    headers.push_back(make_pair<string, string>(iter->first, iter->second));
+    headers.push_back(pair<string, string>(iter->first, iter->second));
   }
 
   int r = process(new_info.method, new_url.c_str());

diff --git a/src/rgw/rgw_rest_conn.cc b/src/rgw/rgw_rest_conn.cc
index 32bf0eaacd39001c748801feaa2aeb37b714a393..a02c324532098018811fe7188781eaedb0c01b7e 100644 (file)
--- a/src/rgw/rgw_rest_conn.cc
+++ b/src/rgw/rgw_rest_conn.cc
@@ -34,13 +34,13 @@ int RGWRESTConn::forward(const string& uid, req_info& info, obj_version *objv, s
   if (ret < 0)
     return ret;
   list<pair<string, string> > params;
-  params.push_back(make_pair<string, string>(RGW_SYS_PARAM_PREFIX "uid", uid));
-  params.push_back(make_pair<string, string>(RGW_SYS_PARAM_PREFIX "region", region));
+  params.push_back(pair<string, string>(RGW_SYS_PARAM_PREFIX "uid", uid));
+  params.push_back(pair<string, string>(RGW_SYS_PARAM_PREFIX "region", region));
   if (objv) {
-    params.push_back(make_pair<string, string>(RGW_SYS_PARAM_PREFIX "tag", objv->tag));
+    params.push_back(pair<string, string>(RGW_SYS_PARAM_PREFIX "tag", objv->tag));
     char buf[16];
     snprintf(buf, sizeof(buf), "%lld", (long long)objv->ver);
-    params.push_back(make_pair<string, string>(RGW_SYS_PARAM_PREFIX "ver", buf));
+    params.push_back(pair<string, string>(RGW_SYS_PARAM_PREFIX "ver", buf));
   }
   RGWRESTSimpleRequest req(cct, url, NULL, &params);
   return req.forward_request(key, info, max_response, inbl, outbl);
@@ -61,8 +61,8 @@ int RGWRESTConn::put_obj_init(const string& uid, rgw_obj& obj, uint64_t obj_size
     return ret;
 
   list<pair<string, string> > params;
-  params.push_back(make_pair<string, string>(RGW_SYS_PARAM_PREFIX "uid", uid));
-  params.push_back(make_pair<string, string>(RGW_SYS_PARAM_PREFIX "region", region));
+  params.push_back(pair<string, string>(RGW_SYS_PARAM_PREFIX "uid", uid));
+  params.push_back(pair<string, string>(RGW_SYS_PARAM_PREFIX "region", region));
   *req = new RGWRESTStreamWriteRequest(cct, url, NULL, &params);
   return (*req)->put_obj_init(key, obj, obj_size, attrs);
 }
@@ -84,10 +84,10 @@ int RGWRESTConn::get_obj(const string& uid, req_info *info /* optional */, rgw_o
     return ret;
 
   list<pair<string, string> > params;
-  params.push_back(make_pair<string, string>(RGW_SYS_PARAM_PREFIX "uid", uid));
-  params.push_back(make_pair<string, string>(RGW_SYS_PARAM_PREFIX "region", region));
+  params.push_back(pair<string, string>(RGW_SYS_PARAM_PREFIX "uid", uid));
+  params.push_back(pair<string, string>(RGW_SYS_PARAM_PREFIX "region", region));
   if (prepend_metadata) {
-    params.push_back(make_pair<string, string>(RGW_SYS_PARAM_PREFIX "prepend-metadata", region));
+    params.push_back(pair<string, string>(RGW_SYS_PARAM_PREFIX "prepend-metadata", region));
   }
   *req = new RGWRESTStreamReadRequest(cct, url, cb, NULL, &params);
   map<string, string> extra_headers;