Le 10/10/2017 à 20:46, Seth Hillbrand a écrit :
> ​Oh, that is much better.  Thanks for the suggestion.
> 
> Attached is the updated patch using EDA_RECT calls.
> 
> -S​

Hi Seth,
That was really fast!

I committed your patch.

Thanks.

> 
> On Tue, Oct 10, 2017 at 11:16 AM, jp charras <[email protected] 
> <mailto:[email protected]>>
> wrote:
> 
>     Le 10/10/2017 à 18:41, Seth Hillbrand a écrit :
>     > Thank you for testing, JP.  I've corrected the issue.  I had missed the 
> inverted Y-axis in lib
>     > editor.  The attached patch functions correctly on all circles, in all 
> quadrants.
>     >
> 
>     Thanks.
>     It works fine now.
> 
>     However, in order to simplify this code (and avoid duplicate code) could 
> you consider using
>     bool EDA_RECT::Intersects( const EDA_RECT& aRect ) const;
>     to calculate if two rectangles intersect, both for text and circle.
> 
>     Again, thanks.
> 
> 
>     --
>     Jean-Pierre CHARRAS
> 
> 


-- 
Jean-Pierre CHARRAS

_______________________________________________
Mailing list: https://launchpad.net/~kicad-developers
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~kicad-developers
More help   : https://help.launchpad.net/ListHelp

Reply via email to