2015-07-18 10:25 GMT+02:00 Peter Uhnák <i.uh...@gmail.com>:

> The discussion was that Rectangle is used by Morphic which uses integer
> coordinates so rounding is required. I think the idea was to split the two
> classes?
>
> Peter
>
> On Sat, Jul 18, 2015 at 9:50 AM, Stephan Eggermont <step...@stack.nl>
> wrote:
>
>> On 17/07/15 18:16, Jan Blizničenko wrote:
>>
>>> Shouldn't the rounded method be separated as for
>>> example roundedCenter or something like that to eliminate the confusion?
>>>
>>
I would like to keep Rectangles current implementation, because the
Rectangle class
is used for Screen-Rectangles and the screen only knows about integer/pixel
positions.

But on the other hand, you are right, we already use and allow rectangles
with non-integer corners.
And changing that (resp. enforcing integer coordinates) is maybe much more
difficult.
And at latest when we use Athens for rendering the full world, Rectangles
with integer rounded position for every shape, does'nt make much sense.


If you do some time comparsions, can you use

center
    "Answer the point at the center of the receiver."

    ^self topLeft + self bottomRight /  2.0

(instead of "/ 2" ) (fraction vs. float)





>
>> I think I've seen discussions about this before and vaguely
>> remember the conclusion that there was too much current
>> code depending on it for this to be an easy change at the
>> moment
>>
>> Stephan
>>
>>
>>
>>
>

Reply via email to