[libvirt] [PATCH] Use gcc style for structure fields declaration

Jim Meyering jim at meyering.net
Fri Aug 29 06:02:13 UTC 2008


"Nguyen Anh Quynh" <aquynh at gmail.com> wrote:
> Hmm there are some typos (listDomain -> listDefinedDomain,
> numOfDomains -> numOfDefineDomains)
>
> Please take this patch instead.
>  static virDriver lxcDriver = {
> -    VIR_DRV_LXC, /* the number virDrvNo */
> -    "LXC", /* the name of the driver */
> -    LIBVIR_VERSION_NUMBER, /* the version of the backend */
> -    lxcProbe, /* probe */
...
> +    .no = VIR_DRV_LXC, /* the number virDrvNo */
> +    .name = "LXC", /* the name of the driver */
> +    .ver = LIBVIR_VERSION_NUMBER, /* the version of the backend */
> +    .probe = lxcProbe,
...

Thank you.  That looks like a fine improvement.

Have you already compared the pre- and post-patch .o files?
This patch appears to be one that would evoke no change in any
recompiled .o file.
If so, and if you can report that all four pre/post .o-file pairs are
identical, we can be sure this improvement does not hide a subtle bug.




More information about the libvir-list mailing list