On 01/31/2014 11:10 AM, Eric Blake wrote:
>
>> > @@ -658,7 +658,7 @@ int qcow2_snapshot_list(BlockDriverState *bs,
>> > QEMUSnapshotInfo **psn_tab)
>> > return s->nb_snapshots;
>> > }
>> >
>> > - sn_tab = g_malloc0(s->nb_snapshots * sizeof(QEMUSnapshotInfo));
>> > + sn_tab = g_new0(QEMUSnapshotInfo, s->nb_snapshots);
>> > for(i = 0; i < s->nb_snapshots; i++) {
> Should checkpatch.pl care about the spacing after 'for'? Then again,
> it's unrelated to your conversion, so omitting a whitespace cleanup from
> this patch doesn't hurt.
>
checkpatch.pl won't check lines that aren't touched, as you see here.
r~
