Il 30/07/2012 16:47, Kevin Wolf ha scritto:
>> > +BlockJobInfo *block_job_query(BlockJob *job)
>> > +{
>> > + BlockJobInfo *info = g_new(BlockJobInfo, 1);
>> > + info->type = g_strdup(job->job_type->job_type);
>> > + info->device = g_strdup(bdrv_get_device_name(job->bs));
>> > + info->len = job->len;
>> > + info->offset = job->offset;
>> > + info->speed = job->speed;
>> > + return info;
>> > +}
> Why did you convert the initialisation to separate statement? If you
> really want to do this, I think using g_new0 would be safer now, but I
> actually like compound literals better.
Later on I will have some more initialization beyond the list of fields,
so I preferred an explicit list. I can change it back if you prefer.
Paolo
