diff options
author | Lev Dragunov <l.dragunov@corp.mail.ru> | 2015-02-10 13:58:13 +0300 |
---|---|---|
committer | Alex Zolotarev <alex@maps.me> | 2015-09-23 02:37:25 +0300 |
commit | 04b7fc51e669a6244ce72d90a0231f353b70a993 (patch) | |
tree | a1dc336338f54567b5317dc925235f8398490f3c /geometry | |
parent | a59fbb74e5e00c0bcf5e9b8b3fa0748037726c66 (diff) |
Code review fixes
Diffstat (limited to 'geometry')
-rw-r--r-- | geometry/geometry_tests/region_test.cpp | 4 | ||||
-rw-r--r-- | geometry/region2d.hpp | 6 |
2 files changed, 5 insertions, 5 deletions
diff --git a/geometry/geometry_tests/region_test.cpp b/geometry/geometry_tests/region_test.cpp index d35570a92f..944dc60903 100644 --- a/geometry/geometry_tests/region_test.cpp +++ b/geometry/geometry_tests/region_test.cpp @@ -206,8 +206,8 @@ UNIT_TEST(Region_point_at_border_test) P p4(5.0, 1.0); P p5(5.0, 1.01); - TEST(!region.atBorder(p1, 0.01), ("Point lies outside border")); - TEST(!region.atBorder(p2, 0.01), ("Point lies inside but not at border")); + TEST(!region.atBorder(p1, 0.01), ("Point lies outside the border")); + TEST(!region.atBorder(p2, 0.01), ("Point lies strictly inside the border")); TEST(region.atBorder(p3, 0.01), ("Point has same point with border")); TEST(region.atBorder(p4, 0.01), ("Point lies at border")); TEST(region.atBorder(p5, 0.01), ("Point lies at delta interval near border")); diff --git a/geometry/region2d.hpp b/geometry/region2d.hpp index 94931d7c4d..5a41222f66 100644 --- a/geometry/region2d.hpp +++ b/geometry/region2d.hpp @@ -193,7 +193,7 @@ namespace m2 return Contains(pt, typename TraitsT::EqualType()); } - /// Slow point at border realisation + /// Slow check that point lies at the border. template <class EqualF> bool atBorder(PointT const & pt, double const delta, EqualF equalF) const { @@ -213,7 +213,7 @@ namespace m2 { PointT const curr = m_points[i]; - // Borders often has same points with ways + // Borders often have same points with ways if (equalF.EqualPoints(m_points[i], pt)) return true; @@ -224,7 +224,7 @@ namespace m2 prev = curr; } - return false; // outside + return false; // Point lies outside the border. } bool atBorder(PointT const & pt, double const delta) const |