Feature #16296
openAlternative behavior for `...` in method body if `...` is not in method definition
Description
In #16253 we settled on a syntax where the remainder arguments captured via ...
in the method definition can be forwarded via ...
in the method body. I think that was the correct decision.
But I can't forget about the use case presented by zverok (and in #15049) where the method definition is used to specify mandatory and default parameters and then forward all of them to another method. I've also experienced that same use case in my code. Using the current syntax we would need to do this:
def get(path:, accept: :json, headers: {}, ...)
_request(method: :get, path: path, accept: accept, headers: headers, ...)
end
def post(path:, body:, accept: :json, headers: {}, ...)
_request(method: :post, path: path, body: body, accept: accept, headers: headers, ...)
end
Which feels pointlessly repetitive to me. So I was thinking that maybe if ...
is not present in the method definition, then in the method body ...
could take on the meaning of "all arguments of the method". Then the code would look like this:
def get(path:, accept: :json, headers: {}, **opts)
_request(method: :get, ...)
end
def post(path:, body:, accept: :json, headers: {}, **opts)
_request(method: :post, ...)
end
In those examples (no positional parameters) it would also allow Hash[...]
or {}.replace(...)
to get the hash of all keyword arguments.
Pro: it allows a new useful and powerful behavior
Con: some may consider it 'unclean' to change the behavior of ...
based on the method definition