On Tue, Nov 26, 2013 at 01:00:38PM -0700, Eric Blake wrote: > On 11/26/2013 10:00 AM, Daniel P. Berrange wrote: > > From: "Daniel P. Berrange" <berra...@redhat.com> > > > > The python code generator tries to figure out what class a > > method should be in by looking at the list of arguments for > > any which are object types. Unfortunately missing break > > statements meant that methods which have multiple object > > arguments (eg migrate as a virDomainPtr followed by a > > virConnectPtr) got added to multiple classes. The migrate > > methods should only be visible in the virDomain class, and > > the versions in the virConnect class have fubar arguments. > > > > Signed-off-by: Daniel P. Berrange <berra...@redhat.com> > > --- > > generator.py | 2 ++ > > 1 file changed, 2 insertions(+) > > ACK. Wow - this repo split has forced us to address some long-standing > bugs.
Automated testing found to be useful :-P I found this fixed some other broken methods too, so I'm changing the commit message to this Avoid generating the methods in multiple classes The python code generator tries to figure out what class a method should be in by looking at the list of arguments for any which are object types. Unfortunately missing break statements meant that methods which have multiple object arguments (eg migrate as a virDomainPtr followed by a virConnectPtr) got added to multiple classes. The following incorrect methods are removed by this change virStream.download (dup of virStorageVol.download) virStream.screenshot (dup of virDomain.screenshot) virStream.upload (dup of virStorageVol.upload) virConnect.migrate (dup of virDomain.migrate) virConnect.migrate2 (dup of virDomain.migrate2) virConnect.migrate3 (dup of virDomain.migrate3) virConnect.migrateToURI3 (dup of virDomain.migrateToURI3) Daniel -- |: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :| |: http://libvirt.org -o- http://virt-manager.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :| -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list