[libvirt] [PATCH 1/2] python: treat flags as default argument with value 0

2013-03-21 Thread Guannan Ren
The following four functions have not changed because default arguments have to come after positional arguments. Changing them will break the the binding APIs. migrate(self, dconn, flags, dname, uri, bandwidth): migrate2(self, dconn, dxml, flags, dname, uri, bandwidth): migrateToURI(self, duri,

Re: [libvirt] [PATCH 1/2] python: treat flags as default argument with value 0

2013-03-21 Thread Osier Yang
On 2013年03月21日 16:41, Guannan Ren wrote: The following four functions have not changed because default arguments have to come after positional arguments. Changing them will break the the binding APIs. migrate(self, dconn, flags, dname, uri, bandwidth): migrate2(self, dconn, dxml, flags, dname,

Re: [libvirt] [PATCH 1/2] python: treat flags as default argument with value 0

2013-03-21 Thread Guannan Ren
On 03/21/2013 06:47 PM, Osier Yang wrote: On 2013年03月21日 16:41, Guannan Ren wrote: The following four functions have not changed because default arguments have to come after positional arguments. Changing them will break the the binding APIs. migrate(self, dconn, flags, dname, uri, bandwidth):

Re: [libvirt] [PATCH 1/2] python: treat flags as default argument with value 0

2013-03-21 Thread Eric Blake
On 03/21/2013 04:47 AM, Osier Yang wrote: On 2013年03月21日 16:41, Guannan Ren wrote: The following four functions have not changed because default arguments have to come after positional arguments. Changing them will break the the binding APIs. migrate(self, dconn, flags, dname, uri,

Re: [libvirt] [PATCH 1/2] python: treat flags as default argument with value 0

2013-03-21 Thread Eric Blake
On 03/21/2013 09:24 PM, Guannan Ren wrote: migrate(self, dconn, flags, dname, uri, bandwidth): migrate2(self, dconn, dxml, flags, dname, uri, bandwidth): migrateToURI(self, duri, flags, dname, bandwidth): migrateToURI2(self, dconnuri, miguri, dxml, flags, dname, bandwidth): So how are they

Re: [libvirt] [PATCH 1/2] python: treat flags as default argument with value 0

2013-03-21 Thread Guannan Ren
On 03/22/2013 11:31 AM, Eric Blake wrote: On 03/21/2013 09:24 PM, Guannan Ren wrote: migrate(self, dconn, flags, dname, uri, bandwidth): migrate2(self, dconn, dxml, flags, dname, uri, bandwidth): migrateToURI(self, duri, flags, dname, bandwidth): migrateToURI2(self, dconnuri, miguri, dxml,