On Wed, 26 Jul 2017, Graeme Geldenhuys via Lazarus wrote:

On 2017-07-26 14:54, Alexey via Lazarus wrote:
I think LCL has many CompareNNNN funcs, but misses func for TPoint. Here
it's comparing x/y screen coordinates

Why not extend TPoint with an IsEquals() method, so you can do something like


var
  p1: TPoint;
  p2: TPoint;
begin
  // ... populate p1 & p2
  if p1.IsEquals(p2) then
    // do something.


ComparePoints() returning a Integer is not a very friendly return type either, whereas Boolean makes more sense.

First start by defining what ComparePoints is supposed to measure or compare.

If "equals" is needed, why not simply use p1=p2 ?

Michael.
--
_______________________________________________
Lazarus mailing list
Lazarus@lists.lazarus-ide.org
https://lists.lazarus-ide.org/listinfo/lazarus

Reply via email to