@pmatilai commented on this pull request.
> + rpmver v1 = rpmverParse(rpmversion);
+ rpmver v2 = rpmverParse(removed_in);
+ if (v2 && rpmverCmp(v1, v2) < 0)
+ err = 0;
+ rpmverFree(v2);
+ rpmverFree(v1);
+ }
+ lua_pop(L, 1);
+
+ if (err) {
+ luaL_error(L, "%s() is no longer available, use rpm.spawn() or
rpm.execute() instead", func);
+ }
+
+ int warn = 1;
+ if (rpmversion) {
+ rpmver v1 = rpmverParse(rpmversion);
Yup. The reason it's written this way is the luaL_error() return from the
middle - we need allocations freed up before luaL_error() might get called.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3678#discussion_r2012171298
You are receiving this because you are subscribed to this thread.
Message ID: <rpm-software-management/rpm/pull/3678/review/2713878...@github.com>_______________________________________________
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
https://lists.rpm.org/mailman/listinfo/rpm-maint
