Re: fix method delete() in src/main/java/org/libvirt/DomainSnapshot.java

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

 



On Wed, Mar 13, 2019 at 03:13:42PM +0800, 傅姜晨 wrote:
> if VDSP is set to null during the method delete(), Object snapshot can't be freed in method free() in the same class. 
> 
> 
> Showing 1 changed file with 1 addition and 7 deletions.
> View file
>  
> 8  src/main/java/org/libvirt/DomainSnapshot.java
> | | @@ -35,13 +35,7 @@ public DomainSnapshot(Connect virConnect, DomainSnapshotPointer VDSP) { |
> | | | | * @throws LibvirtException |
> | | | | */ |
> | | | | publicintdelete(intflags) throwsLibvirtException { |
> | | | | int success =0; |
> | | | | if (VDSP!=null) { |
> | | | | success = processError(libvirt.virDomainSnapshotDelete(VDSP, flags)); |
> | | | | VDSP=null; |
> | | | | } |
> | | | |
> |
> | | | | return success; |
> | | | | return processError(libvirt.virDomainSnapshotDelete(VDSP, flags)); |
> | | | | } |
> | | | |
> |
> | | | | @Override |

I don't know what's going on there, but this doesn't look like any
patch format I've ever seen. I guess your mail client has mangled
it. Could you resend the actual GIT commit diff, as an attachment
if you can't stop your mail client mangling text.

ie, on the command line run  'git format-patch -1'  which will save
your patch to a file, and then attach the file.

Regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|

--
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]

  Powered by Linux