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

Reply via email to