[v1] Blacklist virGetLastError{Code,Domain}

[libvirt] [python PATCH] Blacklist virGetLastError{Code,Domain}

Daniel P. Berrangé posted 1 patch 5 years, 11 months ago

Diff against v1 v1
Download mbox

Failed in applying to current master (apply log)

sanitytest.py | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)

Expand all Fold all

[libvirt] [python PATCH] Blacklist virGetLastError{Code,Domain}

Posted by Daniel P. Berrangé 5 years, 11 months ago

These methods will not be exposed to apps, since we auto raise
all errors.

Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
---
 sanitytest.py | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

Pushed as a trivial build fix

diff --git a/sanitytest.py b/sanitytest.py
index 190d32b..d5c23dc 100644
--- a/sanitytest.py
+++ b/sanitytest.py
@@ -234,7 +234,9 @@ for name in sorted(basicklassmap):
                     "ErrorFunc", "FreeError",
                     "SaveLastError", "ResetError"]:
             continue
-        elif func in ["GetLastError", "GetLastErrorMessage", "ResetLastError", "Initialize"]:
+        elif func in ["GetLastError", "GetLastErrorMessage",
+                      "GetLastErrorCode", "GetLastErrorDomain",
+                      "ResetLastError", "Initialize"]:
             func = "vir" + func
         elif func == "SetErrorFunc":
             func = "RegisterErrorHandler"
-- 
2.17.0

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list