[Boost-commit] svn:boost r80919 - in trunk/boost/geometry: algorithms multi/algorithms

Subject: [Boost-commit] svn:boost r80919 - in trunk/boost/geometry: algorithms multi/algorithms
From: bruno.lalande_at_[hidden]
Date: 2012-10-09 19:03:20

Author: bruno.lalande
Date: 2012-10-09 19:03:18 EDT (Tue, 09 Oct 2012)
New Revision: 80919
URL: http://svn.boost.org/trac/boost/changeset/80919

Log:
Moved Functor template param from class to function level in dispatch::for_each.
Text files modified: 
   trunk/boost/geometry/algorithms/for_each.hpp       |    92 ++++++++++++++++----------------------- 
   trunk/boost/geometry/multi/algorithms/for_each.hpp |    34 +++-----------                          
   2 files changed, 46 insertions(+), 80 deletions(-)

Modified: trunk/boost/geometry/algorithms/for_each.hpp
==============================================================================
--- trunk/boost/geometry/algorithms/for_each.hpp	(original)
+++ trunk/boost/geometry/algorithms/for_each.hpp	2012-10-09 19:03:18 EDT (Tue, 09 Oct 2012)
@@ -40,9 +40,9 @@
 {
 
 
-template <typename Point, typename Functor>
 struct fe_point_per_point
 {
+    template <typename Point, typename Functor>
     static inline Functor apply(Point& point, Functor f)
     {
         f(point);
@@ -51,9 +51,9 @@
 };
 
 
-template <typename Point, typename Functor>
 struct fe_point_per_segment
 {
+    template <typename Point, typename Functor>
     static inline Functor apply(Point& , Functor f)
     {
         // TODO: if non-const, we should extract the points from the segment
@@ -63,9 +63,9 @@
 };
 
 
-template <typename Range, typename Functor>
 struct fe_range_per_point
 {
+    template <typename Range, typename Functor>
     static inline Functor apply(Range& range, Functor f)
     {
         return (std::for_each(boost::begin(range), boost::end(range), f));
@@ -73,9 +73,9 @@
 };
 
 
-template <typename Range, typename Functor>
 struct fe_range_per_segment
 {
+    template <typename Range, typename Functor>
     static inline Functor apply(Range& range, Functor f)
     {
         typedef typename add_const_if_c
@@ -98,24 +98,18 @@
 };
 
 
-template <typename Polygon, typename Functor>
 struct fe_polygon_per_point
 {
+    template <typename Polygon, typename Functor>
     static inline Functor apply(Polygon& poly, Functor f)
     {
-        typedef fe_range_per_point
-                <
-                    typename ring_type<Polygon>::type,
-                    Functor
-                > per_ring;
-
-        f = per_ring::apply(exterior_ring(poly), f);
+        f = fe_range_per_point::apply(exterior_ring(poly), f);
 
         typename interior_return_type<Polygon>::type rings
                     = interior_rings(poly);
         for (BOOST_AUTO_TPL(it, boost::begin(rings)); it != boost::end(rings); ++it)
         {
-            f = per_ring::apply(*it, f);
+            f = fe_range_per_point::apply(*it, f);
         }
 
         return f;
@@ -124,24 +118,18 @@
 };
 
 
-template <typename Polygon, typename Functor>
 struct fe_polygon_per_segment
 {
+    template <typename Polygon, typename Functor>
     static inline Functor apply(Polygon& poly, Functor f)
     {
-        typedef fe_range_per_segment
-            <
-                typename ring_type<Polygon>::type,
-                Functor
-            > per_ring;
-
-        f = per_ring::apply(exterior_ring(poly), f);
+        f = fe_range_per_segment::apply(exterior_ring(poly), f);
 
         typename interior_return_type<Polygon>::type rings
                     = interior_rings(poly);
         for (BOOST_AUTO_TPL(it, boost::begin(rings)); it != boost::end(rings); ++it)
         {
-            f = per_ring::apply(*it, f);
+            f = fe_range_per_segment::apply(*it, f);
         }
 
         return f;
@@ -161,65 +149,63 @@
 template
 <
     typename Tag,
-    typename Geometry,
-    typename Functor
+    typename Geometry
 >
 struct for_each_point {};
 
 
-template <typename Point, typename Functor>
-struct for_each_point<point_tag, Point, Functor>
-    : detail::for_each::fe_point_per_point<Point, Functor>
+template <typename Point>
+struct for_each_point<point_tag, Point>
+    : detail::for_each::fe_point_per_point
 {};
 
 
-template <typename Linestring, typename Functor>
-struct for_each_point<linestring_tag, Linestring, Functor>
-    : detail::for_each::fe_range_per_point<Linestring, Functor>
+template <typename Linestring>
+struct for_each_point<linestring_tag, Linestring>
+    : detail::for_each::fe_range_per_point
 {};
 
 
-template <typename Ring, typename Functor>
-struct for_each_point<ring_tag, Ring, Functor>
-    : detail::for_each::fe_range_per_point<Ring, Functor>
+template <typename Ring>
+struct for_each_point<ring_tag, Ring>
+    : detail::for_each::fe_range_per_point
 {};
 
 
-template <typename Polygon, typename Functor>
-struct for_each_point<polygon_tag, Polygon, Functor>
-    : detail::for_each::fe_polygon_per_point<Polygon, Functor>
+template <typename Polygon>
+struct for_each_point<polygon_tag, Polygon>
+    : detail::for_each::fe_polygon_per_point
 {};
 
 
 template
 <
     typename Tag,
-    typename Geometry,
-    typename Functor
+    typename Geometry
 >
 struct for_each_segment {};
 
-template <typename Point, typename Functor>
-struct for_each_segment<point_tag, Point, Functor>
-    : detail::for_each::fe_point_per_segment<Point, Functor>
+template <typename Point>
+struct for_each_segment<point_tag, Point>
+    : detail::for_each::fe_point_per_segment
 {};
 
 
-template <typename Linestring, typename Functor>
-struct for_each_segment<linestring_tag, Linestring, Functor>
-    : detail::for_each::fe_range_per_segment<Linestring, Functor>
+template <typename Linestring>
+struct for_each_segment<linestring_tag, Linestring>
+    : detail::for_each::fe_range_per_segment
 {};
 
 
-template <typename Ring, typename Functor>
-struct for_each_segment<ring_tag, Ring, Functor>
-    : detail::for_each::fe_range_per_segment<Ring, Functor>
+template <typename Ring>
+struct for_each_segment<ring_tag, Ring>
+    : detail::for_each::fe_range_per_segment
 {};
 
 
-template <typename Polygon, typename Functor>
-struct for_each_segment<polygon_tag, Polygon, Functor>
-    : detail::for_each::fe_polygon_per_segment<Polygon, Functor>
+template <typename Polygon>
+struct for_each_segment<polygon_tag, Polygon>
+    : detail::for_each::fe_polygon_per_segment
 {};
 
 
@@ -249,8 +235,7 @@
     return dispatch::for_each_point
         <
             typename tag_cast<typename tag<Geometry>::type, multi_tag>::type,
-            Geometry,
-            Functor
+            Geometry
         >::apply(geometry, f);
 }
 
@@ -276,8 +261,7 @@
     return dispatch::for_each_segment
         <
             typename tag_cast<typename tag<Geometry>::type, multi_tag>::type,
-            Geometry,
-            Functor
+            Geometry
         >::apply(geometry, f);
 }
 

Modified: trunk/boost/geometry/multi/algorithms/for_each.hpp
==============================================================================
--- trunk/boost/geometry/multi/algorithms/for_each.hpp	(original)
+++ trunk/boost/geometry/multi/algorithms/for_each.hpp	2012-10-09 19:03:18 EDT (Tue, 09 Oct 2012)
@@ -36,14 +36,10 @@
 
 // Implementation of multi, for both point and segment,
 // just calling the single version.
-template
-<
-    typename MultiGeometry,
-    typename Functor,
-    typename Policy
->
+template <typename Policy>
 struct for_each_multi
 {
+    template <typename MultiGeometry, typename Functor>
     static inline Functor apply(MultiGeometry& multi, Functor f)
     {
         for(BOOST_AUTO_TPL(it, boost::begin(multi)); it != boost::end(multi); ++it)
@@ -63,16 +59,10 @@
 namespace dispatch
 {
 
-template
-<
-    typename MultiGeometry,
-    typename Functor
->
-struct for_each_point<multi_tag, MultiGeometry, Functor>
+template <typename MultiGeometry>
+struct for_each_point<multi_tag, MultiGeometry>
     : detail::for_each::for_each_multi
         <
-            MultiGeometry,
-            Functor,
             // Specify the dispatch of the single-version as policy
             for_each_point
                 <
@@ -84,23 +74,16 @@
                         <
                             is_const<MultiGeometry>::value,
                             typename boost::range_value<MultiGeometry>::type
-                        >::type,
-                    Functor
+                        >::type
                 >
         >
 {};
 
 
-template
-<
-    typename MultiGeometry,
-    typename Functor
->
-struct for_each_segment<multi_tag, MultiGeometry, Functor>
+template <typename MultiGeometry>
+struct for_each_segment<multi_tag, MultiGeometry>
     : detail::for_each::for_each_multi
         <
-            MultiGeometry,
-            Functor,
             // Specify the dispatch of the single-version as policy
             for_each_segment
                 <
@@ -112,8 +95,7 @@
                         <
                             is_const<MultiGeometry>::value,
                             typename boost::range_value<MultiGeometry>::type
-                        >::type,
-                    Functor
+                        >::type
                 >
         >
 {};