Hi Rich,
Good stuff, looks fine to commit to me.
Just a nitpick:
On Fri, 2007-03-02 at 15:00 +0000, Richard W.M. Jones wrote:
- activeNames = vshMalloc(ctl, sizeof(int) * maxactive);
+ activeNames = vshMalloc(ctl, sizeof (char *) * maxactive);
^
- qsort(&activeNames[0], maxactive, sizeof(int),
namesorter);
+ qsort(activeNames, maxactive, sizeof (char *), namesorter);
^
Extra whitespace here ...
Cheers,
Mark.