Ambiguous variables should probably check arity

[rrrene]

Ambiguous variables should probably check arity

[rrrene]

That's a valid argument.

I am a bit uncertain whether or not to keep these checks at all. Originally you could write something like

defp server(https \\ false) do
  # ... code ...
end

# somewhere in a function:
server = server(true)

Now, with a regular call to server it might have become unclear if you were calling the variable or the function (using default arguments). But since Elixir 1.2 you would have to call server() to call the function, so this check might be outdated ...

Long story short, you're suggestion to check the arity is a very good one to solve those problems. 👍

[philipgiuliani]

Woho, awesome! :)

[rrrene]

v0.4.5 is live and contains this fix.

[net]

Yay!