swiftlang/swift

[SR-978] Warn on do {} with only one empty catch {} block

Open

#43,590 opened on 2016年3月18日

GitHub で見る
 (10 comments) (0 reactions) (0 assignees)Swift (69,989 stars) (10,719 forks)batch import
compilerdiagnostics qualitydo catchgood first issueimprovementmissing warningstatementsswift 5.9

説明
Previous ID SR-978
Radar None
Original Reporter @jckarter
Type Improvement
Votes 0
Component/s Compiler
Labels Improvement, StarterBug
Assignee MultiColourPixel (JIRA)
Priority Medium

md5: d5f41243d2aaf067f1ecd57198c44970

Issue Description:

Eric Knapp caught some sample code in the wild unwisely using an empty catch block to swallow errors:

https://twitter.com/ejknapp/status/710589235464523776

We should warn on this to encourage people to use try! to trap on unexpected errors instead.

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

技術スタック
swift
領域
developer experience
Issue 種別
feature
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
3
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1-2 days
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
stale
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
clear
前提条件
understanding of Swift syntaxbasic compiler diagnostics knowledge
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
60
調査方針
The issue requests a warning for `do {} catch {}` blocks where the catch block is empty (i.e., only one empty catch block). To implement this, you need to add a diagnostic in the Swift compiler's semantic analysis phase. Look at existing warnings like 'unused variable' or 'redundant pattern' in the Swift compiler source (likely under `lib/Sema/` or `include/swift/AST/Diagnostics*`). The issue has a JIRA reference but no code changes yet. Check the comments for any prior discussion or proposed implementation details. The goal is to emit a warning when a catch clause is empty, encouraging use of `try!` instead.