From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43)
	id 1M7Rg5-00052v-4O
	for qemu-devel@nongnu.org; Fri, 22 May 2009 06:08:37 -0400
Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43)
	id 1M7Rfy-00052g-CV
	for qemu-devel@nongnu.org; Fri, 22 May 2009 06:08:36 -0400
Received: from [199.232.76.173] (port=57499 helo=monty-python.gnu.org)
	by lists.gnu.org with esmtp (Exim 4.43) id 1M7Rfy-00052d-8g
	for qemu-devel@nongnu.org; Fri, 22 May 2009 06:08:30 -0400
Received: from moutng.kundenserver.de ([212.227.126.177]:50807)
	by monty-python.gnu.org with esmtp (Exim 4.60)
	(envelope-from <weil@mail.berlios.de>) id 1M7Rfx-0006Dz-JF
	for qemu-devel@nongnu.org; Fri, 22 May 2009 06:08:29 -0400
Message-ID: <4A167996.6030200@mail.berlios.de>
Date: Fri, 22 May 2009 12:08:22 +0200
From: Stefan Weil <weil@mail.berlios.de>
MIME-Version: 1.0
Subject: Re: [Qemu-devel] [PATCH] Add new function qemu_register_machines
References: <4A159CBE.9020606@mail.berlios.de>
	<200905212333.21133.paul@codesourcery.com>
In-Reply-To: <200905212333.21133.paul@codesourcery.com>
Content-Type: text/plain; charset=ISO-8859-15
Content-Transfer-Encoding: 7bit
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: Paul Brook <paul@codesourcery.com>
Cc: Anthony Liguori <aliguori@us.ibm.com>, qemu-devel@nongnu.org

Paul Brook schrieb:
> On Thursday 21 May 2009, Stefan Weil wrote:
>> Add new function qemu_register_machines.
>>
>> The patch removes the unused prototype register_machines
>> and adds a new function which makes registration of
>> more than one machine a little easier.
>
> I don't see this as an improvement.
>
> Paul

Hi Paul,

well, it's a matter of personal taste whether you prefer
to have one or two interfaces for machine registration.

The new function is an improvement if you want to reduce
lines of code and binary code size.

Removing unused entries like register_machines from
header files is also an improvement. This part of
my patch is fixed now in QEMU HEAD, thanks Anthony!

And adding "static" to a machine declaration which is
only used locally is also an improvement (hw/spitz.c).

I assume that you agree that those last two points are
an improvement.

So the patch improves two details and adds something
where people disagree about its usefulness.

What now? Will the patch be rejected because you
don't see an improvement? Will you fix the missing
"static" yourself? Do you want a new patch without
qemu_register_machines? Or a new patch which uses
qemu_register_machines for all files with more than
one machine?

Your short feedback is not really helpful because there
remain too many questions.

Stefan