Two-column argument layout breaks with different AST shapes

[dansanduleac]

We've hit an obscure edge case for the two-column argList logic, where it doesn't end up keeping the pairs together.
The logic tries to assert that N/2+1 rows have the same "kind" of tree in the AST.
However, the values in the example seem to be

• Kind.PLUS (BinaryTree of + operations)
• Kind.VARIABLE
• Kind.METHOD_INVOCATION

palantir-java-format/palantir-java-format/src/main/java/com/palantir/javaformat/java/JavaInputAstVisitor.java

Lines 3222 to 3226 in fc94362

	for (int i = 1; i < size0; i++) {
	if (!expressionsAreParallel(rows, i, rows.size() / 2 + 1)) {
	return -1;
	}
	}

I honestly don't see much reason for this kind of heuristic (at least for values), so it might make sense to remove it.

Originally posted by @dansanduleac in palantir/conjure-verification#311 (comment)