$include_dir="/home/hyper-archives/boost-commit/include"; include("$include_dir/msg-header.inc") ?>
Subject: [Boost-commit] svn:boost r71309 - in trunk/boost/geometry: algorithms algorithms/detail algorithms/detail/equals algorithms/detail/overlay algorithms/detail/sections domains/gis/io/wkt extensions/algorithms extensions/gis/projections extensions/nsphere/geometries extensions/strategies geometries iterators strategies strategies/cartesian
From: barend.gehrels_at_[hidden]
Date: 2011-04-16 07:32:02
Author: barendgehrels
Date: 2011-04-16 07:31:53 EDT (Sat, 16 Apr 2011)
New Revision: 71309
URL: http://svn.boost.org/trac/boost/changeset/71309
Log:
Moved assign-variants to detail, reverted (manually) move of convert to detail
Added:
   trunk/boost/geometry/algorithms/convert.hpp
      - copied, changed from r71271, /trunk/boost/geometry/algorithms/detail/convert.hpp
Removed:
   trunk/boost/geometry/algorithms/detail/convert.hpp
Text files modified: 
   trunk/boost/geometry/algorithms/assign.hpp                                      |    36 ++++++++++++++++++------------------    
   trunk/boost/geometry/algorithms/centroid.hpp                                    |     6 +++---                                  
   trunk/boost/geometry/algorithms/convert.hpp                                     |     7 -------                                 
   trunk/boost/geometry/algorithms/detail/equals/collect_vectors.hpp               |     2 +-                                      
   trunk/boost/geometry/algorithms/detail/overlay/clip_linestring.hpp              |     6 +++---                                  
   trunk/boost/geometry/algorithms/detail/overlay/convert_ring.hpp                 |     8 ++++----                                
   trunk/boost/geometry/algorithms/detail/overlay/copy_segment_point.hpp           |     4 ++--                                    
   trunk/boost/geometry/algorithms/detail/overlay/get_intersection_points.hpp      |     4 ++--                                    
   trunk/boost/geometry/algorithms/detail/overlay/get_turn_info.hpp                |    14 +++++++-------                          
   trunk/boost/geometry/algorithms/detail/overlay/intersection_insert.hpp          |     6 +++---                                  
   trunk/boost/geometry/algorithms/detail/sections/sectionalize.hpp                |     4 +++-                                    
   trunk/boost/geometry/algorithms/distance.hpp                                    |     8 ++++----                                
   trunk/boost/geometry/algorithms/length.hpp                                      |     4 ++--                                    
   trunk/boost/geometry/algorithms/simplify.hpp                                    |     4 ++--                                    
   trunk/boost/geometry/algorithms/transform.hpp                                   |     8 ++++----                                
   trunk/boost/geometry/domains/gis/io/wkt/read_wkt.hpp                            |     8 ++++----                                
   trunk/boost/geometry/domains/gis/io/wkt/write_wkt.hpp                           |     8 ++++----                                
   trunk/boost/geometry/extensions/algorithms/dissolve.hpp                         |     4 ++--                                    
   trunk/boost/geometry/extensions/gis/projections/project_inverse_transformer.hpp |     2 +-                                      
   trunk/boost/geometry/extensions/gis/projections/project_transformer.hpp         |     2 +-                                      
   trunk/boost/geometry/extensions/nsphere/geometries/nsphere.hpp                  |     4 ++--                                    
   trunk/boost/geometry/extensions/strategies/buffer_join_round.hpp                |     6 +++---                                  
   trunk/boost/geometry/geometries/box.hpp                                         |     6 +++---                                  
   trunk/boost/geometry/iterators/box_iterator.hpp                                 |     4 +---                                    
   trunk/boost/geometry/iterators/segment_range_iterator.hpp                       |     4 ++--                                    
   trunk/boost/geometry/strategies/cartesian/distance_projected_point.hpp          |     8 ++++----                                
   trunk/boost/geometry/strategies/strategy_transform.hpp                          |     4 ++--                                    
   27 files changed, 87 insertions(+), 94 deletions(-)
Modified: trunk/boost/geometry/algorithms/assign.hpp
==============================================================================
--- trunk/boost/geometry/algorithms/assign.hpp	(original)
+++ trunk/boost/geometry/algorithms/assign.hpp	2011-04-16 07:31:53 EDT (Sat, 16 Apr 2011)
@@ -26,7 +26,7 @@
 #include <boost/type_traits.hpp>
 
 #include <boost/geometry/algorithms/detail/assign_values.hpp>
-#include <boost/geometry/algorithms/detail/convert.hpp>
+#include <boost/geometry/algorithms/convert.hpp>
 
 #include <boost/geometry/arithmetic/arithmetic.hpp>
 #include <boost/geometry/algorithms/append.hpp>
@@ -53,12 +53,11 @@
 \param geometry \param_geometry
 \param range \param_range_point
 
