On 04/21/2017 07:26 AM, Fam Zheng wrote:
> These are patches to:
>
> 1) reorder the function parameters so that Error **errp comes
> last.
>
> Error pointer in the middle of a function parameter list is very uncommon, and
> does caused mistakes, thus is not a good style. Change to the usual way.
>
> 2) apply the error_propagate_null.cocci semantics patch again.
>
> Fam Zheng (14):
> socket: Make errp the last parameter of socket_connect
> socket: Make errp the last parameter of inet_connect_saddr
> socket: Make errp the last parameter of unix_connect_saddr
> socket: Make errp the last parameter of vsock_connect_saddr
> block: Make errp the last parameter of bdrv_img_create
> crypto: Make errp the last parameter of functions
> mirror: Make errp the last parameter of mirror_start_job
> block: Make errp the last parameter of commit_active_start
> nfs: Make errp the last parameter of nfs_client_open
> fdc: Make errp the last parameter of fdctrl_connect_drives
> scsi: Make errp the last parameter of virtio_scsi_common_realize
> migration: Make errp the last parameter of local functions
> qga: Make errp the last parameter of qga_vss_fsfreeze
> error: Apply error_propagate_null.cocci again
Series:
Reviewed-by: Eric Blake <eblake@redhat.com>
but you have a typo in the commit message of 6/14
--
Eric Blake, Principal Software Engineer
Red Hat, Inc. +1-919-301-3266
Virtualization: qemu.org | libvirt.org