On Tue, Apr 30, 2013 at 11:27 AM, phoenix <284281...@qq.com> wrote:
>
> I have updated my proposal to reflect this. But I still have several
> questions:
>
> The current version of openNURBS in BRL-CAD still has the ON_X_EVENT (and
> ON_SSX_EVENT) code. So should I move this existed code to libbrep and
> remove the opennurbs_x.h and opennurbs_x.cpp files (since the newest
> openNURBS doesn't have them)?
>
Yes, that's probably the way to go. That will probably require some header
changes and (if I recall correctly) changes to opennurbs_basic.cpp - there
may also be functions that are worth moving over that do some basic work
using those types.
> For the naming - should I remove the ON_ prefix in the new code in libbrep
> and use something else (e.g. BREP_*)? They are no longer openNURBS members
> and live in libbrep.
>
Either way - I've generally been leaving the ON_ prefix for the time
being. It should be fairly simple to change later, since it will just be a
re-name, but if you prefer to distance the names from openNURBS that's fine
too.
> And the test_ssi program - I currently use pipe primitives to represent
> the intersection curves so that they can be written to a file and displayed
> in MGED, since they are originally polyline curves and pipes can show us
> their shapes. But using pipes seems a little awkward - are their any better
> options for this?
>
The most "correct" way would probably be to use the sketch primitive, but
that will have the drawback of not showing up in raytraced images as 3D
geometry. For BRL-CAD in its current form the pipe is actually a decent
solution, as long as it's not introducing too much complexity into your
code.
Cheers,
CY
------------------------------------------------------------------------------
Introducing AppDynamics Lite, a free troubleshooting tool for Java/.NET
Get 100% visibility into your production application - at no cost.
Code-level diagnostics for performance bottlenecks with <2% overhead
Download for free and get started troubleshooting in minutes.
http://p.sf.net/sfu/appdyn_d2d_ap1
_______________________________________________
BRL-CAD Developer mailing list
brlcad-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/brlcad-devel
