Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Brace-less syntax for try/catch/finally gets confused by parentheses #5887

Closed
scabug opened this issue Jun 6, 2012 · 2 comments · Fixed by scala/scala#8071
Closed

Brace-less syntax for try/catch/finally gets confused by parentheses #5887

scabug opened this issue Jun 6, 2012 · 2 comments · Fixed by scala/scala#8071
Assignees
@som-snytt
Labels
help wanted parser
Milestone
2.13.1

Comments

@scabug
Copy link

scabug commented Jun 6, 2012

The parser accepts expressions without braces after "try" but when an open paren occurs, it fails to parse it as part of the expressions and expects the whole expression to be enclosed in parens:

scala> try 1 + 1 finally ()
res0: Int = 2

scala> try (1) + 1 finally ()
<console>:1: error: ';' expected but identifier found.
       try (1) + 1 finally ()
               ^
@scabug
Copy link
Author

scabug commented Jun 6, 2012

Imported From: https://issues.scala-lang.org/browse/SI-5887?orig=1
Reporter: @szeiger
Affected Versions: 2.10.0-M3, 2.10.0

@scabug
Copy link
Author

scabug commented Feb 15, 2015

@som-snytt said:
scala/scala#4334

@scabug scabug added this to the Backlog milestone Apr 7, 2017
@scabug scabug mentioned this issue Apr 7, 2017
Closed
@som-snytt som-snytt mentioned this issue May 17, 2019
Closed
@som-snytt som-snytt mentioned this issue May 18, 2019
Merged
@SethTisue SethTisue modified the milestones: Backlog, 2.13.1 Jun 18, 2019
@dwijnand dwijnand mentioned this issue Sep 10, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@som-snytt som-snytt

Labels
help wanted parser
Projects
None yet
Milestone
2.13.1
Development

Successfully merging a pull request may close this issue.

Try any expression som-snytt/scala
3 participants
@SethTisue @som-snytt @scabug