Re: [libvirt] [PATCH RFC 10/27] storage: Avoid forward declaration of virStorageVolDelete
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
Attachments:
- signature.asc (application/pgp-signature — 621 bytes)