search

wxWidgets / wxWidgets

Cross-Platform C++ GUI Library
https://www.wxwidgets.org/
5.75k stars 1.69k forks source link

adding wx3DPoint class #6363

Closed wxtrac closed 2 years ago

wxtrac commented 20 years ago

Issue migrated from trac ticket # 6363

priority: normal

2003-12-01 00:19:16: mgilabert created the issue

Adding a class which handles 3D points (like wxPoint or wxRealPoint), which have members: x, y, and z.

Members are in data type double.

It has the same behaviour as wxPoint and wxRealPoint (operator ==, !=, +, -, -=, +=)

Very usefull to handle 3D Graphics easilly.

Have updated latex documentation (adding a new file for doucmenting this class: 3dpoint.tex), updating point.tex and realpnt.tex to have a link to this class.

Hope it will be in the Head and you will like this patch :)

Regards.

wxtrac commented 20 years ago

2003-12-05 05:46:52: ryannpcs commented

Hi Mickael,

There are some problems with this - 1) operators return a normal wxPoint? 2) Nothing in the library actually uses this... 3) You might want to derive from wxRealPoint...

-ryannpcs

wxtrac commented 20 years ago

2003-12-15 22:01:11: mgilabert commented

Hi.

sorry for delay. I was quite busy theses days. But now i have some times to repost my patch:

I have deleted the previous patch and resent a new one with correction.

  1. Oops right,, operators returns now wx3DPoint
  2. Sure, but this could helps people using 3D Libraries or 3D point
  3. Inherits from wxRealPoint is not a good idea in term of performance (operator would be virtual in wxRealpoint), and derive just to use two memebers and not really the function is not so efficient as it is in design term.

i could understand your point of view regarding the fact that wxWindows frameswork doesn't use wx3DPoint anywhere in the libraries, but one thing is sure, is when a usr-devl wants to just have a primitive like 3D Point, it would be cool to have it available in the wxWindows framework. That's why I have suggested it.

Thank you again for your feedback.

Mickael Gilabert

wxtrac commented 20 years ago

2003-12-15 22:01:12: mgilabert uploaded file adding_wx3DPoint_class.patch.newversion (4.4 KiB)

new wx3DPoint file in diff mode with its full latek documentation

wxtrac commented 20 years ago

2003-12-21 01:26:32: ryannpcs commented

Hi,

Since this is a new class, and I'm not sure about it's inclusion, I'll have to swing this by the other devs first.

Thanks!

wxtrac commented 20 years ago

2003-12-22 02:57:01: ryannpcs commented

Hi,

I ran this by the other devs, and unfortunently they don't agree about it's inclusion.

See the wx-dev list archive at lists.wxwindows.org for details.

Thanks, Ryan

wxtrac commented 20 years ago

2003-12-22 11:41:56: mgilabert commented

Ok. I understand wx dev's point of view.

Thanks anyway for replying to my submit.

Maybe next one will be taken ;)

Regards.

Mickael