swiftlang/swift

[SR-1807] Sema should catch and warn about self capture in closures

Open

#44,416 opened on 2016年6月17日

GitHub で見る
 (6 comments) (0 reactions) (0 assignees)Swift (69,989 stars) (10,719 forks)batch import
compilergood first issuenew feature

説明
Previous ID SR-1807
Radar rdar://26865978
Original Reporter @CodaFi
Type New Feature
Votes 0
Component/s Compiler
Labels New Feature, StarterBug
Assignee None
Priority Medium

md5: 3ea294e377274fa3d27e3c3b76ff7bbd

Issue Description:

To maintain parity with Objective-C, we should use our ARC analysis pass in Sema to check for the presence of self capture in blocks without weak or @noescape. This should be a very simple warning modeled after the one in /clang/lib/Sema/SemaChecking.cpp void diagnoseRetainCycle().

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

技術スタック
swiftcpp
領域
tooling
Issue 種別
feature
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
2
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1-3 hours
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
stale
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
clear
前提条件
C++compiler internals
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
60
調査方針
The issue requests adding a warning in the Swift compiler's semantic analysis (Sema) to detect self capture in closures without weak or @noescape. The implementation should follow the existing diagnoseRetainCycle() function in clang/lib/Sema/SemaChecking.cpp. Look into the Swift compiler's ARC analysis pass and add a similar check. The warning should be emitted when a closure captures self strongly and the closure escapes.
[SR-1807] Sema should catch and warn about self capture in closures · swiftlang/swift#44416 | Good First Issue