]> xenbits.xensource.com Git - xen.git/commitdiff
golang/xenlight: add error return type to Context.Cpupoolinfo
authorNick Rosbrook <rosbrookn@gmail.com>
Mon, 23 Dec 2019 15:17:07 +0000 (10:17 -0500)
committerGeorge Dunlap <george.dunlap@citrix.com>
Tue, 24 Dec 2019 09:52:12 +0000 (09:52 +0000)
A previous commit that removed Context.CheckOpen revealed
an ineffectual assignent to err in Context.Cpupoolinfo, as
there is no error return type.

Since it appears that the intent is to return an error here,
add an error return value to the function signature.

Signed-off-by: Nick Rosbrook <rosbrookn@ainfosec.com>
Reviewed-by: George Dunlap <george.dunlap@citrix.com>
tools/golang/xenlight/xenlight.go

index 5f14cab6ce118c122d86eca06ba297e825809661..237f26bce9c6e3744593cb279993170756fb5ebf 100644 (file)
@@ -504,17 +504,17 @@ func (Ctx *Context) ListCpupool() (list []Cpupoolinfo) {
 }
 
 // int libxl_cpupool_info(libxl_ctx *ctx, libxl_cpupoolinfo *info, uint32_t poolid);
-func (Ctx *Context) CpupoolInfo(Poolid uint32) (pool Cpupoolinfo) {
+func (Ctx *Context) CpupoolInfo(Poolid uint32) (pool Cpupoolinfo, err error) {
        var c_cpupool C.libxl_cpupoolinfo
 
        ret := C.libxl_cpupool_info(Ctx.ctx, &c_cpupool, C.uint32_t(Poolid))
        if ret != 0 {
-               //err = Error(-ret)
+               err = Error(-ret)
                return
        }
        defer C.libxl_cpupoolinfo_dispose(&c_cpupool)
 
-       _ = pool.fromC(&c_cpupool)
+       err = pool.fromC(&c_cpupool)
 
        return
 }