Patches item #3610098, was opened at 2013-04-05 09:07 Message generated for change (Comment added) made by ncsaba You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=640804&aid=3610098&group_id=105292
Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: Raytrace >Group: Incomplete Status: Open Resolution: None Priority: 5 >Private: Yes Submitted By: Csaba Nagy (ncsaba) Assigned to: Nobody/Anonymous (nobody) Summary: Refactor pipe element calculation in pipe.c Initial Comment: The calculation of the linear/bent elements needed to construct a pipe based on it's control points is repeated a few times in the code. This patch is extracting it in a separate function, and replaces 2 of the occurrences with call to this new function. There is one more occurrence in the rt_pipe_tess function, but that code has either already drifted or it really does something different, so I didn't touch it - needs review from somebody who knows that code. ---------------------------------------------------------------------- >Comment By: Csaba Nagy (ncsaba) Date: 2013-04-05 10:54 Message: This solution is not satisfactory, I will work on a new patch. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=640804&aid=3610098&group_id=105292 ------------------------------------------------------------------------------ Minimize network downtime and maximize team effectiveness. Reduce network management and security costs.Learn how to hire the most talented Cisco Certified professionals. Visit the Employer Resources Portal http://www.cisco.com/web/learning/employer_resources/index.html _______________________________________________ BRL-CAD Tracker mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/brlcad-tracker
