Re: [Pharo-users] Rectangle center rounding

[stepharo]

It would be nice to design a new one test first.

Stef

Le 19/7/15 18:41, Alexandre Bergel a écrit :

Maybe we can introduce a new rectangle class.

Alexandre

On Jul 19, 2015, at 6:33 PM, Nicolai Hess <nicolaih...@web.de> wrote:



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