[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 0/3] block: Use g_new() & friends more
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH 0/3] block: Use g_new() & friends more |
Date: |
Tue, 01 Jul 2014 12:57:41 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.3 (gnu/linux) |
Kevin Wolf <address@hidden> writes:
> Am 01.07.2014 um 11:06 hat Markus Armbruster geschrieben:
>> Markus Armbruster <address@hidden> writes:
>>
>> > This series is on top of Kevin's "block: Handle failure for
>> > potentially large allocations".
>> >
>> > PATCH 1+2 convert some allocations. While preparing them, I stumbled
>> > over some useless casts, which led to PATCH 3.
>>
>> Fell through the cracks despite Max's R-by, and now it doesn't apply
>> anymore. Pity, because it improves type safety and reduces
>> vulnerability to integer overflows.
>
> Whoops. Not sure if Stefan had any reasons not to merge it (the tree was
> his for three weeks because I was away in mine), but it probably really
> just fell through the cracks.
Happens :) I should've complained earlier.
>> Is this change wanted? Unless I get a "yes" from a maintainer, I'll
>> drop it.
>
> Yes, I think it's a good change.
Okay, I'll redo the series for 2.2.