diff options
| author | Wladimir J. van der Laan <[email protected]> | 2016-05-04 16:05:17 +0200 |
|---|---|---|
| committer | Wladimir J. van der Laan <[email protected]> | 2016-05-05 08:27:12 +0200 |
| commit | f0188f9178a22fd493ed228c008d4cc25ac2952d (patch) | |
| tree | 9277158f758727c16e6b1dc0451f53d834d82862 /src/httpserver.cpp | |
| parent | http: Change boost::scoped_ptr to std::unique_ptr in HTTPRequest (diff) | |
| download | discoin-f0188f9178a22fd493ed228c008d4cc25ac2952d.tar.xz discoin-f0188f9178a22fd493ed228c008d4cc25ac2952d.zip | |
http: use std::move to move HTTPRequest into HTTPWorkItem
Thanks to Cory Fields for the idea.
Diffstat (limited to 'src/httpserver.cpp')
| -rw-r--r-- | src/httpserver.cpp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/httpserver.cpp b/src/httpserver.cpp index c193d2af1..812940eaf 100644 --- a/src/httpserver.cpp +++ b/src/httpserver.cpp @@ -44,8 +44,8 @@ static const size_t MAX_HEADERS_SIZE = 8192; class HTTPWorkItem : public HTTPClosure { public: - HTTPWorkItem(HTTPRequest* req, const std::string &path, const HTTPRequestHandler& func): - req(req), path(path), func(func) + HTTPWorkItem(std::unique_ptr<HTTPRequest> req, const std::string &path, const HTTPRequestHandler& func): + req(std::move(req)), path(path), func(func) { } void operator()() @@ -281,7 +281,7 @@ static void http_request_cb(struct evhttp_request* req, void* arg) // Dispatch to worker thread if (i != iend) { - std::unique_ptr<HTTPWorkItem> item(new HTTPWorkItem(hreq.release(), path, i->handler)); + std::unique_ptr<HTTPWorkItem> item(new HTTPWorkItem(std::move(hreq), path, i->handler)); assert(workQueue); if (workQueue->Enqueue(item.get())) item.release(); /* if true, queue took ownership */ |