From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1MUkIt-0002CF-UU for qemu-devel@nongnu.org; Sat, 25 Jul 2009 12:41:00 -0400 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1MUkIp-00026w-As for qemu-devel@nongnu.org; Sat, 25 Jul 2009 12:40:59 -0400 Received: from [199.232.76.173] (port=40983 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1MUkIp-00026t-6T for qemu-devel@nongnu.org; Sat, 25 Jul 2009 12:40:55 -0400 Received: from mail-fx0-f224.google.com ([209.85.220.224]:37826) by monty-python.gnu.org with esmtp (Exim 4.60) (envelope-from ) id 1MUkIo-0006YT-LH for qemu-devel@nongnu.org; Sat, 25 Jul 2009 12:40:54 -0400 Received: by fxm24 with SMTP id 24so1877699fxm.34 for ; Sat, 25 Jul 2009 09:40:53 -0700 (PDT) Sender: Paolo Bonzini Message-ID: <4A6B3593.8050608@gnu.org> Date: Sat, 25 Jul 2009 18:40:51 +0200 From: Paolo Bonzini MIME-Version: 1.0 References: <4A6B0E6E.6030600@gnu.org> In-Reply-To: Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Subject: [Qemu-devel] Re: [PATCH 45/46] Remove AUDIO_DRIVERS define List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: malc Cc: aliguori@us.ibm.com, qemu-devel@nongnu.org, Juan Quintela On 07/25/2009 05:13 PM, malc wrote: > On Sat, 25 Jul 2009, Paolo Bonzini wrote: > >> On 07/25/2009 01:13 PM, malc wrote: >>> On Fri, 24 Jul 2009, Juan Quintela wrote: >>> >>>> Signed-off-by: Juan Quintela >>>> --- >>>> audio/audio.c | 25 ++++++++++++++++++++++++- >>>> configure | 3 --- >>>> 2 files changed, 24 insertions(+), 4 deletions(-) >>> That's how it used to be before, i see no reason of going back to this >>> style. >> Personally I prefer Juan's. It makes things more uniform; even though for >> example I disagree on changing autoconfy HAVE_* defines to CONFIG_*, it >> wouldn't be a problem at all to have CONFIG_* in qemu. > > For starters this is not a functional equivalent of the things currently > in place, please read the commit message of: 0c58ac1c7 You're right. Paolo