[PATCH 04/12] util: storage: Add notice for extension of struct virStorageSource

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



As we now have a deep copy function for struct virStorageSource add a
notice that extensions of the structure require also appropriate changes
to the virStorageSourceCopy func.
---
 src/util/virstoragefile.h | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/src/util/virstoragefile.h b/src/util/virstoragefile.h
index 2583e10..7f3f353 100644
--- a/src/util/virstoragefile.h
+++ b/src/util/virstoragefile.h
@@ -228,9 +228,11 @@ typedef virStorageDriverData *virStorageDriverDataPtr;
 typedef struct _virStorageSource virStorageSource;
 typedef virStorageSource *virStorageSourcePtr;

-/* Stores information related to a host resource.  In the case of
- * backing chains, multiple source disks join to form a single guest
- * view.  */
+/* Stores information related to a host resource.  In the case of backing
+ * chains, multiple source disks join to form a single guest view.
+ *
+ * IMPORTANT: When adding fields to this struct it's also necessary to add
+ * apropriate code to thevirStorageSourceCopy deep copy function */
 struct _virStorageSource {
     int type; /* virStorageType */
     char *path;
-- 
2.1.0

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]