From: Stefan Weil <Stefan.Weil@weilnetz.de>
To: Anthony Liguori <aliguori@us.ibm.com>
Cc: QEMU Developers <qemu-devel@nongnu.org>
Subject: Re: [Qemu-devel] [PATCH] Add static to local machine declaration.
Date: Wed, 27 May 2009 21:17:55 +0200 [thread overview]
Message-ID: <4A1D91E3.6070502@weilnetz.de> (raw)
In-Reply-To: <4A1D50A0.9070207@us.ibm.com>
[-- Attachment #1: Type: text/plain, Size: 481 bytes --]
Anthony Liguori schrieb:
> Stefan Weil wrote:
>> Variable akitapda_machine is only used locally,
>> so the static attribute avoids a compiler warning.
>>
>> Signed-off-by: Stefan Weil <weil@mail.berlios.de>
>>
>
> This patch is whitespace broken as is two other patches I have in my
> queue for you. Could you try using something like git-send-email?
>
> Regards,
>
> Anthony Liguori
>
Sorry, I am still a git-learner...
The appended patch will work.
Regards,
Stefan Weil
[-- Attachment #2: 0001-Add-static-to-local-machine-declaration.patch --]
[-- Type: text/x-diff, Size: 859 bytes --]
>From 3c4e4df0b882fd553aa89616e5d3d08a7eca301c Mon Sep 17 00:00:00 2001
From: Stefan Weil <weil@mail.berlios.de>
Date: Sat, 23 May 2009 23:49:14 +0200
Subject: [PATCH] Add static to local machine declaration.
Variable akitapda_machine is only used locally,
so the static attribute avoids a compiler warning.
Signed-off-by: Stefan Weil <weil@mail.berlios.de>
---
hw/spitz.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/hw/spitz.c b/hw/spitz.c
index aa1487b..ef0bace 100644
--- a/hw/spitz.c
+++ b/hw/spitz.c
@@ -1041,7 +1041,7 @@ static void terrier_init(ram_addr_t ram_size,
kernel_cmdline, initrd_filename, cpu_model, terrier, 0x33f);
}
-QEMUMachine akitapda_machine = {
+static QEMUMachine akitapda_machine = {
.name = "akita",
.desc = "Akita PDA (PXA270)",
.init = akita_init,
--
1.5.6.5
prev parent reply other threads:[~2009-05-27 19:18 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-05-23 21:52 [Qemu-devel] [PATCH] Add static to local machine declaration Stefan Weil
2009-05-27 14:39 ` Anthony Liguori
2009-05-27 19:17 ` Stefan Weil [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4A1D91E3.6070502@weilnetz.de \
--to=stefan.weil@weilnetz.de \
--cc=aliguori@us.ibm.com \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.