Re: [Qemu-devel] [PATCH 1/4 v5] vmware_vga: Coding style cleanup

[BALATON Zoltan <balaton@eik.bme.hu>]

On Sat, 3 Nov 2012, Blue Swirl wrote:
> But this and the others introduce new problems:
> WARNING: suspect code indent for conditional statements (4, 9)
> #301: FILE: hw/vmware_vga.c:320:
> +    for (line = h; line > 0; line--, src += bypl, dst += bypl) {
>           memcpy(dst, src, width);
> 
> WARNING: suspect code indent for conditional statements (4, 9)
> #378: FILE: hw/vmware_vga.c:482:
> +    if (s->vga.ds->cursor_define) {
>           s->vga.ds->cursor_define(qc);
> 
> WARNING: suspect code indent for conditional statements (4, 9)
> #393: FILE: hw/vmware_vga.c:495:
> +    if (!s->config || !s->enable) {
>           return 0;
> 
> WARNING: suspect code indent for conditional statements (4, 9)
> #398: FILE: hw/vmware_vga.c:499:
> +    if (num < 0) {
>           num += CMD(max) - CMD(min);
> 
> Please recheck the patches.

I did check patches before submitting. It must have been my mailer 
mangling whitespace. Resent it again, now it should be correct at last. 
Sorry for the inconvenience.

Regards,
BALATON Zoltan