On 14 February 2013 00:01, Nicolas Cellier
<[email protected]> wrote:
> So here is what I did:
.....

>
> My feeling is that you are not thru with degenerated rectangles :)
> You'd better enforce the invariant in
> http://code.google.com/p/pharo/issues/detail?id=2601
> rathert than remove it as requested in
> http://code.google.com/p/pharo/issues/detail?id=2441

After replacement of intersect: with intersect:ifNone:
there is no need to enforcing invariants in #intersect: anymore, because
users can decide by themselves what to do if two rects are not intersecting..

the behavior of #intersect: for non-intersecting rects is totally
absurd and changing it to answer empty rect
(like with all zeroes) is not a good solution either, comparing to
providing a closure to handle it.

>
> Nicolas
>


-- 
Best regards,
Igor Stasenko.

Reply via email to