Hi,

Sean has left the following comment at Implement a primitive volume function ... for gridded volumes (VOL) http://www.google-melange.com/gci/task/view/google/gci2012/7998215:


using the right tool


Using the right tool can make all the diference. The patch formatting looks correct now.

Looking at the code in the patch, have you tried compiling this? I see a type error on the volume= line that I'd imagine the compiler would have reported. Also, the function should be well-behaved if the caller passes a NULL pointer.


Greetings,
The Google Open Source Programs Team


---
You are receiving this message because you are subscribed to Implement a primitive volume function ... for gridded volumes (VOL). To stop receiving these messages, go to: http://www.google-melange.com/gci/task/view/google/gci2012/7998215.

------------------------------------------------------------------------------
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