Makes sense, and the edits/documentation cleanups look consistent throughout. There a couple of lingering comments that'll be misleading to anyone reading the bspline.c file. One final patch, attached, will kill those. Thanks, Rhys On Mon, Dec 8, 2008 at 3:11 PM, Brian Gough wrote: > I've added this with some minor changes (I've moved the bspline > workspace out of the derivative workspace, to avoid any confusion > about the ownership of the memory -- so the derivative functions take > the normal bspline workspace and a derivative workspace as arguments). > >