From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43)
	id 1KGw4Q-0002un-MH
	for qemu-devel@nongnu.org; Thu, 10 Jul 2008 09:20:26 -0400
Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43)
	id 1KGw4O-0002rH-Ph
	for qemu-devel@nongnu.org; Thu, 10 Jul 2008 09:20:25 -0400
Received: from [199.232.76.173] (port=34817 helo=monty-python.gnu.org)
	by lists.gnu.org with esmtp (Exim 4.43) id 1KGw4O-0002r0-Bv
	for qemu-devel@nongnu.org; Thu, 10 Jul 2008 09:20:24 -0400
Received: from il.qumranet.com ([212.179.150.194]:58422)
	by monty-python.gnu.org with esmtp (Exim 4.60)
	(envelope-from <avi@qumranet.com>) id 1KGw4N-0004YV-V3
	for qemu-devel@nongnu.org; Thu, 10 Jul 2008 09:20:24 -0400
Message-ID: <48760C96.1010405@qumranet.com>
Date: Thu, 10 Jul 2008 16:20:22 +0300
From: Avi Kivity <avi@qumranet.com>
MIME-Version: 1.0
References: <1215695074-8939-1-git-send-email-avi@qumranet.com>
In-Reply-To: <1215695074-8939-1-git-send-email-avi@qumranet.com>
Content-Type: multipart/mixed; boundary="------------040909080503040904050603"
Subject: [Qemu-devel] Re: [PATCH 0/2] Fix the monitor 'screendump' command
Reply-To: qemu-devel@nongnu.org
List-Id: qemu-devel.nongnu.org
List-Unsubscribe: <http://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.gnu.org/pipermail/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <http://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: qemu-devel@nongnu.org

This is a multi-part message in MIME format.
--------------040909080503040904050603
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit

Avi Kivity wrote:
> The screendump command was recently broken in qemu-svn.  Since many
> test suites depend on the command, it's important to get it fixed.
>
> The following two patches do just that.
>   

Another patch is needed (attached).


-- 
error compiling committee.c: too many arguments to function


--------------040909080503040904050603
Content-Type: text/plain;
 name="force-resize.patch"
Content-Transfer-Encoding: 7bit
Content-Disposition: inline;
 filename="force-resize.patch"

diff --git a/console.c b/console.c
index 061135f..055a3f1 100644
--- a/console.c
+++ b/console.c
@@ -1334,7 +1334,8 @@ CharDriverState *text_console_init(DisplayState *ds, const char *p)
 
 void qemu_console_resize(QEMUConsole *console, int width, int height)
 {
-    if (console->g_width != width || console->g_height != height) {
+    if (console->g_width != width || console->g_height != height
+        || !console->ds->data) {
         console->g_width = width;
         console->g_height = height;
         if (active_console == console) {

--------------040909080503040904050603--