Merge pull request #33 from achim-k/patch-1

jihoonl · web-flow · commit c89f9f48d736 · 2017-04-13T09:18:14.000+02:00
web_video_server: fix bool function not returning
diff --git a/src/web_video_server.cpp b/src/web_video_server.cpp
@@ -28,11 +28,14 @@ static bool ros_connection_logger(async_web_server_cpp::HttpServerRequestHandler
   try
   {
     forward(request, connection, begin, end);
+    return true;
   }
   catch (std::exception &e)
   {
     ROS_WARN_STREAM("Error Handling Request: " << e.what());
+    return false;
   }
+  return false;
 }
 
 WebVideoServer::WebVideoServer(ros::NodeHandle &nh, ros::NodeHandle &private_nh) :

Original file line number	Diff line number	Diff line change
`@@ -28,11 +28,14 @@ static bool ros_connection_logger(async_web_server_cpp::HttpServerRequestHandler`
`28`	`28`	`try`
`29`	`29`	`{`
`30`	`30`	`forward(request, connection, begin, end);`
	`31`	`+ return true;`
`31`	`32`	`}`
`32`	`33`	`catch (std::exception &e)`
`33`	`34`	`{`
`34`	`35`	`ROS_WARN_STREAM("Error Handling Request: " << e.what());`
	`36`	`+ return false;`
`35`	`37`	`}`
	`38`	`+ return false;`
`36`	`39`	`}`
`37`	`40`
`38`	`41`	`WebVideoServer::WebVideoServer(ros::NodeHandle &nh, ros::NodeHandle &private_nh) :`