> What I objected here is exposing false instead of nil when something is unset.

But there's no such thing in the PR linked to this item at all! That's what's 
so frustrating to me here and why I've been urging you to look at what was 
actually implemented for this ticket: 
https://github.com/rpm-software-management/rpm/commit/67abf72ef57e58251271f5b218f867357d78a4e0

"opts" table always exists so you don't need to test *that* against nil each 
and every time, and beyond that you get nil's for undefined options, and 
strings for defined (empty vs non-empty depending on whether it takes an 
argument or not). The other part is "arg" table (also always existing) where 
you get leftover arguments as a native Lua sequence, accessible with arg[n] and 
count with #arg. 

Other improvements in this area are in the works and some already done, but 
that's beyond the scope here.


-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/issues/1092#issuecomment-707141799
_______________________________________________
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint

Reply via email to