[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/6] slirp: Remove default_mon usage
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH 1/6] slirp: Remove default_mon usage |
Date: |
Wed, 12 Mar 2014 09:13:18 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.2 (gnu/linux) |
Cole Robinson <address@hidden> writes:
> These errors don't seem user initiated, so forcibly printing to the
> monitor doesn't seem right. Just print to stderr.
>
> Drop lprint since it's now unused.
>
> Cc: Jan Kiszka <address@hidden>
> Signed-off-by: Cole Robinson <address@hidden>
> ---
> checkpatch flags some pre-existing tab issues, but I didn't retab. Should I?
>
> slirp/misc.c | 13 ++-----------
> slirp/slirp.c | 8 ++++----
> slirp/slirp.h | 2 --
> 3 files changed, 6 insertions(+), 17 deletions(-)
>
> diff --git a/slirp/misc.c b/slirp/misc.c
> index 6c1636f..662fb1d 100644
> --- a/slirp/misc.c
> +++ b/slirp/misc.c
> @@ -136,7 +136,7 @@ fork_exec(struct socket *so, const char *ex, int do_pty)
> if ((s = qemu_socket(AF_INET, SOCK_STREAM, 0)) < 0 ||
> bind(s, (struct sockaddr *)&addr, addrlen) < 0 ||
> listen(s, 1) < 0) {
> - lprint("Error: inet socket: %s\n", strerror(errno));
> + fprintf(stderr, "Error: inet socket: %s\n",
> strerror(errno));
> closesocket(s);
>
> return 0;
Why not error_report()?
[...]
[Qemu-devel] [PATCH 3/6] error: Privatize error_print_loc, Cole Robinson, 2014/03/11
[Qemu-devel] [PATCH 4/6] monitor: Remove unused monitor_print_filename, Cole Robinson, 2014/03/11