]> xenbits.xensource.com Git - people/liuw/libxenctrl-split/libvirt.git/commitdiff
conf: Avoid double assignment in virDomainDiskRemove
authorMichal Privoznik <mprivozn@redhat.com>
Fri, 13 Apr 2012 09:19:29 +0000 (11:19 +0200)
committerMichal Privoznik <mprivozn@redhat.com>
Fri, 13 Apr 2012 10:01:59 +0000 (12:01 +0200)
Although it should be harmless to do:
    disk = disk = def->disks[i]
some not-so-wise compilers may fool around.
Besides, such assignment is useless here.

src/conf/domain_conf.c

index c6b97e1e5241433e06700337bf8fcf25edb469ac..a9c5cbce21ac01df51ad45c69c97f94dd198e564 100644 (file)
@@ -7124,7 +7124,7 @@ void virDomainDiskInsertPreAlloced(virDomainDefPtr def,
 virDomainDiskDefPtr
 virDomainDiskRemove(virDomainDefPtr def, size_t i)
 {
-    virDomainDiskDefPtr disk = disk = def->disks[i];
+    virDomainDiskDefPtr disk = def->disks[i];
 
     if (def->ndisks > 1) {
         memmove(def->disks + i,