From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:33334) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZdyvC-00026V-6I for qemu-devel@nongnu.org; Mon, 21 Sep 2015 07:06:14 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Zdyv6-0003kK-Fp for qemu-devel@nongnu.org; Mon, 21 Sep 2015 07:06:10 -0400 Received: from mx1.redhat.com ([209.132.183.28]:53349) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Zdyv6-0003gX-Ap for qemu-devel@nongnu.org; Mon, 21 Sep 2015 07:06:04 -0400 References: <1442674777-14793-1-git-send-email-comethalley61@gmail.com> From: Michal Privoznik Message-ID: <55FFE497.3010503@redhat.com> Date: Mon, 21 Sep 2015 13:05:59 +0200 MIME-Version: 1.0 In-Reply-To: <1442674777-14793-1-git-send-email-comethalley61@gmail.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [FIRST-PATCH] updated the negation STREQ to STRNEQ List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Lasya , qemu-devel@nongnu.org [You've got the wrong list. This is a libvirt patch and as such it should have been sent to the libvir-list@redhat.com] 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