-\qbk{distinguish, with a range}
 \qbk{
 [heading Notes]
 [note Assign automatically clears the geometry before assigning (use append if you don't want that)]
 [heading Example]
-[assign_with_range] [assign_with_range_output]
+[assign_points] [assign_points_output]
 
 [heading See also]
 \* [link geometry.reference.algorithms.append.append append]
@@ -124,6 +123,12 @@
 }
 
 
+#ifndef DOXYGEN_NO_DETAIL
+namespace detail 
+{
+// Note: this is moved to namespace detail because the names and parameter orders
+// are not yet 100% clear.
+
 /*!
 \brief Assign the four points of a 2D box
 \ingroup assign
@@ -231,12 +236,9 @@
         >::apply(geometry, point);
 }
 
+} // namespace detail
+#endif // DOXYGEN_NO_DETAIL
 
-}} // namespace boost::geometry
-
-
-namespace boost { namespace geometry
-{
 
 /*!
 \brief Assigns one geometry to another geometry
@@ -245,24 +247,22 @@
 \ingroup assign
 \tparam Geometry1 \tparam_geometry
 \tparam Geometry2 \tparam_geometry
-\param geometry1 \param_geometry (source)
-\param geometry2 \param_geometry (target)
-\note It is moved to namespace detail because it overlaps functionality
-    of assign. So assign will be changed such that it also can convert.
+\param geometry1 \param_geometry (target)
+\param geometry2 \param_geometry (source)
  */
 template <typename Geometry1, typename Geometry2>
-inline void assign(Geometry1 const& geometry1, Geometry2& geometry2)
+inline void assign_rev(Geometry1& geometry1, Geometry2 const& geometry2)
 {
-    concept::check_concepts_and_equal_dimensions<Geometry1 const, Geometry2>();
+    concept::check_concepts_and_equal_dimensions<Geometry1, Geometry2 const>();
 
     dispatch::convert
         <
-            typename tag<Geometry1>::type,
             typename tag<Geometry2>::type,
+            typename tag<Geometry1>::type,
             dimension<Geometry1>::type::value,
-            Geometry1,
-            Geometry2
-        >::apply(geometry1, geometry2);
+            Geometry2,
+            Geometry1
+        >::apply(geometry2, geometry1);
 }
 
 
Modified: trunk/boost/geometry/algorithms/centroid.hpp
==============================================================================
--- trunk/boost/geometry/algorithms/centroid.hpp	(original)
+++ trunk/boost/geometry/algorithms/centroid.hpp	2011-04-16 07:31:53 EDT (Sat, 16 Apr 2011)
@@ -28,7 +28,7 @@
 #include <boost/geometry/core/interior_rings.hpp>
 #include <boost/geometry/core/tag_cast.hpp>
 
-#include <boost/geometry/algorithms/detail/convert.hpp>
+#include <boost/geometry/algorithms/convert.hpp>
 #include <boost/geometry/algorithms/distance.hpp>
 #include <boost/geometry/geometries/concepts/check.hpp>
 #include <boost/geometry/strategies/centroid.hpp>
@@ -83,7 +83,7 @@
     static inline void apply(Point const& point, PointCentroid& centroid,
             Strategy const&)
     {
-        geometry::detail::convert(point, centroid);
+        geometry::convert(point, centroid);
     }
 };
 
@@ -163,7 +163,7 @@
     else // if (n == 1)
     {
         // Take over the first point in a "coordinate neutral way"
-        geometry::detail::convert(*boost::begin(range), centroid);
+        geometry::convert(*boost::begin(range), centroid);
         return false;
     }
     return true;
Copied: trunk/boost/geometry/algorithms/convert.hpp (from r71271, /trunk/boost/geometry/algorithms/detail/convert.hpp)
==============================================================================
--- /trunk/boost/geometry/algorithms/detail/convert.hpp	(original)
+++ trunk/boost/geometry/algorithms/convert.hpp	2011-04-16 07:31:53 EDT (Sat, 16 Apr 2011)
@@ -235,10 +235,6 @@
 #endif // DOXYGEN_NO_DISPATCH
 
 
-#ifndef DOXYGEN_NO_DETAIL
-namespace detail
-{
-
 /*!
 \brief Converts one geometry to another geometry
 \details The convert algorithm converts one geometry, e.g. a BOX, to another geometry, e.g. a RING. This only
@@ -266,9 +262,6 @@
         >::apply(geometry1, geometry2);
 }
 
-}
-#endif // DOXYGEN_NO_DETAIL
-
 
 }} // namespace boost::geometry
 
Deleted: trunk/boost/geometry/algorithms/detail/convert.hpp
==============================================================================
--- trunk/boost/geometry/algorithms/detail/convert.hpp	2011-04-16 07:31:53 EDT (Sat, 16 Apr 2011)
+++ (empty file)
@@ -1,276 +0,0 @@
-// Boost.Geometry (aka GGL, Generic Geometry Library)
-
-// Copyright (c) 2007-2011 Barend Gehrels, Amsterdam, the Netherlands.
-// Copyright (c) 2008-2011 Bruno Lalande, Paris, France.
-// Copyright (c) 2009-2011 Mateusz Loskot, London, UK.
-
-// Parts of Boost.Geometry are redesigned from Geodan's Geographic Library
-// (geolib/GGL), copyright (c) 1995-2010 Geodan, Amsterdam, the Netherlands.
-
-// Use, modification and distribution is subject to 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)
-
-#ifndef BOOST_GEOMETRY_ALGORITHMS_DETAIL_CONVERT_HPP
-#define BOOST_GEOMETRY_ALGORITHMS_DETAIL_CONVERT_HPP
-
-
-#include <cstddef>
-
-#include <boost/numeric/conversion/cast.hpp>
-#include <boost/range.hpp>
-
-#include <boost/geometry/arithmetic/arithmetic.hpp>
-#include <boost/geometry/algorithms/append.hpp>
-#include <boost/geometry/algorithms/clear.hpp>
-#include <boost/geometry/algorithms/for_each.hpp>
-#include <boost/geometry/algorithms/detail/assign_values.hpp>
-#include <boost/geometry/algorithms/detail/convert_point_to_point.hpp>
-
-#include <boost/geometry/core/cs.hpp>
-#include <boost/geometry/geometries/concepts/check.hpp>
-
-
-namespace boost { namespace geometry
-{
-
-#ifndef DOXYGEN_NO_DETAIL
-namespace detail { namespace conversion
-{
-
-template
-<
-    typename Point,
-    typename Box,
-    std::size_t Index,
-    std::size_t Dimension,
-    std::size_t DimensionCount
->
-struct point_to_box
-{
-    static inline void apply(Point const& point, Box& box)
-    {
-        typedef typename coordinate_type<Box>::type coordinate_type;
-
-        set<Index, Dimension>(box,
-                boost::numeric_cast<coordinate_type>(get<Dimension>(point)));
-        point_to_box
-            <
-                Point, Box,
-                Index, Dimension + 1, DimensionCount
-            >::apply(point, box);
-    }
-};
-
-
-template
-<
-    typename Point,
-    typename Box,
-    std::size_t Index,
-    std::size_t DimensionCount
->
-struct point_to_box<Point, Box, Index, DimensionCount, DimensionCount>
-{
-    static inline void apply(Point const& , Box& )
-    {}
-};
-
-
-}} // namespace detail::conversion
-#endif // DOXYGEN_NO_DETAIL
-
-
-#ifndef DOXYGEN_NO_DISPATCH
-namespace dispatch
-{
-
-template
-<
-    typename Tag1, typename Tag2,
-    std::size_t DimensionCount,
-    typename Geometry1, typename Geometry2
->
-struct convert
-{
-};
-
-
-template
-<
-    typename Tag,
-    std::size_t DimensionCount,
-    typename Geometry1, typename Geometry2
->
-struct convert<Tag, Tag, DimensionCount, Geometry1, Geometry2>
-{
-    // Same geometry type -> copy coordinates from G1 to G2
-    // Actually: we try now to just copy it
-    static inline void apply(Geometry1 const& source, Geometry2& destination)
-    {
-        destination = source;
-    }
-};
-
-
-template
-<
-    std::size_t DimensionCount,
-    typename Geometry1, typename Geometry2
->
-struct convert<point_tag, point_tag, DimensionCount, Geometry1, Geometry2>
-    : detail::conversion::point_to_point<Geometry1, Geometry2, 0, DimensionCount>
-{};
-
-
-template <std::size_t DimensionCount, typename Ring1, typename Ring2>
-struct convert<ring_tag, ring_tag, DimensionCount, Ring1, Ring2>
-{
-    static inline void apply(Ring1 const& source, Ring2& destination)
-    {
-        geometry::clear(destination);
-        for (typename boost::range_iterator<Ring1 const>::type it
-            = boost::begin(source);
-            it != boost::end(source);
-            ++it)
-        {
-            geometry::append(destination, *it);
-        }
-    }
-};
-
-
-template <typename Box, typename Ring>
-struct convert<box_tag, ring_tag, 2, Box, Ring>
-{
-    static inline void apply(Box const& box, Ring& ring)
-    {
-        // go from box to ring -> add coordinates in correct order
-        geometry::clear(ring);
-        typename point_type<Box>::type point;
-
-        geometry::assign_values(point, get<min_corner, 0>(box), get<min_corner, 1>(box));
-        geometry::append(ring, point);
-
-        geometry::assign_values(point, get<min_corner, 0>(box), get<max_corner, 1>(box));
-        geometry::append(ring, point);
-
-        geometry::assign_values(point, get<max_corner, 0>(box), get<max_corner, 1>(box));
-        geometry::append(ring, point);
-
-        geometry::assign_values(point, get<max_corner, 0>(box), get<min_corner, 1>(box));
-        geometry::append(ring, point);
-
-        geometry::assign_values(point, get<min_corner, 0>(box), get<min_corner, 1>(box));
-        geometry::append(ring, point);
-    }
-};
-
-
-template <typename Box, typename Polygon>
-struct convert<box_tag, polygon_tag, 2, Box, Polygon>
-{
-    static inline void apply(Box const& box, Polygon& polygon)
-    {
-        typedef typename ring_type<Polygon>::type ring_type;
-
-        convert
-            <
-                box_tag, ring_tag,
-                2, Box, ring_type
-            >::apply(box, exterior_ring(polygon));
-    }
-};
-
-
-template <typename Point, std::size_t DimensionCount, typename Box>
-struct convert<point_tag, box_tag, DimensionCount, Point, Box>
-{
-    static inline void apply(Point const& point, Box& box)
-    {
-        detail::conversion::point_to_box
-            <
-                Point, Box, min_corner, 0, DimensionCount
-            >::apply(point, box);
-        detail::conversion::point_to_box
-            <
-                Point, Box, max_corner, 0, DimensionCount
-            >::apply(point, box);
-    }
-};
-
-
-template <typename Ring, std::size_t DimensionCount, typename Polygon>
-struct convert<ring_tag, polygon_tag, DimensionCount, Ring, Polygon>
-{
-    static inline void apply(Ring const& ring, Polygon& polygon)
-    {
-        typedef typename ring_type<Polygon>::type ring_type;
-        convert
-            <
-                ring_tag, ring_tag, DimensionCount,
-                Ring, ring_type
-            >::apply(ring, exterior_ring(polygon));
-    }
-};
-
-
-template <typename Polygon, std::size_t DimensionCount, typename Ring>
-struct convert<polygon_tag, ring_tag, DimensionCount, Polygon, Ring>
-{
-    static inline void apply(Polygon const& polygon, Ring& ring)
-    {
-        typedef typename ring_type<Polygon>::type ring_type;
-
-        convert
-            <
-                ring_tag, ring_tag, DimensionCount,
-                ring_type, Ring
-            >::apply(exterior_ring(polygon), ring);
-    }
-};
-
-
-} // namespace dispatch
-#endif // DOXYGEN_NO_DISPATCH
-
-
-#ifndef DOXYGEN_NO_DETAIL
-namespace detail
-{
-
-/*!
-\brief Converts one geometry to another geometry
-\details The convert algorithm converts one geometry, e.g. a BOX, to another geometry, e.g. a RING. This only
-if it is possible and applicable.
-\ingroup convert
-\tparam Geometry1 \tparam_geometry
-\tparam Geometry2 \tparam_geometry
-\param geometry1 \param_geometry (source)
-\param geometry2 \param_geometry (target)
-\note It is moved to namespace detail because it overlaps functionality
-    of assign.
- */
-template <typename Geometry1, typename Geometry2>
-inline void convert(Geometry1 const& geometry1, Geometry2& geometry2)
-{
-    concept::check_concepts_and_equal_dimensions<Geometry1 const, Geometry2>();
-
-    dispatch::convert
-        <
-            typename tag<Geometry1>::type,
-            typename tag<Geometry2>::type,
-            dimension<Geometry1>::type::value,
-            Geometry1,
-            Geometry2
-        >::apply(geometry1, geometry2);
-}
-
-}
-#endif // DOXYGEN_NO_DETAIL
-
-
-}} // namespace boost::geometry
-
-
-#endif // BOOST_GEOMETRY_ALGORITHMS_DETAIL_CONVERT_HPP
Modified: trunk/boost/geometry/algorithms/detail/equals/collect_vectors.hpp
==============================================================================
--- trunk/boost/geometry/algorithms/detail/equals/collect_vectors.hpp	(original)
+++ trunk/boost/geometry/algorithms/detail/equals/collect_vectors.hpp	2011-04-16 07:31:53 EDT (Sat, 16 Apr 2011)
@@ -162,7 +162,7 @@
     {
         typename point_type<Box>::type lower_left, lower_right,
                 upper_left, upper_right;
-        assign_box_corners(box, lower_left, lower_right,
+        geometry::detail::assign_box_corners(box, lower_left, lower_right,
                 upper_left, upper_right);
 
         typedef typename boost::range_value<Collection>::type item;
Modified: trunk/boost/geometry/algorithms/detail/overlay/clip_linestring.hpp
==============================================================================
--- trunk/boost/geometry/algorithms/detail/overlay/clip_linestring.hpp	(original)
+++ trunk/boost/geometry/algorithms/detail/overlay/clip_linestring.hpp	2011-04-16 07:31:53 EDT (Sat, 16 Apr 2011)
@@ -13,7 +13,7 @@
 
 #include <boost/geometry/algorithms/append.hpp>
 #include <boost/geometry/algorithms/clear.hpp>
-#include <boost/geometry/algorithms/detail/convert.hpp>
+#include <boost/geometry/algorithms/convert.hpp>
 
 #include <boost/geometry/util/select_coordinate_type.hpp>
 #include <boost/geometry/geometries/segment.hpp>
@@ -181,8 +181,8 @@
             ++previous, ++vertex)
     {
         point_type p1, p2;
-        geometry::detail::convert(*previous, p1);
-        geometry::detail::convert(*vertex, p2);
+        geometry::convert(*previous, p1);
+        geometry::convert(*vertex, p2);
 
         // Clip the segment. Five situations:
         // 1. Segment is invisible, finish line if any (shouldn't occur)
Modified: trunk/boost/geometry/algorithms/detail/overlay/convert_ring.hpp
==============================================================================
--- trunk/boost/geometry/algorithms/detail/overlay/convert_ring.hpp	(original)
+++ trunk/boost/geometry/algorithms/detail/overlay/convert_ring.hpp	2011-04-16 07:31:53 EDT (Sat, 16 Apr 2011)
@@ -19,7 +19,7 @@
 #include <boost/geometry/core/interior_rings.hpp>
 #include <boost/geometry/algorithms/detail/ring_identifier.hpp>
 
-#include <boost/geometry/algorithms/detail/convert.hpp>
+#include <boost/geometry/algorithms/convert.hpp>
 
 
 namespace boost { namespace geometry
@@ -50,7 +50,7 @@
     {
         if (! append)
         {
-            geometry::detail::convert(source, destination);
+            geometry::convert(source, destination);
             if (reverse)
             {
                 boost::reverse(destination);
@@ -69,7 +69,7 @@
     {
         if (! append)
         {
-            geometry::detail::convert(source, exterior_ring(destination));
+            geometry::convert(source, exterior_ring(destination));
             if (reverse)
             {
                 boost::reverse(exterior_ring(destination));
@@ -79,7 +79,7 @@
         {
             interior_rings(destination).resize(
                         interior_rings(destination).size() + 1);
-            geometry::detail::convert(source, interior_rings(destination).back());
+            geometry::convert(source, interior_rings(destination).back());
             if (reverse)
             {
                 boost::reverse(interior_rings(destination).back());
Modified: trunk/boost/geometry/algorithms/detail/overlay/copy_segment_point.hpp
==============================================================================
--- trunk/boost/geometry/algorithms/detail/overlay/copy_segment_point.hpp	(original)
+++ trunk/boost/geometry/algorithms/detail/overlay/copy_segment_point.hpp	2011-04-16 07:31:53 EDT (Sat, 16 Apr 2011)
@@ -17,7 +17,7 @@
 #include <boost/geometry/core/ring_type.hpp>
 #include <boost/geometry/core/exterior_ring.hpp>
 #include <boost/geometry/core/interior_rings.hpp>
-#include <boost/geometry/algorithms/detail/convert.hpp>
+#include <boost/geometry/algorithms/convert.hpp>
 #include <boost/geometry/geometries/concepts/check.hpp>
 #include <boost/geometry/views/closeable_view.hpp>
 #include <boost/geometry/views/reversible_view.hpp>
@@ -71,7 +71,7 @@
         rview_type view(cview);
 
 
-        geometry::detail::convert(*(boost::begin(view) + index), point);
+        geometry::convert(*(boost::begin(view) + index), point);
         return true;
     }
 };
Modified: trunk/boost/geometry/algorithms/detail/overlay/get_intersection_points.hpp
==============================================================================
--- trunk/boost/geometry/algorithms/detail/overlay/get_intersection_points.hpp	(original)
+++ trunk/boost/geometry/algorithms/detail/overlay/get_intersection_points.hpp	2011-04-16 07:31:53 EDT (Sat, 16 Apr 2011)
@@ -12,7 +12,7 @@
 
 #include <cstddef>
 
-#include <boost/geometry/algorithms/detail/convert.hpp>
+#include <boost/geometry/algorithms/convert.hpp>
 #include <boost/geometry/algorithms/detail/overlay/get_turns.hpp>
 
 
@@ -64,7 +64,7 @@
         {
 
             TurnInfo tp;
-            geometry::detail::convert(result.template get<0>().intersections[i], tp.point);
+            geometry::convert(result.template get<0>().intersections[i], tp.point);
             *out++ = tp;
         }
 
Modified: trunk/boost/geometry/algorithms/detail/overlay/get_turn_info.hpp
==============================================================================
--- trunk/boost/geometry/algorithms/detail/overlay/get_turn_info.hpp	(original)
+++ trunk/boost/geometry/algorithms/detail/overlay/get_turn_info.hpp	2011-04-16 07:31:53 EDT (Sat, 16 Apr 2011)
@@ -14,7 +14,7 @@
 #include <boost/geometry/core/access.hpp>
 #include <boost/geometry/strategies/intersection.hpp>
 
-#include <boost/geometry/algorithms/detail/convert.hpp>
+#include <boost/geometry/algorithms/convert.hpp>
 #include <boost/geometry/algorithms/detail/overlay/turn_info.hpp>
 
 
@@ -94,7 +94,7 @@
                 DirInfo const& dir_info)
     {
         ti.method = method_touch_interior;
-        geometry::detail::convert(intersection_info.intersections[0], ti.point);
+        geometry::convert(intersection_info.intersections[0], ti.point);
 
         // Both segments of q touch segment p somewhere in its interior
         // 1) We know: if q comes from LEFT or RIGHT
@@ -213,7 +213,7 @@
                 DirInfo const& dir_info)
     {
         ti.method = method_touch;
-        geometry::detail::convert(intersection_info.intersections[0], ti.point);
+        geometry::convert(intersection_info.intersections[0], ti.point);
 
         int const side_qi_p1 = dir_info.sides.template get<1, 0>();
         int const side_qk_p1 = SideStrategy::apply(pi, pj, qk);
@@ -404,7 +404,7 @@
     {
         ti.method = method_equal;
         // Copy the SECOND intersection point
-        geometry::detail::convert(intersection_info.intersections[1], ti.point);
+        geometry::convert(intersection_info.intersections[1], ti.point);
 
         int const side_pk_q2  = SideStrategy::apply(qj, qk, pk);
         int const side_pk_p = SideStrategy::apply(pi, pj, pk);
@@ -486,7 +486,7 @@
                 DirInfo const& dir_info)
     {
         ti.method = method_collinear;
-        geometry::detail::convert(intersection_info.intersections[1], ti.point);
+        geometry::convert(intersection_info.intersections[1], ti.point);
 
         int const arrival = dir_info.arrival[0];
         // Should not be 0, this is checked before
@@ -585,7 +585,7 @@
         // If P arrives within Q, set info on P (which is done above, index=0),
         // this turn-info belongs to the second intersection point, index=1
         // (see e.g. figure CLO1)
-        geometry::detail::convert(intersection_info.intersections[1 - Index], tp.point);
+        geometry::convert(intersection_info.intersections[1 - Index], tp.point);
         return true;
     }
 
@@ -661,7 +661,7 @@
                 DirInfo const& dir_info)
     {
         ti.method = method_crosses;
-        geometry::detail::convert(intersection_info.intersections[0], ti.point);
+        geometry::convert(intersection_info.intersections[0], ti.point);
 
         // In all casees:
         // If Q crosses P from left to right
Modified: trunk/boost/geometry/algorithms/detail/overlay/intersection_insert.hpp
==============================================================================
--- trunk/boost/geometry/algorithms/detail/overlay/intersection_insert.hpp	(original)
+++ trunk/boost/geometry/algorithms/detail/overlay/intersection_insert.hpp	2011-04-16 07:31:53 EDT (Sat, 16 Apr 2011)
@@ -21,7 +21,7 @@
 #include <boost/geometry/core/point_order.hpp>
 #include <boost/geometry/core/reverse_dispatch.hpp>
 #include <boost/geometry/geometries/concepts/check.hpp>
-#include <boost/geometry/algorithms/detail/convert.hpp>
+#include <boost/geometry/algorithms/convert.hpp>
 #include <boost/geometry/algorithms/detail/overlay/clip_linestring.hpp>
 #include <boost/geometry/algorithms/detail/overlay/get_intersection_points.hpp>
 #include <boost/geometry/algorithms/detail/overlay/overlay.hpp>
@@ -65,7 +65,7 @@
         for (std::size_t i = 0; i < is.count; i++)
         {
             PointOut p;
-            geometry::detail::convert(is.intersections[i], p);
+            geometry::convert(is.intersections[i], p);
             *out++ = p;
         }
         return out;
@@ -95,7 +95,7 @@
             it = boost::begin(turns); it != boost::end(turns); ++it)
         {
             PointOut p;
-            geometry::detail::convert(it->point, p);
+            geometry::convert(it->point, p);
             *out++ = p;
         }
         return out;
Modified: trunk/boost/geometry/algorithms/detail/sections/sectionalize.hpp
==============================================================================
--- trunk/boost/geometry/algorithms/detail/sections/sectionalize.hpp	(original)
+++ trunk/boost/geometry/algorithms/detail/sections/sectionalize.hpp	2011-04-16 07:31:53 EDT (Sat, 16 Apr 2011)
@@ -461,8 +461,10 @@
         // (or polygon would be a helper-type).
         // Therefore we mimic a linestring/std::vector of 5 points
 
+        // TODO: might be replaced by assign_box_corners_oriented 
+        // or just "convert"
         point_type ll, lr, ul, ur;
-        assign_box_corners(box, ll, lr, ul, ur);
+        geometry::detail::assign_box_corners(box, ll, lr, ul, ur);
 
         std::vector<point_type> points;
         points.push_back(ll);
Modified: trunk/boost/geometry/algorithms/distance.hpp
==============================================================================
--- trunk/boost/geometry/algorithms/distance.hpp	(original)
+++ trunk/boost/geometry/algorithms/distance.hpp	2011-04-16 07:31:53 EDT (Sat, 16 Apr 2011)
@@ -78,8 +78,8 @@
             >::type segment_strategy;
 
         typename point_type<Segment>::type p[2];
-        geometry::assign_point_from_index<0>(segment, p[0]);
-        geometry::assign_point_from_index<1>(segment, p[1]);
+        geometry::detail::assign_point_from_index<0>(segment, p[0]);
+        geometry::detail::assign_point_from_index<1>(segment, p[1]);
         return segment_strategy.apply(point, p[0], p[1]);
     }
 };
@@ -419,8 +419,8 @@
     {
         
         typename point_type<Segment>::type p[2];
-        geometry::assign_point_from_index<0>(segment, p[0]);
-        geometry::assign_point_from_index<1>(segment, p[1]);
+        geometry::detail::assign_point_from_index<0>(segment, p[0]);
+        geometry::detail::assign_point_from_index<1>(segment, p[1]);
         return strategy.apply(point, p[0], p[1]);
     }
 };
Modified: trunk/boost/geometry/algorithms/length.hpp
==============================================================================
--- trunk/boost/geometry/algorithms/length.hpp	(original)
+++ trunk/boost/geometry/algorithms/length.hpp	2011-04-16 07:31:53 EDT (Sat, 16 Apr 2011)
@@ -50,8 +50,8 @@
     {
         typedef typename point_type<Segment>::type point_type;
         point_type p1, p2;
-        assign_point_from_index<0>(segment, p1);
-        assign_point_from_index<1>(segment, p2);
+        geometry::detail::assign_point_from_index<0>(segment, p1);
+        geometry::detail::assign_point_from_index<1>(segment, p2);
         return strategy.apply(p1, p2);
     }
 };
Modified: trunk/boost/geometry/algorithms/simplify.hpp
==============================================================================
--- trunk/boost/geometry/algorithms/simplify.hpp	(original)
+++ trunk/boost/geometry/algorithms/simplify.hpp	2011-04-16 07:31:53 EDT (Sat, 16 Apr 2011)
@@ -32,7 +32,7 @@
 #include <boost/geometry/strategies/concepts/simplify_concept.hpp>
 
 #include <boost/geometry/algorithms/clear.hpp>
-#include <boost/geometry/algorithms/detail/convert.hpp>
+#include <boost/geometry/algorithms/convert.hpp>
 #include <boost/geometry/algorithms/num_interior_rings.hpp>
 
 
@@ -181,7 +181,7 @@
     static inline void apply(Point const& point, Point& out,
                     Distance const& max_distance, Strategy const& strategy)
     {
-        geometry::detail::convert(point, out);
+        geometry::convert(point, out);
     }
 };
 
Modified: trunk/boost/geometry/algorithms/transform.hpp
==============================================================================
--- trunk/boost/geometry/algorithms/transform.hpp	(original)
+++ trunk/boost/geometry/algorithms/transform.hpp	2011-04-16 07:31:53 EDT (Sat, 16 Apr 2011)
@@ -101,15 +101,15 @@
         typedef typename point_type<Geometry2>::type point_type2;
 
         point_type1 source_point[2];
-        assign_point_from_index<0>(source, source_point[0]);
-        assign_point_from_index<1>(source, source_point[1]);
+        geometry::detail::assign_point_from_index<0>(source, source_point[0]);
+        geometry::detail::assign_point_from_index<1>(source, source_point[1]);
 
         point_type2 target_point[2];
         if (strategy.apply(source_point[0], target_point[0])
             && strategy.apply(source_point[1], target_point[1]))
         {
-            assign_point_to_index<0>(target_point[0], target);
-            assign_point_to_index<1>(target_point[1], target);
+            geometry::detail::assign_point_to_index<0>(target_point[0], target);
+            geometry::detail::assign_point_to_index<1>(target_point[1], target);
             return true;
         }
         return false;
Modified: trunk/boost/geometry/domains/gis/io/wkt/read_wkt.hpp
==============================================================================
--- trunk/boost/geometry/domains/gis/io/wkt/read_wkt.hpp	(original)
+++ trunk/boost/geometry/domains/gis/io/wkt/read_wkt.hpp	2011-04-16 07:31:53 EDT (Sat, 16 Apr 2011)
@@ -531,8 +531,8 @@
             throw read_wkt_exception("Box should have 2,4 or 5 points", wkt);
         }
 
-        assign_point_to_index<min_corner>(points.front(), box);
-        assign_point_to_index<max_corner>(points[index], box);
+        geometry::detail::assign_point_to_index<min_corner>(points.front(), box);
+        geometry::detail::assign_point_to_index<max_corner>(points[index], box);
     }
 };
 
