@ffesti commented on this pull request.
> - rpmver v1 = rpmverParse(lua_tostring(L, -1));
+ if (lua_isstring(L, -1))
+ rpmversion = lua_tostring(L, -1);
+ if (rpmversion) {
+ 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);
+ }
The way this suppresses the warning from below is not quite clear to me and
should probably be more obvious in the code.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3678#pullrequestreview-2713593224
You are receiving this because you are subscribed to this thread.
Message ID: <rpm-software-management/rpm/pull/3678/review/2713593...@github.com>_______________________________________________
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
https://lists.rpm.org/mailman/listinfo/rpm-maint
