Indentation for generic parameter lists about swift-mode HOT 3 CLOSED

Short answer: it could be improved. I will try it.

Discussion:

< and > are used as either binary operators or angle brackets. Treating them as a bracket unconditionally would break indentation of expressions. So we must distinguish them.

There are two cases where < ... > is used other than binary operator:

• generic-parameter-clause

  • typealias-declaration
  • function-declaration
  • union-style-enum
  • raw-value-style-enum
  • struct-declaration
  • class-declaration
  • protocol-method-declaration
  • protocol-initializer-declaration
  • initializer-declaration
  • subscript-head

• generic-argument-clause

  • type-identifier
  • primary-expression
  • explicit-member-expression

generic-argument-clause is ambiguous: foo(A < B, C > (d)) could be parsed as foo((A < B), (C > (d))) or foo(A<B, C>(d)) (invocation of A with two type arguments B, C and one value d). On the other hand, generic-parameter-clause seems to be much simpler; looking back one or two tokens is sufficient to recognize the open bracket.

Summary of grammers of generic-parameter-clause:

• typealias-declaration:

  • "typealias" identifier "<"

• function-declaration:

  • "func" identifier "<"
  • "func" operator "<"

• union-style-enum:

  • "enum" identifier "<"

• raw-value-style-enum

  • "enum" identifier "<"

• struct-declaration

  • "struct" identifier "<"

• class-declaration

  • "class" identifier "<"

• protocol-method-declaration:

  • "func" identifier "<"
  • "func" operator "<"

• protocol-initializer-declaration

  • "init" "<"
  • "init" "?" "<"
  • "init" "!" "<"

• initializer-declaration:

  • "init" "<"
  • "init" "?" "<"
  • "init" "!" "<"

• subscript-head:

  • "subscript" "<"

from swift-mode.

Fixed by 29759f5.
Thank you for reporting.

from swift-mode.

from swift-mode.