Hi,

Arjun Govindjee has left the following comment at Create a COMPREHENSIVE unit test for bn_dist_pt3_pt3() http://www.google-melange.com/gci/task/view/google/gci2012/7973230:


It seems OK?


First of all, when I sscanf("%lf,%lf,%lf",p,p+1,p+2), p is a double[3], so that works correctly.

Also with regard to VMOVE(), I loop through p and q, which are both double[3], and cast each element to fastf_t and place them in a and b which are both fastf_t[3]. I cannot use VMOVE() because I need to perform the cast from double to fastf_t while moving the elements.

Also I do not believe it is appropriate to use point_t instead of fastf_t[3], because the function bn_dist_pt3_pt3() takes 2 fastf_t pointers as arguments, not 2 point_t arguments.

I think everything is actually OK.


Greetings,
The Google Open Source Programs Team


---
You are receiving this message because you are subscribed to Create a COMPREHENSIVE unit test for bn_dist_pt3_pt3(). To stop receiving these messages, go to: http://www.google-melange.com/gci/task/view/google/gci2012/7973230.

------------------------------------------------------------------------------
Master Visual Studio, SharePoint, SQL, ASP.NET, C# 2012, HTML5, CSS,
MVC, Windows 8 Apps, JavaScript and much more. Keep your skills current
with LearnDevNow - 3,200 step-by-step video tutorials by Microsoft
MVPs and experts. SALE $99.99 this month only -- learn more at:
http://p.sf.net/sfu/learnmore_122912
_______________________________________________
BRL-CAD Tracker mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/brlcad-tracker

Reply via email to