Align lines with different columns count

[poslegm]

Fixes #1783

TL;DR

//before
libraryDependencies ++= Seq(
  "a"   % "x" %% "1",
  "b" %%% "y"  % "2.2.2.2" % Test,
  "c"  %% "z"  % "3" % Test
)
//after
libraryDependencies ++= Seq(
  "a"   % "x" %% "1",
  "b" %%% "y"  % "2.2.2.2" % Test,
  "c"  %% "z"  % "3"       % Test
)

[kitbellew]

also, if you like lengthCompare, line.lengthCompare(column) > 0, although not necessarily more readable :)

[poslegm]

When I originally wrote lengthCompare I overlooked that line is Array. So lengthCompare is redundant optimization here :)

[poslegm]

@kitbellew Also when running this changes on scala-repos I discovered idempotency violations on master:

scala-repos diff: poslegm/scala-repos@4554a0a

broken example:

<<< ONLY step 1
private def getTopN[T](s: Iterable[T], n: Int)(
      implicit ord: Ordering[T]): Seq[T]
>>>
private def getTopN[T](s: Iterable[T], n: Int)(
    implicit ord: Ordering[T]	   
): Seq[T]
<<< ONLY step 2
private def getTopN[T](s: Iterable[T], n: Int)(
    implicit ord: Ordering[T]	   
): Seq[T]
>>>
private def getTopN[T](s: Iterable[T], n: Int)(implicit
    ord: Ordering[T]
): Seq[T]

(works correct at 2.4.2)

Is it related to your latest changes?

[kitbellew]

@kitbellew Also when running this changes on scala-repos I discovered idempotency violations on master:

scala-repos diff: poslegm/scala-repos@4554a0a

broken example:

<<< ONLY step 1
private def getTopN[T](s: Iterable[T], n: Int)(
      implicit ord: Ordering[T]): Seq[T]
>>>
private def getTopN[T](s: Iterable[T], n: Int)(
    implicit ord: Ordering[T]	   
): Seq[T]
<<< ONLY step 2
private def getTopN[T](s: Iterable[T], n: Int)(
    implicit ord: Ordering[T]	   
): Seq[T]
>>>
private def getTopN[T](s: Iterable[T], n: Int)(implicit
    ord: Ordering[T]
): Seq[T]

(works correct at 2.4.2)

Is it related to your latest changes?

probably :( implicit handling is new. can you disable them for now, open a ticket for me to track, please?

[poslegm]

#1787