@@ -570,8 +570,8 @@
 
         if (boost::size(points) == 2)
         {
-            assign_point_to_index<0>(points.front(), segment);
-            assign_point_to_index<1>(points.back(), segment);
+            geometry::detail::assign_point_to_index<0>(points.front(), segment);
+            geometry::detail::assign_point_to_index<1>(points.back(), segment);
         }
         else
         {
Modified: trunk/boost/geometry/domains/gis/io/wkt/write_wkt.hpp
==============================================================================
--- trunk/boost/geometry/domains/gis/io/wkt/write_wkt.hpp	(original)
+++ trunk/boost/geometry/domains/gis/io/wkt/write_wkt.hpp	2011-04-16 07:31:53 EDT (Sat, 16 Apr 2011)
@@ -24,7 +24,7 @@
 
 
 #include <boost/geometry/algorithms/assign.hpp>
-#include <boost/geometry/algorithms/detail/convert.hpp>
+#include <boost/geometry/algorithms/convert.hpp>
 #include <boost/geometry/core/exterior_ring.hpp>
 #include <boost/geometry/core/interior_rings.hpp>
 #include <boost/geometry/core/ring_type.hpp>
@@ -199,7 +199,7 @@
         // Convert to ring, then stream
         typedef model::ring<point_type> ring_type;
         ring_type ring;
-        geometry::detail::convert(box, ring);
+        geometry::convert(box, ring);
         os << "POLYGON(";
         wkt_sequence<ring_type>::apply(os, ring);
         os << ")";
@@ -228,8 +228,8 @@
         typedef boost::array<point_type, 2> sequence;
 
         sequence points;
-        assign_point_from_index<0>(segment, points[0]);
-        assign_point_from_index<1>(segment, points[1]);
+        geometry::detail::assign_point_from_index<0>(segment, points[0]);
+        geometry::detail::assign_point_from_index<1>(segment, points[1]);
 
         // In Boost.Geometry a segment is represented
         // in WKT-format like (for 2D): LINESTRING(x y,x y)
Modified: trunk/boost/geometry/extensions/algorithms/dissolve.hpp
==============================================================================
--- trunk/boost/geometry/extensions/algorithms/dissolve.hpp	(original)
+++ trunk/boost/geometry/extensions/algorithms/dissolve.hpp	2011-04-16 07:31:53 EDT (Sat, 16 Apr 2011)
@@ -33,7 +33,7 @@
 #include <boost/geometry/algorithms/detail/overlay/ring_properties.hpp>
 #include <boost/geometry/algorithms/detail/overlay/select_rings.hpp>
 
-#include <boost/geometry/algorithms/detail/convert.hpp>
+#include <boost/geometry/algorithms/convert.hpp>
 
 #include <boost/geometry/geometries/concepts/check.hpp>
 
@@ -134,7 +134,7 @@
         else
         {
             GeometryOut g;
-            geometry::detail::convert(geometry, g);
+            geometry::convert(geometry, g);
             *out++ = g;
             return out;
         }
Modified: trunk/boost/geometry/extensions/gis/projections/project_inverse_transformer.hpp
==============================================================================
--- trunk/boost/geometry/extensions/gis/projections/project_inverse_transformer.hpp	(original)
+++ trunk/boost/geometry/extensions/gis/projections/project_inverse_transformer.hpp	2011-04-16 07:31:53 EDT (Sat, 16 Apr 2011)
@@ -13,7 +13,7 @@
 #include <boost/shared_ptr.hpp>
 
 #include <boost/geometry/core/coordinate_dimension.hpp>
-#include <boost/geometry/algorithms/detail/convert.hpp>
+#include <boost/geometry/algorithms/convert.hpp>
 #include <boost/geometry/extensions/gis/projections/factory.hpp>
 #include <boost/geometry/extensions/gis/projections/parameters.hpp>
 
Modified: trunk/boost/geometry/extensions/gis/projections/project_transformer.hpp
==============================================================================
--- trunk/boost/geometry/extensions/gis/projections/project_transformer.hpp	(original)
+++ trunk/boost/geometry/extensions/gis/projections/project_transformer.hpp	2011-04-16 07:31:53 EDT (Sat, 16 Apr 2011)
@@ -13,7 +13,7 @@
 #include <boost/shared_ptr.hpp>
 
 #include <boost/geometry/core/coordinate_dimension.hpp>
-#include <boost/geometry/algorithms/detail/convert.hpp>
+#include <boost/geometry/algorithms/convert.hpp>
 #include <boost/geometry/extensions/gis/projections/factory.hpp>
 #include <boost/geometry/extensions/gis/projections/parameters.hpp>
 
Modified: trunk/boost/geometry/extensions/nsphere/geometries/nsphere.hpp
==============================================================================
--- trunk/boost/geometry/extensions/nsphere/geometries/nsphere.hpp	(original)
+++ trunk/boost/geometry/extensions/nsphere/geometries/nsphere.hpp	2011-04-16 07:31:53 EDT (Sat, 16 Apr 2011)
@@ -17,7 +17,7 @@
 #include <cstddef>
 
 #include <boost/geometry/algorithms/assign.hpp>
-#include <boost/geometry/algorithms/detail/convert.hpp>
+#include <boost/geometry/algorithms/convert.hpp>
 #include <boost/geometry/geometries/concepts/point_concept.hpp>
 
 
@@ -63,7 +63,7 @@
     nsphere(P const& center, T const& radius)
         : m_radius(radius)
     {
-        geometry::detail::convert(center, m_center);
+        geometry::convert(center, m_center);
     }
 
     inline P const& center() const { return m_center; }
Modified: trunk/boost/geometry/extensions/strategies/buffer_join_round.hpp
==============================================================================
--- trunk/boost/geometry/extensions/strategies/buffer_join_round.hpp	(original)
+++ trunk/boost/geometry/extensions/strategies/buffer_join_round.hpp	2011-04-16 07:31:53 EDT (Sat, 16 Apr 2011)
@@ -16,7 +16,7 @@
 
 
 
-#include <boost/geometry/algorithms/detail/convert.hpp>
+#include <boost/geometry/algorithms/convert.hpp>
 #include <boost/geometry/arithmetic/arithmetic.hpp>
 #include <boost/geometry/arithmetic/dot_product.hpp>
 #include <boost/geometry/core/cs.hpp>
@@ -51,7 +51,7 @@
     static inline Vector create_vector(Point1 const& p1, Point2 const& p2)
     {
         Vector v;
-        geometry::detail::convert(p1, v);
+        geometry::convert(p1, v);
         subtract_point(v, p2);
         return v;
     }
@@ -111,7 +111,7 @@
                     PointOut projected_point;
 
                     multiply_value(v, b);
-                    geometry::detail::convert(vertex, projected_point);
+                    geometry::convert(vertex, projected_point);
                     add_point(projected_point, v);
 
                     coordinate_type projected_distance = geometry::distance(projected_point, mid_point);
Modified: trunk/boost/geometry/geometries/box.hpp
==============================================================================
--- trunk/boost/geometry/geometries/box.hpp	(original)
+++ trunk/boost/geometry/geometries/box.hpp	2011-04-16 07:31:53 EDT (Sat, 16 Apr 2011)
@@ -18,7 +18,7 @@
 
 #include <boost/concept/assert.hpp>
 
-#include <boost/geometry/algorithms/detail/convert.hpp>
+#include <boost/geometry/algorithms/convert.hpp>
 #include <boost/geometry/geometries/concepts/point_concept.hpp>
 
 
@@ -57,8 +57,8 @@
     */
     inline box(Point const& min_corner, Point const& max_corner)
     {
-        geometry::detail::convert(min_corner, m_min_corner);
-        geometry::detail::convert(max_corner, m_max_corner);
+        geometry::convert(min_corner, m_min_corner);
+        geometry::convert(max_corner, m_max_corner);
     }
 
     inline Point const& min_corner() const { return m_min_corner; }
Modified: trunk/boost/geometry/iterators/box_iterator.hpp
==============================================================================
--- trunk/boost/geometry/iterators/box_iterator.hpp	(original)
+++ trunk/boost/geometry/iterators/box_iterator.hpp	2011-04-16 07:31:53 EDT (Sat, 16 Apr 2011)
@@ -95,9 +95,7 @@
 
     inline void init(Box const& box)
     {
-        // asb -> lower_left, lower_right, upper_left, upper_right
-        // we want: clockwise
-        assign_box_corners(box, m_points[0], m_points[3], m_points[1], m_points[2]);
+        detail::assign_box_corners_oriented<false>(box, m_points);
     }
 
     // Copy points here - box might define them otherwise
Modified: trunk/boost/geometry/iterators/segment_range_iterator.hpp
==============================================================================
--- trunk/boost/geometry/iterators/segment_range_iterator.hpp	(original)
+++ trunk/boost/geometry/iterators/segment_range_iterator.hpp	2011-04-16 07:31:53 EDT (Sat, 16 Apr 2011)
@@ -115,8 +115,8 @@
 
     inline void init(Segment const& segment)
     {
-        assign_point_from_index<0>(segment, m_points[0]);
-        assign_point_from_index<1>(segment, m_points[1]);
+        geometry::detail::assign_point_from_index<0>(segment, m_points[0]);
+        geometry::detail::assign_point_from_index<1>(segment, m_points[1]);
     }
 
     // We HAVE TO copy the points, because a segment does not need
Modified: trunk/boost/geometry/strategies/cartesian/distance_projected_point.hpp
==============================================================================
--- trunk/boost/geometry/strategies/cartesian/distance_projected_point.hpp	(original)
+++ trunk/boost/geometry/strategies/cartesian/distance_projected_point.hpp	2011-04-16 07:31:53 EDT (Sat, 16 Apr 2011)
@@ -22,7 +22,7 @@
 #include <boost/geometry/core/access.hpp>
 #include <boost/geometry/core/point_type.hpp>
 
-#include <boost/geometry/algorithms/detail/convert.hpp>
+#include <boost/geometry/algorithms/convert.hpp>
 #include <boost/geometry/arithmetic/arithmetic.hpp>
 #include <boost/geometry/arithmetic/dot_product.hpp>
 
@@ -132,8 +132,8 @@
         // For consistency we define w also in FP
         fp_vector_type v, w;
 
-        geometry::detail::convert(p2, v);
-        geometry::detail::convert(p, w);
+        geometry::convert(p2, v);
+        geometry::convert(p, w);
         subtract_point(v, p1);
         subtract_point(w, p1);
 
@@ -163,7 +163,7 @@
                 >::apply(strategy);
 
         fp_point_type projected;
-        geometry::detail::convert(p1, projected);
+        geometry::convert(p1, projected);
         multiply_value(v, b);
         add_point(projected, v);
 
Modified: trunk/boost/geometry/strategies/strategy_transform.hpp
==============================================================================
--- trunk/boost/geometry/strategies/strategy_transform.hpp	(original)
+++ trunk/boost/geometry/strategies/strategy_transform.hpp	2011-04-16 07:31:53 EDT (Sat, 16 Apr 2011)
@@ -20,7 +20,7 @@
 
 #include <boost/numeric/conversion/cast.hpp>
 
-#include <boost/geometry/algorithms/detail/convert.hpp>
+#include <boost/geometry/algorithms/convert.hpp>
 #include <boost/geometry/arithmetic/arithmetic.hpp>
 #include <boost/geometry/core/access.hpp>
 #include <boost/geometry/core/coordinate_dimension.hpp>
@@ -104,7 +104,7 @@
         // Defensive check, dimensions are equal, selected by specialization
         assert_dimension_equal<P1, P2>();
 
-        geometry::detail::convert(p1, p2);
+        geometry::convert(p1, p2);
         return true;
     }
 };