Merge pull request #139 from mloskot/ml/trac-ticket-7092

Remove re-assignment of functor from for_each_pixel (Trac 7092)

include/boost/gil/algorithm.hpp

@@ -765,7 +765,7 @@ F for_each_pixel(const V& img, F fun) {
         return std::for_each(img.begin().x(), img.end().x(), fun);
     } else {
         for (std::ptrdiff_t y=0; y<img.height(); ++y)
-            fun = std::for_each(img.row_begin(y),img.row_end(y),fun);
+            std::for_each(img.row_begin(y),img.row_end(y),fun);
         return fun;
     }
 }

test/algorithm/Jamfile

@@ -0,0 +1,17 @@
+# Boost.GIL (Generic Image Library) - tests
+#
+# Copyright (c) 2018 Mateusz Loskot <mateusz at loskot dot net>
+#
+# Distributed under the Boost Software License, Version 1.0.
+# (See accompanying file LICENSE_1_0.txt or
+# copy at http://www.boost.org/LICENSE_1_0.txt)
+
+import testing ;
+
+project
+    : requirements
+    <include>$(BOOST_ROOT)
+    <include>..
+    ;
+
+run for_each_pixel.cpp ;

test/algorithm/for_each_pixel.cpp

@@ -0,0 +1,32 @@
+//
+// Copyright 2018 Mateusz Loskot <mateusz at loskot dot net>
+//
+// Distributed under the Boost Software License, Version 1.0
+// See accompanying file LICENSE_1_0.txt or copy at
+// http://www.boost.org/LICENSE_1_0.txt
+//
+#include <boost/gil/algorithm.hpp>
+#include <boost/gil/image.hpp>
+
+#include <boost/core/lightweight_test.hpp>
+
+namespace gil = boost::gil;
+
+void test_lambda_expression()
+{
+    gil::gray8_pixel_t const gray128(128);
+    gil::gray8_image_t image(2, 2, gray128);
+
+    int sum{0};
+    gil::for_each_pixel(gil::view(image), [&sum](gil::gray8_pixel_t& p) {
+        sum += gil::at_c<0>(p);
+    });
+    BOOST_TEST(sum == 2 * 2 * 128);
+}
+
+int main()
+{
+    test_lambda_expression();
+
+    return boost::report_errors();
+}