On Wed, Jan 22, 2014 at 1:57 AM, José Valim <notifications@github.com>wrote:

> In lib/elixir/lib/exception.ex:
>
> >
> > -    if is_list(arity) do
> > -      inspected = lc x inlist arity, do: inspect(x)
> > -      "#{inspect module}.#{fun}(#{Enum.join(inspected, ", ")})"
> > -    else
> > -      "#{inspect module}.#{fun}/#{arity}"
> > -    end
> > +  def format_arity(arity) when is_list(arity) do
>
> If we are going to expose this function, we need docs. Otherwise it needs
> to be defp (I would vote for defp).
>

Agreed