diff --git a/Mesh_3/benchmark/Mesh_3/StdAfx.h b/Mesh_3/benchmark/Mesh_3/StdAfx.h index 95803115a15..8587b921602 100644 --- a/Mesh_3/benchmark/Mesh_3/StdAfx.h +++ b/Mesh_3/benchmark/Mesh_3/StdAfx.h @@ -13,6 +13,7 @@ #include #include #include +#include // Windows #include @@ -24,7 +25,6 @@ #include #include #include -#include #include #include #include diff --git a/Periodic_3_mesh_3/include/CGAL/Periodic_3_mesh_3/Protect_edges_sizing_field.h b/Periodic_3_mesh_3/include/CGAL/Periodic_3_mesh_3/Protect_edges_sizing_field.h index 5e60842dd3d..5be36b550e6 100644 --- a/Periodic_3_mesh_3/include/CGAL/Periodic_3_mesh_3/Protect_edges_sizing_field.h +++ b/Periodic_3_mesh_3/include/CGAL/Periodic_3_mesh_3/Protect_edges_sizing_field.h @@ -2113,8 +2113,10 @@ insert_balls(const Vertex_handle& vp, curve_index, d_sign) << ")\n"; #endif + const FT sgn = (d_sign == CGAL::POSITIVE) ? 1. + : (d_sign == CGAL::NEGATIVE ? -1. : 0.); const Bare_point new_point = - domain_.construct_point_on_curve(vpp, curve_index, d_sign * d / 2); + domain_.construct_point_on_curve(vpp, curve_index, sgn * d / 2); const int dim = 1; // new_point is on edge const Index index = domain_.index_from_curve_index(curve_index); const FT point_weight = CGAL::square(size_(new_point, dim, index)); diff --git a/Profiling_tools/include/CGAL/Profile_counter.h b/Profiling_tools/include/CGAL/Profile_counter.h index 537d7640bee..518ccbc43eb 100644 --- a/Profiling_tools/include/CGAL/Profile_counter.h +++ b/Profiling_tools/include/CGAL/Profile_counter.h @@ -121,9 +121,9 @@ struct Profile_histogram_counter { private: #ifdef CGAL_CONCURRENT_PROFILE - typedef tbb::concurrent_hash_map Counters; + typedef tbb::concurrent_hash_map CounterMap; #else - typedef std::map Counters; + typedef std::map CounterMap; #endif public: @@ -133,7 +133,7 @@ public: void operator()(unsigned i) { #ifdef CGAL_CONCURRENT_PROFILE - Counters::accessor a; + CounterMap::accessor a; counters.insert(a, i); ++a->second; #else @@ -144,7 +144,7 @@ public: ~Profile_histogram_counter() { unsigned total=0; - for (Counters::const_iterator it=counters.begin(), end=counters.end(); + for (CounterMap::const_iterator it=counters.begin(), end=counters.end(); it != end; ++it) { std::cerr << "[CGAL::Profile_histogram_counter] " << s; std::cerr << " [ " << std::setw(10) << internal::dot_it(it->first) << " : " @@ -158,7 +158,7 @@ public: } private: - Counters counters; + CounterMap counters; const std::string s; }; diff --git a/STL_Extension/include/CGAL/STL_Extension/internal/boost/array_binary_tree.hpp b/STL_Extension/include/CGAL/STL_Extension/internal/boost/array_binary_tree.hpp index dba4dc93fb8..3e1fa963176 100644 --- a/STL_Extension/include/CGAL/STL_Extension/internal/boost/array_binary_tree.hpp +++ b/STL_Extension/include/CGAL/STL_Extension/internal/boost/array_binary_tree.hpp @@ -21,10 +21,9 @@ #define CGAL_INTERNAL_ARRAY_BINARY_TREE_HPP #include - #include -#include +#include #include namespace CGAL { namespace internal { diff --git a/Surface_mesh_simplification/test/Surface_mesh_simplification/test_edge_collapse_Polyhedron_3.cpp b/Surface_mesh_simplification/test/Surface_mesh_simplification/test_edge_collapse_Polyhedron_3.cpp index ca29638a015..7c641c5eb69 100644 --- a/Surface_mesh_simplification/test/Surface_mesh_simplification/test_edge_collapse_Polyhedron_3.cpp +++ b/Surface_mesh_simplification/test/Surface_mesh_simplification/test_edge_collapse_Polyhedron_3.cpp @@ -306,12 +306,12 @@ public : virtual void OnCollected(const Profile& aProfile, const boost::optional& aCost) const { - TEST_TRACE(str (format("Collecting %1% : cost=%2%") % edge2str(aProfile.v0_v1()) % optfloat2str(aCost))); + TEST_TRACE(str (boost::format("Collecting %1% : cost=%2%") % edge2str(aProfile.v0_v1()) % optfloat2str(aCost))); } virtual void OnCollapsing(const Profile& aProfile, const boost::optional& aP) const { - TEST_TRACE(str (format("S %1% - Collapsing %2% : placement=%3%") % mStep % edge2str(aProfile.v0_v1()) % optpoint2str(aP))); + TEST_TRACE(str (boost::format("S %1% - Collapsing %2% : placement=%3%") % mStep % edge2str(aProfile.v0_v1()) % optpoint2str(aP))); //mBefore = create_edge_link(aProfile); } @@ -323,10 +323,10 @@ public : { SurfaceSP lAfter = create_vertex_link(aProfile, aV); - write(mBefore, str(format("%1%.step-%2%-before.off") % mTestCase % mStep)); - write(lAfter , str(format("%1%.step-%2%-after.off") % mTestCase % mStep)); + write(mBefore, str(boost::format("%1%.step-%2%-before.off") % mTestCase % mStep)); + write(lAfter , str(boost::format("%1%.step-%2%-after.off") % mTestCase % mStep)); - REPORT_ERROR(str(format("Resulting surface self-intersects after step %1% (%2% edges left)") % mStep % (aProfile.surface().size_of_halfedges() / 2))); + REPORT_ERROR(str(boost::format("Resulting surface self-intersects after step %1% (%2% edges left)") % mStep % (aProfile.surface().size_of_halfedges() / 2))); } ++mStep;