qemu-devel
[Top][All Lists]
Advanced

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

Re: [Qemu-devel] [PATCH v3 0/3] arch_init: Move soundhw code to hw/audio


From: Eduardo Habkost
Subject: Re: [Qemu-devel] [PATCH v3 0/3] arch_init: Move soundhw code to hw/audio/soundhw.c
Date: Tue, 9 May 2017 00:13:06 -0300
User-agent: Mutt/1.8.0 (2017-02-23)

On Mon, May 08, 2017 at 03:54:18PM -0700, address@hidden wrote:
[...]
> === OUTPUT BEGIN ===
> Checking PATCH 1/3: audio: Move arch_init audio code to hw/audio/soundhw.c...
> ERROR: suspect code indent for conditional statements (8, 13)
> #240: FILE: hw/audio/soundhw.c:76:
> +        if (soundhw_count) {
> +             printf("Valid sound card names (comma separated):\n");
> 
> ERROR: suspect code indent for conditional statements (13, 17)
> #242: FILE: hw/audio/soundhw.c:78:
> +             for (c = soundhw; c->name; ++c) {
> +                 printf ("%-11s %s\n", c->name, c->descr);
> 
> ERROR: space prohibited between function name and open parenthesis '('
> #243: FILE: hw/audio/soundhw.c:79:
> +                 printf ("%-11s %s\n", c->name, c->descr);
> 
> ERROR: else should follow close brace '}'
> #252: FILE: hw/audio/soundhw.c:88:
> +    }
> +    else {
> 
> ERROR: else should follow close brace '}'
> #281: FILE: hw/audio/soundhw.c:117:
> +                }
> +                else {
> 
> WARNING: line over 80 characters
> #299: FILE: hw/audio/soundhw.c:135:
> +    ISABus *isa_bus = (ISABus *) object_resolve_path_type("", TYPE_ISA_BUS, 
> NULL);
> 
> WARNING: line over 80 characters
> #300: FILE: hw/audio/soundhw.c:136:
> +    PCIBus *pci_bus = (PCIBus *) object_resolve_path_type("", TYPE_PCI_BUS, 
> NULL);
> 
> total: 5 errors, 2 warnings, 320 lines checked
> 
> Your patch has style problems, please review.  If any of these errors
> are false positives report them to the maintainer, see
> CHECKPATCH in MAINTAINERS.

The style problems are not in new code, but in code being moved
to another file. If anybody wants to volunteer to fix those
issues, they should be addressed in follow-up patches.

-- 
Eduardo



reply via email to

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