|
From: | Stefan Weil |
Subject: | Re: [Qemu-devel] [PATCH 1/4] console: a few cleanups |
Date: | Wed, 18 Jan 2012 06:50:28 +0100 |
User-agent: | Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.2.24) Gecko/20111108 Thunderbird/3.1.16 |
Am 18.01.2012 01:49, schrieb Anthony Liguori:
There are missing braces in several modified blocks, for example: if (s->vga.ds->cursor_define) - s->vga.ds->cursor_define(qc); + s->vga.ds->cursor_define(s->vga.ds, qc); cursor_put(qc); checkpatch.pl does not detect this because the change was in the one line block, not in the conditional statement: $ scripts/checkpatch.pl /home/stefan/Downloads/1-4-console-a-few-cleanups.patch WARNING: line over 80 characters #85: FILE: hw/vmware_vga.c:908: + s->vga.ds->mouse_set(s->vga.ds, s->cursor.x, s->cursor.y, s->cursor.on); total: 0 errors, 1 warnings, 89 lines checked It's quite common in other patches to fix the braces when blocks without braces are changed. Regards, Stefan Weil |
[Prev in Thread] | Current Thread | [Next in Thread] |