Bug 3672 – GW: KWQRect -- CGRect and other small additions

• Component: New Bugs
• Status: RESOLVED FIXED

[Eric Seidel (no email)]

KWQRect -- CGRect and other small additions

including:
QRect (const QPoint &tl, const QPoint &br)
QRect::top()
QRect::left()
QRect::topLeft()
QRect::bottomRight()
QRect::contiains(QRect)
QRect::normalize()

I'm not 100% sure of the correctness of all these changes... only due to the possible off-by-one error 
associated with QRect.  QRect has this funny +1 width/height property, and I'm not sure that I'm 
correctly adding/removing that +1 when converting to/from CGRect, etc.  Hopefully someone who has 
worked with QRect a bit longer than I can comment more.

That said, the code is otherwise tested and in use locally.

[Eric Seidel (no email)]

Created attachment 2576 [details]
KWQRect -- CGRect and other small additions

[Darin Adler]

Comment on attachment 2576 [details]
KWQRect -- CGRect and other small additions

Looks good. A few comments:

The QRect contructor that takes two points is too long to be inline. Please
move it into the .mm file.

Why did you make left() and top() non-inline? They shouldn't be changed unless
there's a reason.

In QRect::normalize, you should use "-w" rather than "(w * -1)" and "-h" rather
than "(h * -1)".

No need for the extra blank line after operator CGRect. Better to keep the
file's format consistent, even in such a trivial detail.

[Eric Seidel (no email)]

Created attachment 2603 [details]
Formatting oversights now fixed.

[Darin Adler]

Comment on attachment 2603 [details]
Formatting oversights now fixed.

r=me