Response: addHeader(): Pass by value, not reference
This commit is contained in:
		| @@ -32,12 +32,12 @@ Response::Response(int http_status_code, std::string html) | |||||||
| 	this->html = std::move(html); | 	this->html = std::move(html); | ||||||
| } | } | ||||||
|  |  | ||||||
| void Response::addHeader(const std::string &key, const std::string &value) | void Response::addHeader(std::string key, std::string value) | ||||||
| { | { | ||||||
| 	this->responseHeaders.insert(std::make_pair(key, value)); | 	this->responseHeaders.insert(std::make_pair(key, value)); | ||||||
| } | } | ||||||
|  |  | ||||||
| Response Response::redirectTemporarily(const std::string &url) | Response Response::redirectTemporarily(std::string url) | ||||||
| { | { | ||||||
| 	Response result; | 	Response result; | ||||||
| 	result.addHeader("Location", url); | 	result.addHeader("Location", url); | ||||||
|   | |||||||
| @@ -27,8 +27,8 @@ class Response | |||||||
| 		return this->html; | 		return this->html; | ||||||
| 	} | 	} | ||||||
|  |  | ||||||
| 	void addHeader(const std::string &key, const std::string &value); | 	void addHeader(std::string key, std::string value); | ||||||
| 	static Response redirectTemporarily(const std::string &url); | 	static Response redirectTemporarily(std::string url); | ||||||
|  |  | ||||||
| 	void setStatus(int status) | 	void setStatus(int status) | ||||||
| 	{ | 	{ | ||||||
|   | |||||||
		Reference in New Issue
	
	Block a user