You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by "Matt Juntunen (Jira)" <ji...@apache.org> on 2020/01/01 22:37:00 UTC

[jira] [Commented] (GEOMETRY-82) Make Region extend Predicate

    [ https://issues.apache.org/jira/browse/GEOMETRY-82?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17006506#comment-17006506 ] 

Matt Juntunen commented on GEOMETRY-82:
---------------------------------------

Yeah, I think you're right. If a region needs to be used as a predicate a simple method reference to {{contains}} would suffice. There's no need to extend {{Predicate}} here.

> Make Region extend Predicate
> ----------------------------
>
>                 Key: GEOMETRY-82
>                 URL: https://issues.apache.org/jira/browse/GEOMETRY-82
>             Project: Apache Commons Geometry
>          Issue Type: Improvement
>            Reporter: Matt Juntunen
>            Priority: Major
>
> The {{Region}} interface has a {{contains(Point)}} method that returns a boolean indicating whether or not a point lies inside or on the boundary of the region. I think this makes it a good candidate for extending the {{Predicate}} interface. There are two options:
> 1. Rename {{contains}} to {{test}}
> 2. Add {{test}} with a default implementation that simply calls {{contains}}



--
This message was sent by Atlassian Jira
(v8.3.4#803005)