Daniel P. Berrange wrote:
On Thu, Feb 12, 2009 at 02:37:46PM -0500, Cole Robinson wrote:
> The recent API additions virSaveLastError and virFreeError upset the
> python bindings generator. Add a rule to explicitly skip them for the
> bindings, since (as far as I can tell) the libvirtError class provides
> everything we need.
Correct, this is not required for python layer, since we turn the
thread local error straight into an exception object the moment
we get it.
> diff --git a/python/generator.py b/python/generator.py
> index 9c71c05..0e8cca7 100755
> --- a/python/generator.py
> +++ b/python/generator.py
> @@ -344,6 +344,8 @@ skip_function = (
> 'virDefaultErrorFunc', # Python virErrorFuncHandler impl calls this from
C
> 'virConnectDomainEventRegister', # overridden in virConnect.py
> 'virConnectDomainEventDeregister', # overridden in virConnect.py
> + 'virSaveLastError', # We have our own python error wrapper
> + 'virFreeError', # Only needed if we use virSaveLastError
> )
ACK
Daniel
Pushed now.
Thanks,
Cole