swiftlang/swift

[SR-7230] Improve error message for statement that begins with open brace

Open

#49,778 opened on 2018年3月19日

GitHub で見る
 (0 comments) (0 reactions) (0 assignees)Swift (69,989 stars) (10,719 forks)batch import
bugcompilerdiagnostics qualitygood first issue

説明
Previous ID SR-7230
Radar None
Original Reporter @jckarter
Type Bug
Votes 0
Component/s Compiler
Labels Bug, DiagnosticsQoI, StarterBug
Assignee None
Priority Medium

md5: 809fd159923a9a4b4bcf2169a9eb2a66

Issue Description:

Swift doesn't allow a statement to begin with an open brace, and the error message you get when you do is hostile to beginners, talking about "unused closures" rather than the syntactic issue:

https://twitter.com/twostraws/status/975740708601593857

This is likely to come up when a user is moving code around, so the error and associated fixits could be greatly improved.

コントリビューターガイド

技術スタック
swift
領域
developer experience
Issue 種別
bug
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
2
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1-3 hours
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
fresh
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
clear
前提条件
basic Swift syntaxunderstanding of compiler diagnostics
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
85
調査方針
Investigate the Swift compiler's parser in lib/Parse/Parser.cpp or related files to locate where errors for statements starting with open braces are emitted. The current error message mentions 'unused closures' instead of a syntax error. Improve the diagnostic to clearly indicate that a statement cannot begin with a brace and suggest possible corrections (e.g., wrapping in a closure or using a construct like `do {}`). Also update any fix it hints accordingly.