qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [FIRST-PATCH] updated the negation STREQ to STRNEQ


From: Michal Privoznik
Subject: Re: [Qemu-devel] [FIRST-PATCH] updated the negation STREQ to STRNEQ
Date: Mon, 21 Sep 2015 13:05:59 +0200
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.2.0

[You've got the wrong list. This is a libvirt patch and as such it
should have been sent to the address@hidden

On 19.09.2015 16:59, Lasya wrote:
> Hi,
> This is my first patch in QEMU. I have solved a bite-sized bug. I have 
> updated the !STREQ with STRNEQ in the following files. Thanks.
> 
> ---
>  src/bhyve/bhyve_driver.c              |  2 +-
>  src/conf/network_conf.c               |  4 ++--
>  src/conf/nwfilter_conf.c              |  2 +-
>  src/conf/nwfilter_params.c            |  2 +-
>  src/conf/storage_conf.c               |  2 +-
>  src/lxc/lxc_fuse.c                    |  4 ++--
>  src/openvz/openvz_driver.c            |  2 +-
>  src/qemu/qemu_command.c               |  6 +++---
>  src/qemu/qemu_domain.c                |  2 +-
>  src/qemu/qemu_hotplug.c               |  2 +-
>  src/security/security_manager.c       |  2 +-
>  src/security/security_selinux.c       | 12 ++++++------
>  src/storage/storage_backend_logical.c |  2 +-
>  src/util/virfile.c                    |  2 +-
>  src/util/virsystemd.c                 |  2 +-
>  src/vz/vz_driver.c                    |  2 +-
>  src/vz/vz_sdk.c                       |  2 +-
>  src/xen/xend_internal.c               |  2 +-
>  src/xenconfig/xen_sxpr.c              |  2 +-
>  tests/commandtest.c                   | 10 +++++-----
>  tests/securityselinuxlabeltest.c      |  2 +-
>  tests/virauthconfigtest.c             |  2 +-
>  tests/virbitmaptest.c                 | 12 ++++++------
>  tests/vircgrouptest.c                 |  2 +-
>  tests/virkeyfiletest.c                |  8 ++++----
>  tests/virnetsockettest.c              |  2 +-
>  tests/virtypedparamtest.c             |  2 +-
>  tests/viruritest.c                    | 16 ++++++++--------
>  28 files changed, 56 insertions(+), 56 deletions(-)

Okay, you've got all the occurrences. However, the syntax-check rule is
missing. You may want to look at cfg.mk for some examples. syntax-check
is our make target that checks the code if it complies with our coding
standards. After this patch we want a rule that forbids !STREQ or !STRNEQ.

Otherwise looking very good. Looking forward to v2.

Michal



reply via email to

[Prev in Thread] Current Thread [Next in Thread]