[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 00/11] apply volume on client side (v3)
From: |
malc |
Subject: |
Re: [Qemu-devel] [PATCH 00/11] apply volume on client side (v3) |
Date: |
Tue, 13 Mar 2012 16:09:03 +0400 (MSK) |
User-agent: |
Alpine 2.00 (LNX 1167 2008-08-23) |
On Tue, 13 Mar 2012, Marc-Andr? Lureau wrote:
> On Tue, Mar 13, 2012 at 11:19 AM, Gerd Hoffmann <address@hidden> wrote:
> > I think we should remove the mixemu configure option. It makes code
> > bitrot. Patch #4 proves that. If you want to keep it because of the
> > overhead or other reasons I'd suggest to make it a runtime option.
>
> To be fair, the patch #4 only remove dead code (USE_MIXER wasn't
> enabled by mixemu).
>
> Making mixemu a runtime option sounds good to me.
No objection (yet... as i haven't seen how it'(s, d be) implemented)
--
mailto:address@hidden
- [Qemu-devel] [PATCH 10/11] Allow controlling volume with PulseAudio backend, (continued)
- [Qemu-devel] [PATCH 10/11] Allow controlling volume with PulseAudio backend, Marc-André Lureau, 2012/03/12
- Re: [Qemu-devel] [PATCH 00/11] apply volume on client side (v3), malc, 2012/03/12
- Re: [Qemu-devel] [PATCH 00/11] apply volume on client side (v3), Marc-André Lureau, 2012/03/12
- Re: [Qemu-devel] [PATCH 00/11] apply volume on client side (v3), malc, 2012/03/12
- Re: [Qemu-devel] [PATCH 00/11] apply volume on client side (v3), Marc-André Lureau, 2012/03/12
- Re: [Qemu-devel] [PATCH 00/11] apply volume on client side (v3), malc, 2012/03/12
- Re: [Qemu-devel] [PATCH 00/11] apply volume on client side (v3), Gerd Hoffmann, 2012/03/13
- Re: [Qemu-devel] [PATCH 00/11] apply volume on client side (v3), Kevin Wolf, 2012/03/13
- Re: [Qemu-devel] [PATCH 00/11] apply volume on client side (v3), Gerd Hoffmann, 2012/03/13
- Re: [Qemu-devel] [PATCH 00/11] apply volume on client side (v3), Marc-André Lureau, 2012/03/13
- Re: [Qemu-devel] [PATCH 00/11] apply volume on client side (v3),
malc <=