breaking: remove lua.open() function

This function's implementation and tests are messy and the function has
no real benefit.

Instead of using lua.open(.{}), use the individual functions like
lua.openBase() or lua.openOS().

lua.openLibs() opens all libraries.

Author: natecraddock

examples/zig-fn.zig

@@ -50,7 +50,7 @@ pub fn main() anyerror!void {
     lua.setGlobal("add");
 
     // We need to open the base library so the global print() is available
-    lua.open(.{ .base = true });
+    lua.openBase();
 
     // Our "program" is an inline string
     lua.doString(

src/lib.zig

@@ -435,13 +435,6 @@ pub const Libs53 = packed struct {
     debug: bool = false,
 };
 
-/// Bitflag for the Lua standard libraries
-pub const Libs = switch (lang) {
-    .lua51, .luajit => Libs51,
-    .lua52 => Libs52,
-    .lua53, .lua54, .luau => Libs53,
-};
-
 /// The type of the opaque structure that points to a thread and the state of a Lua interpreter
 pub const LuaState = c.lua_State;
 
@@ -2956,30 +2949,6 @@ pub const Lua = struct {
 
     // Standard library loading functions
 
-    /// Opens the specified standard library functions
-    /// Behaves like openLibs, but allows specifying which libraries
-    /// to expose to the global table rather than all of them
-    /// See https://www.lua.org/manual/5.4/manual.html#luaL_openlibs
-    pub fn open(lua: *Lua, libs: Libs) void {
-        if (libs.base) lua.requireF("_G", c.luaopen_base, true);
-        if (libs.string) lua.requireF(c.LUA_STRLIBNAME, c.luaopen_string, true);
-        if (libs.table) lua.requireF(c.LUA_TABLIBNAME, c.luaopen_table, true);
-        if (libs.math) lua.requireF(c.LUA_MATHLIBNAME, c.luaopen_math, true);
-        if (libs.os) lua.requireF(c.LUA_OSLIBNAME, c.luaopen_os, true);
-        if (libs.debug) lua.requireF(c.LUA_DBLIBNAME, c.luaopen_debug, true);
-
-        if (lang != .luau) {
-            if (libs.io) lua.requireF(c.LUA_IOLIBNAME, c.luaopen_io, true);
-            if (libs.package) lua.requireF(c.LUA_LOADLIBNAME, c.luaopen_package, true);
-        }
-
-        if (lang != .lua51 and lang != .luajit and libs.coroutine) lua.requireF(c.LUA_COLIBNAME, c.luaopen_coroutine, true);
-
-        if ((lang == .lua53 or lang == .lua54) and libs.utf8) lua.requireF(c.LUA_UTF8LIBNAME, c.luaopen_utf8, true);
-
-        if (lang == .lua52 and libs.bit) lua.requireF(c.LUA_BITLIBNAME, c.luaopen_bit32, true);
-    }
-
     /// Open all standard libraries
     /// See https://www.lua.org/manual/5.4/manual.html#luaL_openlibs
     pub fn openLibs(lua: *Lua) void {

src/tests.zig

@@ -154,22 +154,6 @@ test "standard library loading" {
         lua.openLibs();
     }
 
-    // open a subset of standard libraries with Zig wrapper
-    {
-        var lua = try Lua.init(&testing.allocator);
-        defer lua.deinit();
-
-        switch (ziglua.lang) {
-            .lua51 => lua.open(.{ .base = true, .package = true, .string = true, .table = true, .math = true, .io = true, .os = true, .debug = true }),
-            .lua52 => lua.open(.{ .base = true, .coroutine = true, .package = true, .string = true, .table = true, .math = true, .io = true, .os = true, .debug = true, .bit = true }),
-            .lua53, .lua54 => lua.open(.{ .base = true, .coroutine = true, .package = true, .string = true, .utf8 = true, .table = true, .math = true, .io = true, .os = true, .debug = true }),
-            .luau => lua.open(.{ .base = true, .coroutine = true, .string = true, .utf8 = true, .table = true, .math = true, .io = true, .os = true, .debug = true }),
-            .luajit => {
-                // TODO: why do tests crash?
-            },
-        }
-    }
-
     // open all standard libraries with individual functions
     // these functions are only useful if you want to load the standard
     // packages into a non-standard table
@@ -631,7 +615,8 @@ test "global table" {
     defer lua.deinit();
 
     // open some libs so we can inspect them
-    lua.open(.{ .math = true, .base = true });
+    lua.openBase();
+    lua.openMath();
     lua.pushGlobalTable();
 
     // find the print function
@@ -1143,7 +1128,7 @@ test "closing vars" {
     var lua = try Lua.init(&testing.allocator);
     defer lua.deinit();
 
-    lua.open(.{ .base = true });
+    lua.openBase();
 
     // do setup in Lua for ease
     try lua.doString(