[libvirt] [PATCH RFC 10/27] storage: Avoid forward declaration of virStorageVolDelete

Eric Blake eblake at redhat.com
Thu Dec 19 04:56:26 UTC 2013


On 12/16/2013 09:32 AM, Peter Krempa wrote:
> Move the code around so that the forward declaration isn't needed.
> ---
>  src/storage/storage_driver.c | 181 ++++++++++++++++++++++---------------------
>  1 file changed, 91 insertions(+), 90 deletions(-)

ACK; mechanical.

> 
> diff --git a/src/storage/storage_driver.c b/src/storage/storage_driver.c
> index f08255e..8b1dcae 100644
> --- a/src/storage/storage_driver.c
> +++ b/src/storage/storage_driver.c
> @@ -1495,7 +1495,97 @@ cleanup:
>      return ret;
>  }
> 
> -static int storageVolDelete(virStorageVolPtr obj, unsigned int flags);
> +
> +static int
> +storageVolDelete(virStorageVolPtr obj,
> +                 unsigned int flags) {

This { should be on its own line; but you were doing straight code
motion so it's okay if you don't clean it up until a later patch.  (Or
you could clean it up here, as long as the commit message mentions that
you did minor tweaking to fix style issues as part of the code motion)

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 621 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20131218/5dec9808/attachment-0001.sig>


More information about the libvir-list mailing list