fixed misuse of remove_if (#35)

artifactz · jihoonl · commit 76facbbdb8dc · 2017-07-27T10:43:32.000+02:00
diff --git a/src/web_video_server.cpp b/src/web_video_server.cpp
@@ -97,8 +97,8 @@ void WebVideoServer::cleanup_inactive_streams()
   if (lock)
   {
     typedef std::vector<boost::shared_ptr<ImageStreamer> >::iterator itr_type;
-    itr_type new_end = std::remove_if(image_subscribers_.begin(), image_subscribers_.end(),
-                                      boost::bind(&ImageStreamer::isInactive, _1));
+    itr_type new_end = std::partition(image_subscribers_.begin(), image_subscribers_.end(),
+                                      !boost::bind(&ImageStreamer::isInactive, _1));
     if (__verbose)
     {
       for (itr_type itr = new_end; itr < image_subscribers_.end(); ++itr)

Original file line number	Diff line number	Diff line change
`@@ -97,8 +97,8 @@ void WebVideoServer::cleanup_inactive_streams()`
`97`	`97`	`if (lock)`
`98`	`98`	`{`
`99`	`99`	`typedef std::vector<boost::shared_ptr<ImageStreamer> >::iterator itr_type;`
`100`		`- itr_type new_end = std::remove_if(image_subscribers_.begin(), image_subscribers_.end(),`
`101`		`- boost::bind(&ImageStreamer::isInactive, _1));`
	`100`	`+ itr_type new_end = std::partition(image_subscribers_.begin(), image_subscribers_.end(),`
	`101`	`+ !boost::bind(&ImageStreamer::isInactive, _1));`
`102`	`102`	`if (__verbose)`
`103`	`103`	`{`
`104`	`104`	`for (itr_type itr = new_end; itr < image_subscribers_.end(); ++itr)`