[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v8 0/5] event: Add source information to SHUTDOW
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH v8 0/5] event: Add source information to SHUTDOWN |
Date: |
Mon, 15 May 2017 15:46:59 -0500 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.1.0 |
On 05/15/2017 03:37 PM, address@hidden wrote:
> Hi,
>
> This series seems to have some coding style problems. See output below for
> more information:
>
> Checking PATCH 5/5: shutdown: Expose bool cause in SHUTDOWN and RESET
> events...
> ERROR: open brace '{' following function declarations go on the next line
> #61: FILE: include/sysemu/sysemu.h:52:
> +static inline bool shutdown_caused_by_guest(ShutdownCause cause) {
>
Oh fine (I have a hook to run checkpatch locally on every commit; wonder
why it didn't gripe at me). I also missed a FIXME that should have been
TODO in 2/5. v9 coming up.
--
Eric Blake, Principal Software Engineer
Red Hat, Inc. +1-919-301-3266
Virtualization: qemu.org | libvirt.org
signature.asc
Description: OpenPGP digital signature
- [Qemu-devel] [PATCH v8 0/5] event: Add source information to SHUTDOWN, Eric Blake, 2017/05/15
- [Qemu-devel] [PATCH v8 2/5] shutdown: Prepare for use of an enum in reset/shutdown_request, Eric Blake, 2017/05/15
- [Qemu-devel] [PATCH v8 3/5] shutdown: Preserve shutdown cause through replay, Eric Blake, 2017/05/15
- [Qemu-devel] [PATCH v8 1/5] shutdown: Simplify shutdown_signal, Eric Blake, 2017/05/15
- [Qemu-devel] [PATCH v8 5/5] shutdown: Expose bool cause in SHUTDOWN and RESET events, Eric Blake, 2017/05/15
- [Qemu-devel] [PATCH v8 4/5] shutdown: Add source information to SHUTDOWN and RESET, Eric Blake, 2017/05/15
- Re: [Qemu-devel] [PATCH v8 0/5] event: Add source information to SHUTDOWN, no-reply, 2017/05/15
- Re: [Qemu-devel] [PATCH v8 0/5] event: Add source information to SHUTDOWN,
Eric Blake <=