Parcourir la source

[5077] Replaced auto

Francis Dupont il y a 8 ans
Parent
commit
8d65faf798

+ 4 - 2
src/lib/http/post_request_json.cc

@@ -41,11 +41,13 @@ PostHttpRequestJson::getBodyAsJson() {
 
 ConstElementPtr
 PostHttpRequestJson::getJsonElement(const std::string& element_name) {
+    typedef std::map<std::string, ConstElementPtr> MapElementType;
     try {
         ConstElementPtr body = getBodyAsJson();
         if (body) {
-            const std::map<std::string, ConstElementPtr>& map_value = body->mapValue();
-            auto map_element = map_value.find(element_name);
+            const MapElementType& map_value = body->mapValue();
+            MapElementType::const_iterator map_element =
+                map_value.find(element_name);
             if (map_element != map_value.end()) {
                 return (map_element->second);
             }

+ 7 - 4
src/lib/http/request.cc

@@ -53,6 +53,8 @@ HttpRequest::requiresBody() const {
 
 void
 HttpRequest::create() {
+    typedef std::vector<HttpHeaderContext> HeadersVector;
+    typedef std::map<std::string, std::string> HeadersMap;
     try {
         // The RequestParser doesn't validate the method name. Thus, this
         // may throw an exception. But, we're fine with lower case names,
@@ -76,7 +78,7 @@ HttpRequest::create() {
         }
 
         // Copy headers from the context.
-        for (auto header = context_->headers_.begin();
+        for (HeadersVector::iterator header = context_->headers_.begin();
              header != context_->headers_.end();
              ++header) {
             headers_[header->name_] = header->value_;
@@ -84,10 +86,10 @@ HttpRequest::create() {
 
         // Iterate over required headers and check that they exist
         // in the HTTP request.
-        for (auto req_header = required_headers_.begin();
+        for (HeadersMap::iterator req_header = required_headers_.begin();
              req_header != required_headers_.end();
              ++req_header) {
-            auto header = headers_.find(req_header->first);
+            HeadersMap::iterator header = headers_.find(req_header->first);
             if (header == headers_.end()) {
                 isc_throw(BadValue, "required header " << header->first
                           << " not found in the HTTP request");
@@ -153,9 +155,10 @@ HttpRequest::getHttpVersion() const {
 
 std::string
 HttpRequest::getHeaderValue(const std::string& header) const {
+    typedef std::map<std::string, std::string> HeadersMap;
     checkCreated();
 
-    auto header_it = headers_.find(header);
+    HeadersMap::const_iterator header_it = headers_.find(header);
     if (header_it != headers_.end()) {
         return (header_it->second);
     }

+ 3 - 1
src/lib/http/tests/post_request_json_unittests.cc

@@ -99,6 +99,7 @@ TEST_F(PostHttpRequestJsonTest, requireContentLength) {
 // This test verifies that JSON body can be retrieved from the
 // HTTP request.
 TEST_F(PostHttpRequestJsonTest, getBodyAsJson) {
+    typedef std::map<std::string, ConstElementPtr> MapElementType;
     // Create HTTP POST request with JSON body.
     setContextBasics("POST", "/isc/org", std::make_pair(1, 0));
     addHeaderToContext("Content-Length", json_body_.length());
@@ -113,7 +114,8 @@ TEST_F(PostHttpRequestJsonTest, getBodyAsJson) {
 
     // Iterate over JSON values and store them in a simple map.
     std::map<std::string, std::string> config_values;
-    for (auto config_element = json->mapValue().begin();
+    for (MapElementType::const_iterator config_element =
+             json->mapValue().begin();
          config_element != json->mapValue().end();
          ++config_element) {
         ASSERT_FALSE(config_element->first.empty());

+ 3 - 3
src/lib/http/tests/request_parser_unittests.cc

@@ -88,10 +88,10 @@ TEST_F(HttpRequestParserTest, postHttpRequestWithJson) {
     ASSERT_NO_THROW(parser.initModel());
 
     // Simulate receiving HTTP request in chunks.
-    for (auto i = 0; i < http_req.size(); i += http_req.size() / 10) {
-        auto done = false;
+    for (size_t i = 0; i < http_req.size(); i += http_req.size() / 10) {
+        bool done = false;
         // Get the size of the data chunk. 
-        auto chunk = http_req.size() / 10;
+        size_t chunk = http_req.size() / 10;
         // When we're near the end of the data stream, the chunk length may
         // vary.
         if (i + chunk > http_req.size()) {