qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Qemu-devel] Re: [PATCH 45/46] Remove AUDIO_DRIVERS define


From: Juan Quintela
Subject: [Qemu-devel] Re: [PATCH 45/46] Remove AUDIO_DRIVERS define
Date: Sat, 25 Jul 2009 20:57:57 +0200
User-agent: Gnus/5.11 (Gnus v5.11) Emacs/22.3 (gnu/linux)

malc <address@hidden> 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<address@hidden>
>> > > ---
>> > >   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

Thanks very much for the review.  I understand now why it was done the
way it is, searching for a solution.

Will try to think a better way of being able to autogenerate
config-host.h and maintaining the functionality of AUDIO_DRIVERS.

Later, Juan.




reply via email to

[Prev in Thread] Current Thread [Next in Thread]