Should match cases use a keyword instead of pipe? about ponyc HOT 5 CLOSED

Another option would be to indent the cases:

match _first
  | None => _first = ex'
  | as ex: Exchange =>
    _first = None
end

This makes things clearer to me, but then has the problem that it looks like there's an end mssing after the last case.

from ponyc.

I prefer the pipe symbol, possibly because it's familiar from OCaml, etc. The only concrete reason I have for preferring it is for cases like this:

match x % 100
| 11 | 12 | 13 => "th"
else
  match x % 10
  | 1 => "st"
  | 2 => "nd"
  | 3 => "rd"
  else
    "th"
  end
end

Note the combination of the initial 11, 12 and 13 cases.

from ponyc.

Let's try the same example with a keyword, I'm using case because I'm lazy.

One case per line option:

match x % 100
case 11
case 12
case 13 => "th"
else
  match x % 10
  case 1 => "st"
  case 2 => "nd"
  case 3 => "rd"
  else
    "th"
  end
end

And the same with case indentation:

match x % 100
  case 11
  case 12
  case 13 => "th"
else
  match x % 10
    case 1 => "st"
    case 2 => "nd"
    case 3 => "rd"
  else
    "th"
  end
end

Cases grouped on a line option:

match x % 100
case 11 case 12 case 13 => "th"
else
  match x % 10
  case 1 => "st"
  case 2 => "nd"
  case 3 => "rd"
  else
    "th"
  end
end

And the same with case indentation:

match x % 100
  case 11 case 12 case 13 => "th"
else
  match x % 10
    case 1 => "st"
    case 2 => "nd"
    case 3 => "rd"
  else
    "th"
  end
end

I think that works well either on separate lines or grouped on a line, but it's definitely better with cases indented

from ponyc.

I agree with Andy that indentation is good. After all, you also use indentation for the two branches of an if-statement.

from ponyc.

We've decided to keep the pipe and not replace it with a keyword.

from ponyc.