U++ framework: U++ Core » Rect_ operators overloading suggestion

Home » U++ Library support » U++ Core » Rect_ operators overloading suggestion

Show: Today's Messages :: Polls :: Message Navigator
E-mail to friend

Re: Rect_ operators overloading suggestion [message #13426 is a reply to message #13412]

Sat, 05 January 2008 11:33

mirek
Messages: 14291
Registered: November 2005

Ultimate Member

chickenk wrote on Fri, 04 January 2008 12:25

I would like to make a suggestion. I may have missed an important point so my suggestion would not make sense; don't hesitate to blame me for that. Smile

I can see these overloads:

Rect_& operator+=(Sz sz){ Offset(sz); return *this; }
Rect_& operator+=(Pt p) { Offset(p); return *this; }
[...]
Rect_& operator-=(Sz sz){ Offset(-sz); return *this; }
Rect_& operator-=(Pt p) { Offset(-p); return *this; }

I agree with the Pt-argumented functions but I think it would make more sense for Sz-argumented functions to modify the size of the Rectangle instead of translating it...

For example, the following code could be used (I've not tested it):

Rect_& operator+=(Sz sz){ SetSize(Size()+sz); return *this; }
[...]
Rect_& operator-=(Sz sz){ SetSize(Size()-sz); return *this; }

What do you think about it ?

regards,
Lionel

I think this classically ambiguos case.... I see good reasons for current overload as well for the proposed one...

Mirek

Report message to a moderator

[Message index]

		Rect_ operators overloading suggestion By: chickenk on Fri, 04 January 2008 18:25
		Re: Rect_ operators overloading suggestion By: mirek on Sat, 05 January 2008 11:33
		Re: Rect_ operators overloading suggestion By: chickenk on Sat, 05 January 2008 20:04
		Re: Rect_ operators overloading suggestion By: mirek on Sat, 05 January 2008 23:29
		Re: Rect_ operators overloading suggestion By: mdelfede on Sat, 05 January 2008 23:44
		Re: Rect_ operators overloading suggestion By: chickenk on Sun, 06 January 2008 18:57

Previous Topic:	A couple of issues with a test app
Next Topic:	Dynamic Variable Names

Goto Forum:

-=] Back to Top [=-

[ Syndicate this forum (XML) ] [

] [

]

Current Time: Sun Oct 11 08:47:28 GMT+2 2026

Total time taken to generate the page: 0.00826 seconds