On Fri, Apr 26, 2013 at 8:45 AM, phoenix <284281...@qq.com> wrote:
>
> And Sean mentioned that there are other types of intersections regarding
> to points, including P/P, P/C and P/S. So I will also add these versions of
> ON_X_EVENT to libbrep (e.g. ON_PPX_EVENT, etc. or just ON_PX_EVENT for all
> three of them).
>
Sounds good - I'd go with the ON_PX_EVENT, if that matches the polymorphism
of the ON_X_EVENT and ON_SSX_EVENT types. (Should ON_X_EVENT be
ON_CX_EVENT for consistency?)
> By saying "moving it over to libbrep", do you mean moving all ON_X_EVENT
> (and ON_SSX_EVENT) features to libbrep, or just put the extensions in
> libbrep (like opennurbs_ext.cpp)?
>
I expect we'll need to move the whole thing over - if you look at the files
in http://brlcad.org/~starseeker/opennurbs/ you'll see that after the
opennurbs_20110202 release they pretty much totally removed the ON_X_EVENT
code - opennurbs_20111122 has an empty opennurbs_x.h file, and it's gone
completely by opennurbs_20120914. If we want to keep tracking openNURBS
releases as they come out, that means either making sure the intersection
system is fully in libbrep or retrofitting it to each new openNURBS release.
There are a few other functions that they've yanked that we need to keep as
well - if it ends up being easier for you to work with the newest openNURBS
to avoid conflicts we can go ahead and try to get those yanked functions
into libbrep so we can use an (almost) vanilla openNURBS.
The plus side of that will be that if you want to extend the *X_EVENT
system in any way, we don't have to worry about conflicts with openNURBS -
you can make it what it needs to be for this application.
Cheers,
CY
------------------------------------------------------------------------------
Try New Relic Now & We'll Send You this Cool Shirt
New Relic is the only SaaS-based application performance monitoring service
that delivers powerful full stack analytics. Optimize and monitor your
browser, app, & servers with just a few lines of code. Try New Relic
and get this awesome Nerd Life shirt! http://p.sf.net/sfu/newrelic_d2d_apr
_______________________________________________
BRL-CAD Developer mailing list
brlcad-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/brlcad-devel
