[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 0/2] Use g_new() & friends where that makes obvi
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH 0/2] Use g_new() & friends where that makes obvious sense |
Date: |
Mon, 03 Feb 2014 09:40:03 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.2 (gnu/linux) |
Peter Maydell <address@hidden> writes:
> On 31 January 2014 15:53, Markus Armbruster <address@hidden> wrote:
>> PATCH 1/2 fixes a bug found by compiling the original PATCH 2/2.
>> Evidence for my claim that PATCH 2/2 lets the compiler catch more type
>> errors :)
>>
>> Markus Armbruster (2):
>> qga: Fix memory allocation pasto
>> Use g_new() & friends where that makes obvious sense
>
>> 186 files changed, 376 insertions(+), 415 deletions(-)
>
> No objection in principle, but I think this is going to be
> hideous merge pain since it touches a huge range of files.
> Could you split it up into separate patches that could
> reasonably go in via the appropriate submaintainer trees?
No merge pain at all if you simply rerun the included Coccinelle patch!
Splitting the patch may shift some pain from the choke point (you) to
submaintainers and me. I don't mind the splitting pain. I do mind the
"chase the nominal maintainer of obscure corner" pain.
If you really want it split: what about splitting off just the busy
and/or well-maintained subsystems?
- Re: [Qemu-devel] [PATCH 0/2] Use g_new() & friends where that makes obvious sense,
Markus Armbruster <=
- Re: [Qemu-devel] [PATCH 0/2] Use g_new() & friends where that makes obvious sense, Peter Maydell, 2014/02/03
- Re: [Qemu-devel] [PATCH 0/2] Use g_new() & friends where that makes obvious sense, Markus Armbruster, 2014/02/04
- Re: [Qemu-devel] [PATCH 0/2] Use g_new() & friends where that makes obvious sense, Markus Armbruster, 2014/02/13
- Re: [Qemu-devel] [PATCH 0/2] Use g_new() & friends where that makes obvious sense, Peter Maydell, 2014/02/13
- Re: [Qemu-devel] [PATCH 0/2] Use g_new() & friends where that makes obvious sense, Markus Armbruster, 2014/02/13
- Re: [Qemu-devel] [PATCH 0/2] Use g_new() & friends where that makes obvious sense, Markus Armbruster, 2014/02/18
- Re: [Qemu-devel] [PATCH 0/2] Use g_new() & friends where that makes obvious sense, Peter Maydell, 2014/02/21
- Re: [Qemu-devel] [PATCH 0/2] Use g_new() & friends where that makes obvious sense, Markus Armbruster, 2014/02/21
- Re: [Qemu-devel] [PATCH 0/2] Use g_new() & friends where that makes obvious sense, Peter Maydell, 2014/02/21