JohnSnow <js...@redhat.com>写到：
>
>
>On 10/14/2016 05:51 AM, Xiubo Li wrote:
>> Make it a bit clear and better readable.
>>
>
>Suggestion: "Make it clearer and more readable."
>
Yes, see the next version.
>>
>> if (qemu_rbd_set_auth(cluster, secretid, errp) < 0) {
>> rados_shutdown(cluster);
>
>Did you mean to remove rados_shutdown() here, too?
>
I will fix this.
>> ret = rbd_create(io_ctx, name, bytes, &obj_order);
>> - rados_ioctx_destroy(io_ctx);
>> - rados_shutdown(cluster);
>> if (ret < 0) {
>> error_setg_errno(errp, -ret, "error rbd create");
>> - return ret;
>> }
>>
>> + rados_ioctx_destroy(io_ctx);
>> +
>> +failed_shutdown:
>
>Since this executes on the non-error pathway too, I might just call
>this
>'shutdown'.
>
Agree.
Thanks very much.
BRS
Xiubo
>> + rados_shutdown(cluster);
>> return ret;
>> }
>>
>>
>
>--
>梛s
