From 71d06a1a206d3a4c1c9ba51572e75e3e59e75603 Mon Sep 17 00:00:00 2001
From: Peter Johnson <johnson.peter@gmail.com>
Date: Thu, 1 Mar 2018 20:01:11 -0800
Subject: [PATCH] HttpRequest: Don't reorder parameters. (#73)

Also now allows duplicate parameters (which is needed for some use cases).
---
 src/main/native/include/support/HttpUtil.inl | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/main/native/include/support/HttpUtil.inl b/src/main/native/include/support/HttpUtil.inl
index d67f3c89dd..00538fc3ae 100644
--- a/src/main/native/include/support/HttpUtil.inl
+++ b/src/main/native/include/support/HttpUtil.inl
@@ -13,11 +13,11 @@ namespace wpi {
 template <typename T>
 HttpRequest::HttpRequest(const HttpLocation& loc, const T& extraParams)
     : host{loc.host}, port{loc.port} {
-  llvm::StringMap<llvm::StringRef> params;
+  llvm::SmallVector<std::pair<llvm::StringRef, llvm::StringRef>, 8> params;
   for (const auto& p : loc.params)
-    params.insert(std::make_pair(GetFirst(p), GetSecond(p)));
+    params.emplace_back(std::make_pair(GetFirst(p), GetSecond(p)));
   for (const auto& p : extraParams)
-    params.insert(std::make_pair(GetFirst(p), GetSecond(p)));
+    params.emplace_back(std::make_pair(GetFirst(p), GetSecond(p)));
   SetPath(loc.path, params);
   SetAuth(loc);
 }