From c9950313373024e4f9b76279df7a007f794e4ebf Mon Sep 17 00:00:00 2001 From: Barend Gehrels Date: Wed, 14 May 2014 13:22:48 +0200 Subject: [PATCH 01/11] [intersection] avoid circulare reference in index/intersection/distance --- .../detail/overlay/intersection_box_box.hpp | 84 +++++++++++++++++++ .../geometry/algorithms/intersection.hpp | 63 +------------- .../algorithms/intersection_content.hpp | 10 ++- .../geometry/strategies/intersection.hpp | 3 +- 4 files changed, 95 insertions(+), 65 deletions(-) create mode 100644 include/boost/geometry/algorithms/detail/overlay/intersection_box_box.hpp diff --git a/include/boost/geometry/algorithms/detail/overlay/intersection_box_box.hpp b/include/boost/geometry/algorithms/detail/overlay/intersection_box_box.hpp new file mode 100644 index 000000000..dd041b0d7 --- /dev/null +++ b/include/boost/geometry/algorithms/detail/overlay/intersection_box_box.hpp @@ -0,0 +1,84 @@ +// Boost.Geometry (aka GGL, Generic Geometry Library) + +// Copyright (c) 2007-2014 Barend Gehrels, 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_OVERLAY_INTERSECTION_BOX_BOX_HPP +#define BOOST_GEOMETRY_ALGORITHMS_DETAIL_OVERLAY_INTERSECTION_BOX_BOX_HPP + + +#include +#include + + +namespace boost { namespace geometry +{ + +#ifndef DOXYGEN_NO_DETAIL +namespace detail { namespace intersection +{ + +template +struct intersection_box_box +{ + template + < + typename Box1, typename Box2, + typename RobustPolicy, + typename BoxOut, + typename Strategy + > + static inline bool apply(Box1 const& box1, + Box2 const& box2, + RobustPolicy const& robust_policy, + BoxOut& box_out, + Strategy const& strategy) + { + typedef typename coordinate_type::type ct; + + ct min1 = get(box1); + ct min2 = get(box2); + ct max1 = get(box1); + ct max2 = get(box2); + + if (max1 < min2 || max2 < min1) + { + return false; + } + // Set dimensions of output coordinate + set(box_out, min1 < min2 ? min2 : min1); + set(box_out, max1 > max2 ? max2 : max1); + + return intersection_box_box + ::apply(box1, box2, robust_policy, box_out, strategy); + } +}; + +template +struct intersection_box_box +{ + template + < + typename Box1, typename Box2, + typename RobustPolicy, + typename BoxOut, + typename Strategy + > + static inline bool apply(Box1 const&, Box2 const&, + RobustPolicy const&, BoxOut&, Strategy const&) + { + return true; + } +}; + + +}} // namespace detail::intersection +#endif // DOXYGEN_NO_DETAIL + +}} // namespace boost::geometry + + +#endif // BOOST_GEOMETRY_ALGORITHMS_DETAIL_OVERLAY_INTERSECTION_BOX_BOX_HPP diff --git a/include/boost/geometry/algorithms/intersection.hpp b/include/boost/geometry/algorithms/intersection.hpp index 500cacc9f..ca3bb0682 100644 --- a/include/boost/geometry/algorithms/intersection.hpp +++ b/include/boost/geometry/algorithms/intersection.hpp @@ -12,6 +12,7 @@ #include #include +#include #include #include @@ -19,68 +20,6 @@ namespace boost { namespace geometry { -#ifndef DOXYGEN_NO_DETAIL -namespace detail { namespace intersection -{ - -template -struct intersection_box_box -{ - template - < - typename Box1, typename Box2, - typename RobustPolicy, - typename BoxOut, - typename Strategy - > - static inline bool apply(Box1 const& box1, - Box2 const& box2, - RobustPolicy const& robust_policy, - BoxOut& box_out, - Strategy const& strategy) - { - typedef typename coordinate_type::type ct; - - ct min1 = get(box1); - ct min2 = get(box2); - ct max1 = get(box1); - ct max2 = get(box2); - - if (max1 < min2 || max2 < min1) - { - return false; - } - // Set dimensions of output coordinate - set(box_out, min1 < min2 ? min2 : min1); - set(box_out, max1 > max2 ? max2 : max1); - - return intersection_box_box - ::apply(box1, box2, robust_policy, box_out, strategy); - } -}; - -template -struct intersection_box_box -{ - template - < - typename Box1, typename Box2, - typename RobustPolicy, - typename BoxOut, - typename Strategy - > - static inline bool apply(Box1 const&, Box2 const&, - RobustPolicy const&, BoxOut&, Strategy const&) - { - return true; - } -}; - - -}} // namespace detail::intersection -#endif // DOXYGEN_NO_DETAIL - - #ifndef DOXYGEN_NO_DISPATCH namespace dispatch diff --git a/include/boost/geometry/index/detail/algorithms/intersection_content.hpp b/include/boost/geometry/index/detail/algorithms/intersection_content.hpp index 955d6eb65..ed615c402 100644 --- a/include/boost/geometry/index/detail/algorithms/intersection_content.hpp +++ b/include/boost/geometry/index/detail/algorithms/intersection_content.hpp @@ -11,7 +11,8 @@ #ifndef BOOST_GEOMETRY_INDEX_DETAIL_ALGORITHMS_INTERSECTION_CONTENT_HPP #define BOOST_GEOMETRY_INDEX_DETAIL_ALGORITHMS_INTERSECTION_CONTENT_HPP -#include +#include +#include #include namespace boost { namespace geometry { namespace index { namespace detail { @@ -25,7 +26,12 @@ inline typename default_content_result::type intersection_content(Box const if ( geometry::intersects(box1, box2) ) { Box box_intersection; - geometry::intersection(box1, box2, box_intersection); + + strategy_intersection_empty dummy; + geometry::detail::intersection::intersection_box_box + < + 0, geometry::dimension::value + >::apply(box1, box2, box_intersection, dummy); return detail::content(box_intersection); } return 0; diff --git a/include/boost/geometry/strategies/intersection.hpp b/include/boost/geometry/strategies/intersection.hpp index d97baf66d..ef1b676fd 100644 --- a/include/boost/geometry/strategies/intersection.hpp +++ b/include/boost/geometry/strategies/intersection.hpp @@ -82,7 +82,8 @@ public: typedef RobustPolicy rescale_policy_type; }; - +// Version for box_box intersection or other detail calls not needing a strategy +struct strategy_intersection_empty {}; }} // namespace boost::geometry From 07696d76fdeedd8ac091f6c9b8a0ff64a6920954 Mon Sep 17 00:00:00 2001 From: Barend Gehrels Date: Wed, 14 May 2014 13:46:22 +0200 Subject: [PATCH 02/11] [overlay] Added missing include / removed obsolete include, causing unit test troubles after distance merge --- .../geometry/algorithms/detail/overlay/get_relative_order.hpp | 2 -- include/boost/geometry/policies/robustness/rescale_policy.hpp | 1 + 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/include/boost/geometry/algorithms/detail/overlay/get_relative_order.hpp b/include/boost/geometry/algorithms/detail/overlay/get_relative_order.hpp index 50a14ea24..d71f4ad51 100644 --- a/include/boost/geometry/algorithms/detail/overlay/get_relative_order.hpp +++ b/include/boost/geometry/algorithms/detail/overlay/get_relative_order.hpp @@ -10,8 +10,6 @@ #define BOOST_GEOMETRY_ALGORITHMS_DETAIL_OVERLAY_GET_RELATIVE_ORDER_HPP -#include - #include diff --git a/include/boost/geometry/policies/robustness/rescale_policy.hpp b/include/boost/geometry/policies/robustness/rescale_policy.hpp index 570ba8bef..5b3b56697 100644 --- a/include/boost/geometry/policies/robustness/rescale_policy.hpp +++ b/include/boost/geometry/policies/robustness/rescale_policy.hpp @@ -16,6 +16,7 @@ #include +#include #include #include From 09365f00869a34acc762c158b52d6991667ef437 Mon Sep 17 00:00:00 2001 From: Barend Gehrels Date: Wed, 14 May 2014 13:46:52 +0200 Subject: [PATCH 03/11] [unit test] Added copyright info --- test/algorithms/from_wkt.hpp | 12 ++++++++++++ 1 file changed, 12 insertions(+) diff --git a/test/algorithms/from_wkt.hpp b/test/algorithms/from_wkt.hpp index 13d646356..2e47f2e24 100644 --- a/test/algorithms/from_wkt.hpp +++ b/test/algorithms/from_wkt.hpp @@ -1,3 +1,15 @@ +// Boost.Geometry (aka GGL, Generic Geometry Library) +// Unit Tests + +// Copyright (c) 2007-2014 Barend Gehrels, Amsterdam, the Netherlands. +// Copyright (c) 2008-2014 Bruno Lalande, Paris, France. +// Copyright (c) 2009-2014 Mateusz Loskot, London, UK. +// Copyright (c) 2013-2014 Adam Wulkiewicz, Lodz, Poland. + +// 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_TEST_FROM_WKT_HPP #define BOOST_GEOMETRY_TEST_FROM_WKT_HPP From afb301f6ed6b52f3968bfcc071128d39470b67ec Mon Sep 17 00:00:00 2001 From: Barend Gehrels Date: Wed, 14 May 2014 13:47:22 +0200 Subject: [PATCH 04/11] [unit test] Made from_wkt inline and avoid abbreviations --- test/algorithms/from_wkt.hpp | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/test/algorithms/from_wkt.hpp b/test/algorithms/from_wkt.hpp index 2e47f2e24..5390db868 100644 --- a/test/algorithms/from_wkt.hpp +++ b/test/algorithms/from_wkt.hpp @@ -16,11 +16,11 @@ #include template -Geometry from_wkt(std::string const& wkt) +inline Geometry from_wkt(std::string const& wkt) { - Geometry res; - boost::geometry::read_wkt(wkt, res); - return res; + Geometry result; + boost::geometry::read_wkt(wkt, result); + return result; } #endif // BOOST_GEOMETRY_TEST_FROM_WKT_HPP From a703cae46e887a49fd70f17cc3e942006a81e4fd Mon Sep 17 00:00:00 2001 From: Barend Gehrels Date: Wed, 14 May 2014 14:18:48 +0200 Subject: [PATCH 05/11] [num_points][test] Added test for num_points which was not yet there --- test/algorithms/Jamfile.v2 | 1 + test/algorithms/num_points.cpp | 65 ++++++++++++++++++++++++++++++++++ 2 files changed, 66 insertions(+) create mode 100644 test/algorithms/num_points.cpp diff --git a/test/algorithms/Jamfile.v2 b/test/algorithms/Jamfile.v2 index de35ec60e..63b7fe3c4 100644 --- a/test/algorithms/Jamfile.v2 +++ b/test/algorithms/Jamfile.v2 @@ -47,6 +47,7 @@ test-suite boost-geometry-algorithms [ run intersects.cpp : : : msvc:/bigobj ] [ run length.cpp ] [ run make.cpp ] + [ run num_points.cpp ] [ run overlaps.cpp ] [ run perimeter.cpp ] [ run point_on_surface.cpp ] diff --git a/test/algorithms/num_points.cpp b/test/algorithms/num_points.cpp new file mode 100644 index 000000000..3abeed449 --- /dev/null +++ b/test/algorithms/num_points.cpp @@ -0,0 +1,65 @@ +// Boost.Geometry (aka GGL, Generic Geometry Library) +// Unit Test + +// Copyright (c) 2007-2014 Barend Gehrels, Amsterdam, the Netherlands. +// Copyright (c) 2008-2014 Bruno Lalande, Paris, France. +// Copyright (c) 2009-2014 Mateusz Loskot, London, UK. +// Copyright (c) 2013-2014 Adam Wulkiewicz, Lodz, Poland. + +// 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_TEST_MODULE +#define BOOST_TEST_MODULE test_num_points +#endif + +#include +#include + +#include +#include +#include +#include + +#include +#include + +template +inline void test_num_points(std::string const& wkt, std::size_t expected) +{ + namespace bg = boost::geometry; + Geometry geometry; + boost::geometry::read_wkt(wkt, geometry); + std::size_t detected = bg::num_points(geometry); + BOOST_CHECK_EQUAL(expected, detected); + detected = bg::num_points(geometry, false); + BOOST_CHECK_EQUAL(expected, detected); + detected = bg::num_points(geometry, true); +} + +BOOST_AUTO_TEST_CASE( test_num_points_closed ) +{ + namespace bg = boost::geometry; + typedef bg::model::point point; + typedef bg::model::linestring linestring; + typedef bg::model::segment segment; + typedef bg::model::box box; + typedef bg::model::ring ring; + typedef bg::model::polygon polygon; + typedef bg::model::multi_point multi_point; + typedef bg::model::multi_linestring multi_linestring; + typedef bg::model::multi_polygon multi_polygon; + + test_num_points("POINT(0 0)", 1u); + test_num_points("LINESTRING(0 0,1 1)", 2u); + test_num_points("LINESTRING(0 0,1 1)", 2u); + test_num_points("POLYGON((0 0,10 10))", 4u); + test_num_points("POLYGON((0 0,1 1,0 1,0 0))", 4u); + test_num_points("POLYGON((0 0,10 10,0 10,0 0))", 4u); + test_num_points("POLYGON((0 0,0 10,10 10,10 0,0 0),(4 4,6 4,6 6,4 6,4 4))", 10u); + test_num_points("MULTIPOINT((0 0),(1 1))", 2u); + test_num_points("MULTILINESTRING((0 0,1 1),(2 2,3 3,4 4))", 5u); + test_num_points("MULTIPOLYGON(((0 0,0 10,10 10,10 0,0 0)),((0 10,1 10,1 9,0 10)))", 9u); +} + From 71be9ab855a6f21113672bb0a2443166a2ef2d82 Mon Sep 17 00:00:00 2001 From: Barend Gehrels Date: Wed, 14 May 2014 14:19:04 +0200 Subject: [PATCH 06/11] [test] added include for multi --- test/algorithms/from_wkt.hpp | 1 + 1 file changed, 1 insertion(+) diff --git a/test/algorithms/from_wkt.hpp b/test/algorithms/from_wkt.hpp index 5390db868..5c78afdea 100644 --- a/test/algorithms/from_wkt.hpp +++ b/test/algorithms/from_wkt.hpp @@ -14,6 +14,7 @@ #define BOOST_GEOMETRY_TEST_FROM_WKT_HPP #include +#include template inline Geometry from_wkt(std::string const& wkt) From f6d0b4054ae735e60e917cf8c56edfa689d91207 Mon Sep 17 00:00:00 2001 From: Menelaos Karavelas Date: Wed, 14 May 2014 15:20:57 +0300 Subject: [PATCH 07/11] [distance] fix unused variable warnings produced by clang++ with -Wextra and warnings by VS (reported by Adam) --- .../algorithms/detail/distance/backward_compatibility.hpp | 6 +++--- .../geometry/algorithms/detail/distance/box_to_box.hpp | 1 + .../algorithms/detail/distance/point_to_geometry.hpp | 4 +++- .../algorithms/detail/distance/range_to_segment_or_box.hpp | 3 +++ .../geometry/algorithms/detail/distance/segment_to_box.hpp | 6 ++++++ 5 files changed, 16 insertions(+), 4 deletions(-) diff --git a/include/boost/geometry/algorithms/detail/distance/backward_compatibility.hpp b/include/boost/geometry/algorithms/detail/distance/backward_compatibility.hpp index 603963cbf..5dd69cbcd 100644 --- a/include/boost/geometry/algorithms/detail/distance/backward_compatibility.hpp +++ b/include/boost/geometry/algorithms/detail/distance/backward_compatibility.hpp @@ -110,7 +110,7 @@ struct distance static inline typename return_type::type>::type apply(Point const& point, Linestring const& linestring, - Strategy const& strategy) + Strategy const&) { typedef typename detail::distance::default_ps_strategy < @@ -143,7 +143,7 @@ struct distance static inline return_type apply(Point const& point, Ring const& ring, - Strategy const& strategy) + Strategy const&) { typedef typename detail::distance::default_ps_strategy < @@ -181,7 +181,7 @@ struct distance static inline return_type apply(Point const& point, Polygon const& polygon, - Strategy const& strategy) + Strategy const&) { typedef typename detail::distance::default_ps_strategy < diff --git a/include/boost/geometry/algorithms/detail/distance/box_to_box.hpp b/include/boost/geometry/algorithms/detail/distance/box_to_box.hpp index 98b3be0f7..7b032a19c 100644 --- a/include/boost/geometry/algorithms/detail/distance/box_to_box.hpp +++ b/include/boost/geometry/algorithms/detail/distance/box_to_box.hpp @@ -43,6 +43,7 @@ struct distance >::type apply(Box1 const& box1, Box2 const& box2, Strategy const& strategy) { + boost::ignore_unused_variable_warning(strategy); return strategy.apply(box1, box2); } }; diff --git a/include/boost/geometry/algorithms/detail/distance/point_to_geometry.hpp b/include/boost/geometry/algorithms/detail/distance/point_to_geometry.hpp index 4d6a598ad..7b89dcad1 100644 --- a/include/boost/geometry/algorithms/detail/distance/point_to_geometry.hpp +++ b/include/boost/geometry/algorithms/detail/distance/point_to_geometry.hpp @@ -390,10 +390,11 @@ struct distance Segment const& segment, Strategy const& strategy) { - typename point_type::type p[2]; geometry::detail::assign_point_from_index<0>(segment, p[0]); geometry::detail::assign_point_from_index<1>(segment, p[1]); + + boost::ignore_unused_variable_warning(strategy); return strategy.apply(point, p[0], p[1]); } }; @@ -413,6 +414,7 @@ struct distance >::type apply(Point const& point, Box const& box, Strategy const& strategy) { + boost::ignore_unused_variable_warning(strategy); return strategy.apply(point, box); } }; diff --git a/include/boost/geometry/algorithms/detail/distance/range_to_segment_or_box.hpp b/include/boost/geometry/algorithms/detail/distance/range_to_segment_or_box.hpp index db2bcf68f..e1d113ff3 100644 --- a/include/boost/geometry/algorithms/detail/distance/range_to_segment_or_box.hpp +++ b/include/boost/geometry/algorithms/detail/distance/range_to_segment_or_box.hpp @@ -106,6 +106,7 @@ private: SegmentPoints const& segment_points, ComparableStrategy const& strategy) { + boost::ignore_unused_variable_warning(strategy); return strategy.apply(point, segment_points[0], segment_points[1]); } }; @@ -126,6 +127,8 @@ private: BoxPoints const& box_points, ComparableStrategy const& strategy) { + boost::ignore_unused_variable_warning(strategy); + comparable_return_type cd_min = strategy.apply(point, box_points[0], box_points[3]); diff --git a/include/boost/geometry/algorithms/detail/distance/segment_to_box.hpp b/include/boost/geometry/algorithms/detail/distance/segment_to_box.hpp index 3efa7673a..4a6f23839 100644 --- a/include/boost/geometry/algorithms/detail/distance/segment_to_box.hpp +++ b/include/boost/geometry/algorithms/detail/distance/segment_to_box.hpp @@ -81,6 +81,9 @@ private: { typedef cast_to_result cast; + boost::ignore_unused_variable_warning(pp_strategy); + boost::ignore_unused_variable_warning(ps_strategy); + // assert that the segment has non-negative slope BOOST_ASSERT( (math::equals(geometry::get<0>(p0), geometry::get<0>(p1)) @@ -217,6 +220,9 @@ private: { typedef cast_to_result cast; + boost::ignore_unused_variable_warning(pp_strategy); + boost::ignore_unused_variable_warning(ps_strategy); + // assert that the segment has negative slope BOOST_ASSERT ( geometry::get<0>(p0) < geometry::get<0>(p1) From 453a2c3784a4e2c625a589d3a2c70cea89d961a2 Mon Sep 17 00:00:00 2001 From: Menelaos Karavelas Date: Wed, 14 May 2014 15:39:12 +0300 Subject: [PATCH 08/11] [set ops L/L] fix unused variable warnings produced with clang++ with -Wextra --- .../geometry/algorithms/detail/overlay/follow_linear_linear.hpp | 2 +- .../geometry/algorithms/detail/turns/filter_continue_turns.hpp | 2 +- .../geometry/algorithms/detail/turns/remove_duplicate_turns.hpp | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/include/boost/geometry/algorithms/detail/overlay/follow_linear_linear.hpp b/include/boost/geometry/algorithms/detail/overlay/follow_linear_linear.hpp index 131154f82..6407706cb 100644 --- a/include/boost/geometry/algorithms/detail/overlay/follow_linear_linear.hpp +++ b/include/boost/geometry/algorithms/detail/overlay/follow_linear_linear.hpp @@ -289,7 +289,7 @@ protected: public: template static inline OutputIterator - apply(Linestring const& linestring, Linear const& linear, + apply(Linestring const& linestring, Linear const&, TurnIterator first, TurnIterator beyond, OutputIterator oit) { diff --git a/include/boost/geometry/algorithms/detail/turns/filter_continue_turns.hpp b/include/boost/geometry/algorithms/detail/turns/filter_continue_turns.hpp index 4f38e3e73..17fbd65dd 100644 --- a/include/boost/geometry/algorithms/detail/turns/filter_continue_turns.hpp +++ b/include/boost/geometry/algorithms/detail/turns/filter_continue_turns.hpp @@ -24,7 +24,7 @@ namespace detail { namespace turns template struct filter_continue_turns { - static inline void apply(Turns& turns) {} + static inline void apply(Turns&) {} }; diff --git a/include/boost/geometry/algorithms/detail/turns/remove_duplicate_turns.hpp b/include/boost/geometry/algorithms/detail/turns/remove_duplicate_turns.hpp index 41f52c059..d48736c8f 100644 --- a/include/boost/geometry/algorithms/detail/turns/remove_duplicate_turns.hpp +++ b/include/boost/geometry/algorithms/detail/turns/remove_duplicate_turns.hpp @@ -23,7 +23,7 @@ namespace detail { namespace turns template struct remove_duplicate_turns { - static inline void apply(Turns& turns) {} + static inline void apply(Turns&) {} }; From aebd3cefe25bd9c6098760baa0a5ef2155547382 Mon Sep 17 00:00:00 2001 From: Menelaos Karavelas Date: Wed, 14 May 2014 15:39:56 +0300 Subject: [PATCH 09/11] [test][set ops L/L] fix unused variable warnings produced with clang++ and -Wextra --- test/algorithms/test_set_ops_linear_linear.hpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/algorithms/test_set_ops_linear_linear.hpp b/test/algorithms/test_set_ops_linear_linear.hpp index 51d9d5794..c3ff6757d 100644 --- a/test/algorithms/test_set_ops_linear_linear.hpp +++ b/test/algorithms/test_set_ops_linear_linear.hpp @@ -92,7 +92,7 @@ struct multilinestring_equals template struct unique { - void operator()(MultiLinestring& mls) + void operator()(MultiLinestring&) { } }; From 1083dce4363c585af23bc810a53281846c3d7b60 Mon Sep 17 00:00:00 2001 From: Menelaos Karavelas Date: Wed, 14 May 2014 15:40:27 +0300 Subject: [PATCH 10/11] [test] fix unused variable warnings produced with clang++ and -Wextra --- test/to_svg.hpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/test/to_svg.hpp b/test/to_svg.hpp index 54891b7bf..9f7c37749 100644 --- a/test/to_svg.hpp +++ b/test/to_svg.hpp @@ -29,7 +29,7 @@ #include template -inline void turns_to_svg(Turns const& turns, Mapper & mapper, bool enrich = false) +inline void turns_to_svg(Turns const& turns, Mapper & mapper, bool /*enrich*/ = false) { // turn points in orange, + enrichment/traversal info typedef typename bg::coordinate_type::type coordinate_type; @@ -196,7 +196,7 @@ struct to_svg_assign_policy }; template -inline void to_svg(G const& g, std::string const& filename, bool sort = true) +inline void to_svg(G const& g, std::string const& filename, bool /*sort*/ = true) { namespace bg = boost::geometry; From 4f19973453f64538e537d7e08d3d4f733a53ba26 Mon Sep 17 00:00:00 2001 From: Barend Gehrels Date: Wed, 14 May 2014 14:41:24 +0200 Subject: [PATCH 11/11] [centroid] removed redundant distance include causing circular reference for unit test "geometries/adapted" --- include/boost/geometry/algorithms/centroid.hpp | 1 - 1 file changed, 1 deletion(-) diff --git a/include/boost/geometry/algorithms/centroid.hpp b/include/boost/geometry/algorithms/centroid.hpp index 106818d67..191866b9a 100644 --- a/include/boost/geometry/algorithms/centroid.hpp +++ b/include/boost/geometry/algorithms/centroid.hpp @@ -32,7 +32,6 @@ #include #include -#include #include #include #include