Boolean return code on resource registration

Sebastiano Merlino · Sebastiano Merlino · commit 7763937d5ca3 · 2014-02-23T23:55:01.000Z
diff --git a/src/httpserver/webserver.hpp b/src/httpserver/webserver.hpp
@@ -261,7 +261,7 @@ class webserver
         static void* select(void* self);
         static void* cleaner(void* self);
 
-        void register_resource(const std::string& resource,
+        bool register_resource(const std::string& resource,
                 details::http_resource_mirror hrm, bool family = false
         );
 
diff --git a/src/webserver.cpp b/src/webserver.cpp
@@ -238,7 +238,7 @@ void webserver::request_completed (
     }
 }
 
-void webserver::register_resource(
+bool webserver::register_resource(
         const std::string& resource,
         details::http_resource_mirror hrm,
         bool family
@@ -250,7 +250,7 @@ void webserver::register_resource(
     details::http_endpoint idx(resource, family, true, regex_checking);
 
     pair<map<details::http_endpoint, details::http_resource_mirror>::iterator, bool> result = registered_resources.insert(
-        pair<details::http_endpoint, details::http_resource_mirror>(idx,hrm)
+        pair<details::http_endpoint, details::http_resource_mirror>(idx, hrm)
     );
 
     if(result.second)
@@ -259,6 +259,8 @@ void webserver::register_resource(
             pair<string, details::http_resource_mirror*>(idx.get_url_complete(), &(result.first->second))
         );
     }
+
+    return result.second;
 }
 
 void* webserver::select(void* self)

Original file line number	Diff line number	Diff line change
`@@ -238,7 +238,7 @@ void webserver::request_completed (`
`238`	`238`	`}`
`239`	`239`	`}`
`240`	`240`
`241`		`-void webserver::register_resource(`
	`241`	`+bool webserver::register_resource(`
`242`	`242`	`const std::string& resource,`
`243`	`243`	`details::http_resource_mirror hrm,`
`244`	`244`	`bool family`
`@@ -250,7 +250,7 @@ void webserver::register_resource(`
`250`	`250`	`details::http_endpoint idx(resource, family, true, regex_checking);`
`251`	`251`
`252`	`252`	`pair<map<details::http_endpoint, details::http_resource_mirror>::iterator, bool> result = registered_resources.insert(`
`253`		`- pair<details::http_endpoint, details::http_resource_mirror>(idx,hrm)`
	`253`	`+ pair<details::http_endpoint, details::http_resource_mirror>(idx, hrm)`
`254`	`254`	`);`
`255`	`255`
`256`	`256`	`if(result.second)`
`@@ -259,6 +259,8 @@ void webserver::register_resource(`
`259`	`259`	`pair<string, details::http_resource_mirror*>(idx.get_url_complete(), &(result.first->second))`
`260`	`260`	`);`
`261`	`261`	`}`
	`262`	`+`
	`263`	`+ return result.second;`
`262`	`264`	`}`
`263`	`265`
`264`	`266`	`void* webserver::select(void* self)`