]> git.proxmox.com Git - qemu.git/commit
checkpatch: don't error out on },{ lines
authorAlexander Graf <agraf@suse.de>
Wed, 29 Jun 2011 06:04:27 +0000 (08:04 +0200)
committerAlexander Graf <agraf@suse.de>
Sat, 16 Jul 2011 23:54:25 +0000 (01:54 +0200)
commit9fbe4784449f6248224ac95e0aa9e56a0bfdd155
tree1363da68e5442d3e3b8c993e74e877aeef9f7df1
parent7cef3f4fdbe813ac7d495e15a81531f1181b19f9
checkpatch: don't error out on },{ lines

When having code like this:

    static PCIDeviceInfo piix_ide_info[] = {
        {
            .qdev.name    = "piix3-ide",
            .qdev.size    = sizeof(PCIIDEState),
            .qdev.no_user = 1,
            .no_hotplug   = 1,
            .init         = pci_piix_ide_initfn,
            .vendor_id    = PCI_VENDOR_ID_INTEL,
            .device_id    = PCI_DEVICE_ID_INTEL_82371SB_1,
            .class_id     = PCI_CLASS_STORAGE_IDE,
        },{
            .qdev.name    = "piix4-ide",
            .qdev.size    = sizeof(PCIIDEState),
            .qdev.no_user = 1,
            .no_hotplug   = 1,
            .init         = pci_piix_ide_initfn,
            .vendor_id    = PCI_VENDOR_ID_INTEL,
            .device_id    = PCI_DEVICE_ID_INTEL_82371AB,
            .class_id     = PCI_CLASS_STORAGE_IDE,
        },{
            /* end of list */
        }
    };

checkpatch currently errors out, claiming that spaces need to follow
commas. However, this particular style of defining structs is pretty
common in qemu code and very readable. So let's declare it as supported
for the above case.

Reported-by: Kevin Wolf <kwolf@redhat.com>
Signed-off-by: Alexander Graf <agraf@suse.de>
scripts/checkpatch.pl