As to the whitespace notes... I try to line things up. I believe it makes
them easier to read.

Dave
On Jan 23, 2014 12:03 PM, "Eric Meadows-Jönsson" <notifications@github.com>
wrote:

> In lib/iex/test/iex/evaluator_test.exs:
>
> > +  end
> > +
> > +  test "calculate_width returns 0 if the different in widths is > 8" do
> > +    assert E.calculate_width([{"a",    "b", "c"},
> > +                              {"aaaaaaaaa",  "b", "c"},
> > +                              {"a",    "b", "c"}], 0) == 0
> > +  end
> > +
> > +  test "pretty_stacktrace returns formatted result in columns" do
> > +    frames = [
> > +      { List,   :one,    1, [file: "loc",    line: 1] },
> > +      { String, :second, 2, [file: "loc2",   line: 102] },
> > +      { IEx,    :three,  3, [file: "longer", line: 1234] },
> > +      { List,   :four,   4, [file: "loc",    line: 1] },
> > +
> > +    ]
>
> Unnecessary whitespace.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/elixir-lang/elixir/pull/2016/files#r9127407>
> .
>