Mutiline args function that use multiple object argument make the format wierd

[blabdouze]

Giving the following code with a line_length of 120 :

func add_road_segment(v1: Vector3, v2: Vector3, v3: Vector3, v4: Vector3, v5: Vector3, v6: Vector3) -> void:
	var uv_quads_1 := HexMesh.generate_quad_uv(0, 1, 0, 0)
	add_quad(
			HexMesh.Vertex.new(v1, HexMesh.Vertex.NO_COLOR, uv_quads_1.get(0)), 
			HexMesh.Vertex.new(v2, HexMesh.Vertex.NO_COLOR, uv_quads_1.get(1)),
			HexMesh.Vertex.new(v4, HexMesh.Vertex.NO_COLOR, uv_quads_1.get(2)),
			HexMesh.Vertex.new(v5, HexMesh.Vertex.NO_COLOR, uv_quads_1.get(3))
	)

it gets reformated as :

func add_road_segment(v1: Vector3, v2: Vector3, v3: Vector3, v4: Vector3, v5: Vector3, v6: Vector3) -> void:
	var uv_quads_1 := HexMesh.generate_quad_uv(0, 1, 0, 0)
	add_quad(HexMesh.Vertex.new(v1, HexMesh.Vertex.NO_COLOR, uv_quads_1.get(0)), HexMesh.Vertex.new(
			v2,
			HexMesh.Vertex.NO_COLOR,
			uv_quads_1.get(1),
		), HexMesh.Vertex.new(v4, HexMesh.Vertex.NO_COLOR, uv_quads_1.get(2)), HexMesh.Vertex.new(
			v5,
			HexMesh.Vertex.NO_COLOR,
			uv_quads_1.get(3),
		))

[NathanLovato]

Thanks. Right now it breaks on the new call and sees that it's enough to fit the line. It works in many cases, but in this case, it really should try to keep one argument per line first then fall-back to wrapping all the method calls at the end of the chain (HexMesh.Vertex.method).

[NathanLovato]

That was a fun one to fix. I really wish GDScript could just gracefully parse vertical call chains like:

a
.b()
.c
.d()

A bunch of complexity and edge cases like the one you encountered stem from that, you have to do more complicated formatting logic things to work around limitations of the official parser and that formatting logic has more edge cases too as a result.