Skip to content
This repository has been archived by the owner on Apr 13, 2023. It is now read-only.

Add support for else case #141

Closed
lucaswerkmeister opened this issue Aug 22, 2017 · 0 comments
Closed

Add support for else case #141

lucaswerkmeister opened this issue Aug 22, 2017 · 0 comments
Labels
Milestone

Comments

@lucaswerkmeister
Copy link
Contributor

See eclipse-archived/ceylon#6806.

@lucaswerkmeister lucaswerkmeister added this to the Next release milestone Aug 22, 2017
lucaswerkmeister added a commit that referenced this issue Aug 22, 2017
lucaswerkmeister added a commit that referenced this issue Aug 27, 2017
Switch cases can be preceded with the keyword `else` to remove the
disjointness requirement. (The `else` token is not recorded in the AST,
so we produce a fresh one.)

Fixes #141.
lucaswerkmeister added a commit that referenced this issue Aug 27, 2017
lucaswerkmeister added a commit to ceylon/ceylon.ast that referenced this issue Sep 28, 2017
Fixes part of #137, but the compiler also supports `else case` in switch
expressions, so ceylon.ast should probably support that too, even though
it’s not in the specification as of Ceylon 1.3.3.

Note that due to the inclusion of a non-disjoint case clause in
completeCompilationUnit, the completeCompilationUnit-related tests need
a ceylon.formatter version with a fix for eclipse-archived/ceylon.formatter#141.
Since the 1.3.3 release branch of ceylon.formatter was cut too early,
there is no public tree of ceylon.formatter with a fix for that issue
and also the version number 1.3.3-SNAPSHOT (as imported by
test.ceylon.ast.redhat); you can get one locally by checking out
ceylon.formatter’s 1.3.3 release tag and then reverting the version
update commit (eclipse-archived/ceylon.formatter@a30506b4ab).
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

1 participant