TestHTTPServer: Make listening an explicit operation that reports failure.

Use this to print the error message when listening fails, and switch to always
stack allocating TestHTTPServer instances for easier cleanup.

Change-Id: I63b2bd38963b66611dc08a5c322615d91a91e675
Reviewed-by: John Brooks <john.brooks@dereferenced.net>
Reviewed-by: J-P Nurmi <jpnurmi@digia.com>
Reviewed-by: Lorn Potter <lorn.potter@jollamobile.com>

1 files changed, 5 insertions, 1 deletions

diff --git a/tests/manual/httpserver/main.cpp b/tests/manual/httpserver/main.cpp
index ea729547ce..4ad44508b0 100644
--- a/tests/manual/httpserver/main.cpp
+++ b/tests/manual/httpserver/main.cpp
@@ -112,7 +112,11 @@ int main(int argc, char *argv[])
                << "\":\n\n" << QDir(directory).entryList(QDir::Files).join(QLatin1Char('\n'))
                << "\n\non http://localhost:" << port << '\n';
 
-    TestHTTPServer server(port);
+    TestHTTPServer server;
+    if (!server.listen(port)) {
+        std::wcout << "Couldn't listen on port " << port << server.errorString().toLocal8Bit();
+        exit(-1);
+    }
     server.serveDirectory(directory);
 
     return a.exec();