From 6180687bbd0ef076c6e8fefe66b0f68003d3a163 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Tomasz=20W=C5=82ostowski?= <tomasz.wlostowski@cern.ch>
Date: Tue, 25 Jun 2019 01:03:54 +0200
Subject: [PATCH] TestSegmentHit: add corner case for null segment (start ==
 end)

Fixes: lp:1833059
* https://bugs.launchpad.net/kicad/+bug/1833059
---
 common/geometry/trigo.cpp | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/common/geometry/trigo.cpp b/common/geometry/trigo.cpp
index 5415fc58fe..ca0f3b9c7d 100644
--- a/common/geometry/trigo.cpp
+++ b/common/geometry/trigo.cpp
@@ -151,6 +151,15 @@ bool TestSegmentHit( const wxPoint &aRefPoint, wxPoint aStart, wxPoint aEnd, int
     if( aStart.y == aEnd.y && aRefPoint.x > xmin && aRefPoint.x < xmax )
         return std::abs( delta.y ) <= aDist;
 
+    // Special case for a segment with start == end (equal to a circle)
+    if ( aStart == aEnd )
+    {
+        long double length_square = (long double) delta.x * delta.x + (long double) delta.y * delta.y;
+        long double dist_square = (long double) aDist * aDist;
+
+        return ( length_square <= dist_square );
+    }
+
     wxPoint len = aEnd - aStart;
     // Precision note here:
     // These are 32-bit integers, so squaring requires 64 bits to represent