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 <ebl...@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
signature.asc
Description: OpenPGP digital signature