Work at SourceForge, help us to make it a better place! We have an immediate need for a Support Technician in our San Francisco or Denver office.
I'm not sure this patch makes sense to me. Can you explain why vsu and vsv are being passed as parameters to brep_newton_joint_iterate()? It looks to me like they should have remained local to the new joint function. Also, that function should be marked static as it is not public API.
The patch could not be applied due to the aforementioned reasons, but you did identify an unnecessary code duplication that has been eliminated in r59356.