From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:56829) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Xlbhf-0004uz-8R for qemu-devel@nongnu.org; Tue, 04 Nov 2014 05:51:17 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1XlbhZ-0005fb-1U for qemu-devel@nongnu.org; Tue, 04 Nov 2014 05:51:11 -0500 From: zhanghailiang Date: Tue, 4 Nov 2014 18:50:18 +0800 Message-ID: <1415098223-32404-1-git-send-email-zhang.zhanghailiang@huawei.com> MIME-Version: 1.0 Content-Type: text/plain Subject: [Qemu-devel] [PATCH v3 0/5] Trivial patch about qemu-char List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-trivial@nongnu.org Cc: zhanghailiang , armbru@redhat.com, mjt@tls.msk.ru, qemu-devel@nongnu.org, peter.huangpeng@huawei.com, kraxel@redhat.com Patch 1 and 2 fix check about parameter in chr_parse_* functions. Patch 3 fix wrong english state in error message in windows part of the code. The last two patches convert some open functions to use Error API. In patch 4, i have tried to convert some codes which will be used in windows hypervisor. I didn't test them. But this is not too complex. Markus has suggested add a separate patch for print_allowed_subtypes to use error_prinf, but i think it's also in the context of "use Error API", so i fix it togther in patch 5. v3: - discard wrong modify about qemu_chr_open_pty (Michael Tokarev) - use !str[0] in place of strlen(str) == 0 which makes more sense for parameter check (Michael Tokarev) - Merge v2's patch 1 and 2 into a single patch (Michael Tokarev) - Add new patch 3 to fix wrong english state in error message - Use error_prinf instead of fprint for print_allowed_subtypes (Markus Armbruster) v2: - don't use error_setg when followed by exit(), it does not report an error (Eric Blake) - check the parameter in qemu_chr_parse_* functions and remove the check in qemu_chr_open_* functions. (Michael Tokarev) Thanks very much for their reviews and suggestions;) zhanghailiang (5): qemu-char: fix parameter check in some qemu_chr_parse_* functions spice-qemu-char: fix parameter checks in qemu_chr_parse_* functions qemu-char: fix incorrect state in error message qemu-char: convert some open functions to use Error API spice-qemu-char: convert some functions to use Error API include/ui/qemu-spice.h | 2 +- qemu-char.c | 93 +++++++++++++++++++++++---------------------- spice-qemu-char.c | 26 ++++--------- stubs/qemu-chr-open-spice.c | 2 +- 4 files changed, 57 insertions(+), 66 deletions(-) -- 1.7.12.4