add more Ex variant functions

Author: natecraddock

src/tests.zig

@@ -252,7 +252,7 @@ test "type of and getting values" {
     lua.pushLightUserdata(&value);
     lua.pushNil();
     lua.pushNumber(0.1);
-    _ = lua.pushThread();
+    lua.pushThread();
     try expectEqualStrings(
         "all your codebase are belong to us",
         lua.pushStringEx("all your codebase are belong to us"),
@@ -1024,18 +1024,18 @@ test "debug interface" {
                 .call => {
                     l.getInfo(.{ .l = true, .r = true }, i);
                     if (i.current_line.? != 2) panic("Expected line to be 2", .{});
-                    _ = l.getLocal(i, i.first_transfer) catch unreachable;
+                    l.getLocal(i, i.first_transfer) catch unreachable;
                     if ((l.toNumber(-1) catch unreachable) != 3) panic("Expected x to equal 3", .{});
                 },
                 .line => if (i.current_line.? == 4) {
                     // modify the value of y to be 0 right before returning
                     l.pushNumber(0);
-                    _ = l.setLocal(i, 2) catch unreachable;
+                    l.setLocal(i, 2) catch unreachable;
                 },
                 .ret => {
                     l.getInfo(.{ .l = true, .r = true }, i);
                     if (i.current_line.? != 4) panic("Expected line to be 4", .{});
-                    _ = l.getLocal(i, i.first_transfer) catch unreachable;
+                    l.getLocal(i, i.first_transfer) catch unreachable;
                     if ((l.toNumber(-1) catch unreachable) != 3) panic("Expected result to equal 3", .{});
                 },
                 else => unreachable,
@@ -1081,7 +1081,7 @@ test "debug upvalues" {
 
     // now make the function an "add five" function
     lua.pushNumber(5);
-    _ = try lua.setUpvalue(-2, 1);
+    try lua.setUpvalue(-2, 1);
 
     // test a bad index (the valid one's result is unpredicable)
     try expectError(Error.Fail, lua.upvalueId(-1, 2));
@@ -1142,7 +1142,7 @@ test "aux check functions" {
             _ = l.checkBytes(3);
             _ = l.checkNumber(4);
             _ = l.checkString(5);
-            _ = l.checkType(6, .boolean);
+            l.checkType(6, .boolean);
             return 0;
         }
     }.inner);

src/ziglua.zig

@@ -829,9 +829,14 @@ pub const Lua = struct {
         return c.lua_pushstring(lua.state, str.ptr).?[0..str.len :0];
     }
 
+    /// Pushes this thread onto the stack
+    pub fn pushThread(lua: *Lua) void {
+        _ = lua.pushThreadEx();
+    }
+
     /// Pushes this thread onto the stack
     /// Returns true if this thread is the main thread of its state
-    pub fn pushThread(lua: *Lua) bool {
+    pub fn pushThreadEx(lua: *Lua) bool {
         return c.lua_pushthread(lua.state) != 0;
     }
 
@@ -1203,7 +1208,13 @@ pub const Lua = struct {
     }
 
     /// Gets information about a local variable
-    pub fn getLocal(lua: *Lua, info: *DebugInfo, n: i32) ![:0]const u8 {
+    pub fn getLocal(lua: *Lua, info: *DebugInfo, n: i32) !void {
+        _ = try lua.getLocalEx(info, n);
+    }
+
+    /// Gets information about a local variable
+    /// Returns the name of the local variable
+    pub fn getLocalEx(lua: *Lua, info: *DebugInfo, n: i32) ![:0]const u8 {
         var ar: Debug = undefined;
         ar.i_ci = @ptrCast(*c.struct_CallInfo, info.private);
         if (c.lua_getlocal(lua.state, &ar, n)) |name| {
@@ -1233,9 +1244,15 @@ pub const Lua = struct {
         c.lua_sethook(lua.state, hook_fn, hook_mask, count);
     }
 
+    /// Sets the value of a local variable
+    pub fn setLocal(lua: *Lua, info: *DebugInfo, n: i32) !void {
+        _ = try lua.setLocalEx(info, n);
+    }
+
     /// Sets the value of a local variable
     /// Returns an error when the index is greater than the number of active locals
-    pub fn setLocal(lua: *Lua, info: *DebugInfo, n: i32) ![:0]const u8 {
+    /// Returns the name of the local variable
+    pub fn setLocalEx(lua: *Lua, info: *DebugInfo, n: i32) ![:0]const u8 {
         var ar: Debug = undefined;
         ar.i_ci = @ptrCast(*c.struct_CallInfo, info.private);
         if (c.lua_setlocal(lua.state, &ar, n)) |name| {
@@ -1245,7 +1262,14 @@ pub const Lua = struct {
     }
 
     /// Sets the value of a closure's upvalue
-    pub fn setUpvalue(lua: *Lua, func_index: i32, n: i32) ![:0]const u8 {
+    /// Returns an error if the upvalu does not exist
+    pub fn setUpvalue(lua: *Lua, func_index: i32, n: i32) !void {
+        _ = try lua.setUpvalueEx(func_index, n);
+    }
+
+    /// Sets the value of a closure's upvalue
+    /// Returns the name of the upvalue or an error if the upvalue does not exist
+    pub fn setUpvalueEx(lua: *Lua, func_index: i32, n: i32) ![:0]const u8 {
         if (c.lua_setupvalue(lua.state, func_index, n)) |name| {
             return std.mem.span(name);
         }