mirrors/prosody
1
0
Fork 0
mirror of https://github.com/bjc/prosody.git synced 2025-04-03 21:27:38 +03:00
Code Issues Projects Releases Packages Wiki Activity Actions

util.argparse: Revise 553c6204fe5b with a different approach

Browse source 
The second return value is (not insensibly) assumed to be an error. Instead of
returning a value there in the success case, copy the positional arguments
into the existing opts table.
This commit is contained in:
Matthew Wild 2022-04-25 15:24:56 +01:00
parent 0eef6dde1e
commit 93347db124
2 changed files with 6 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

2
spec/util_argparse_spec.lua
View file

@ -20,7 +20,7 @@ describe("parse", function()
local arg = { "--foo"; "bar"; "--baz" };
local opts, err = parse(arg);
assert.falsy(err);
assert.same({ foo = true }, opts);
assert.same({ foo = true, "bar", "--baz" }, opts);
assert.same({ "bar"; "--baz" }, arg);
end);

7
util/argparse.lua
View file

@ -5,7 +5,7 @@ local function parse(arg, config)
local parsed_opts = {};
if #arg == 0 then
return parsed_opts, arg;
return parsed_opts;
end
while true do
local raw_param = arg[1];
@ -47,7 +47,10 @@ local function parse(arg, config)
end
parsed_opts[param_k] = param_v;
end
return parsed_opts, arg;
for i = 1, #arg do
parsed_opts[i] = arg[i];
end
return parsed_opts;
end
return {