fix: remove ffi.gc entirely

Author: bycruz

src/init.lua

@@ -114,7 +114,7 @@ lib.git_libgit2_init()
 ---@field indexWrite fun()
 ---@field indexWriteTree fun(): string Tree SHA
 ---@field fetch fun(remote: string)
-
+---@field free fun()
 local sigLayout = ffi.typeof("struct { char *name; char *email; int64_t when_time; int when_offset; } *")
 
 ---@param code integer
@@ -169,7 +169,7 @@ local function openRepo(path, bare)
 		check(lib.git_repository_open(rp, path))
 	end
 	---@type git2.ffi.Repository
-	local repo = ffi.gc(rp[0], lib.git_repository_free)
+	local repo = rp[0]
 
 	---@type git2.Repo
 	local M = {}
@@ -256,6 +256,8 @@ local function openRepo(path, bare)
 		lib.git_remote_free(rmt[0])
 	end
 
+	function M.free() lib.git_repository_free(repo) end
+
 	return M
 end
 

tests/git2.test.lua

@@ -33,25 +33,29 @@ test.it("init creates a non-bare repo", function()
 	test.truthy(repo.path():find(".git"))
 	test.equal(repo.isBare(), false)
 	test.equal(repo.headUnborn(), true)
+	repo.free()
 end)
 
 test.it("init bare creates a bare repo", function()
 	local repo = git2.init(mkTmp("bare"), true)
 	test.equal(repo.isBare(), true)
+	repo.free()
 end)
 
 test.it("open existing repo works", function()
 	local src = debug.getinfo(1, "S").source:sub(2):match("(.*[/\\])")
 	local repo = git2.open(src .. "..")
 	test.truthy(repo.path())
 	test.equal(repo.isBare(), false)
+	repo.free()
 end)
 
 test.it("head returns a 40-char sha", function()
 	local dir = mkTmp("head")
 	mkCommit(dir, "init")
 	local repo = git2.open(dir)
 	test.equal(#repo.head(), 40)
+	repo.free()
 end)
 
 test.it("commitLookup returns correct metadata", function()
@@ -65,11 +69,13 @@ test.it("commitLookup returns correct metadata", function()
 	test.equal(c.author.name, "T")
 	test.equal(c.author.email, "t@t.com")
 	test.truthy(c.time > 0)
+	repo.free()
 end)
 
 test.it("revparse HEAD matches head()", function()
 	local dir = mkTmp("rev")
 	mkCommit(dir, "rev")
 	local repo = git2.open(dir)
 	test.equal(repo.revparse("HEAD"), repo.head())
+	repo.free()
 end)