Re: [PATCH] -Added autocloseable to connect class (set close() return type to void so to adapt it to the autocloseable interface)

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

 



[forwarding my answer back to the list]

Pasquale, please keep the discussion on the list, you should probably
use the "Reply To All" (or whatever GMail calls it) button when
answering.

At Tue, 08 Apr 2014 16:09:31 +0200,
Claudio Bley wrote:
> 
> [Remember /not/ to top-post on technical lists.]
> 
> At Mon, 7 Apr 2014 20:28:28 +0200,
> Pasquale Dir wrote:
> > 
> > Hello,
> > is it ok if I'll send multiple patches for each added function? I'd start
> > with the getCPUStats which is the most complex as it introduces new jna
> > types too.
> 
> Yes, that would be a good start. (although I'd probably do it in
> reverse, starting out with the small changes, then tackling the
> complex ones).
> 
> > As for Thread.sleep() is it ok to substitute it with a wait(long) ? Or you
> > have any alternative? That wait is needed for the alghoritm to work.
> 
> I'm not convinced that this functionality belongs into the library at
> all.  Maybe we could add a convenience method doing the computation
> for two given stats and a time period, but I don't feel strongly in
> support of adding this method.
> 
> Regards,
> Claudio

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