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.

--
Libvir-list mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/libvir-list

Reply via email to