swiftlang/swift

[SR-5984] Error message for accidentally calling function in #selector(foo()) could suggest removing parens

Open

#48 543 ouverte le 26 sept. 2017

Voir sur GitHub
 (3 commentaires) (0 réactions) (0 assignés)Swift (10 719 forks)batch import
bugcompilerdiagnostics qualitygood first issuetype checker

Métriques du dépôt

Stars
 (69 989 stars)
Métriques de merge PR
 (Merge moyen 8j 17h) (510 PRs mergées en 30 j)

Description
Previous ID SR-5984
Radar rdar://problem/34640740
Original Reporter @huonw
Type Bug
Status In Progress
Resolution

Swift 4.0
Votes 0
Component/s Compiler
Labels Bug, DiagnosticsQoI, StarterBug, TypeChecker
Assignee None
Priority Medium

md5: 939952c8851f309540d6ce3aaa7d7c15

Issue Description:

import Foundation

class Foo {
    @objc func foo() {
        _ = #selector(foo())
    }
}

call-selector.swift:5:13: error: argument of '#selector' does not refer to an '@objc' method, property, or initializer
        _ = #selector(foo())
            ^         ~~~~~

This is close to the correct syntax (#selector(foo)) and the compiler could help guide the user to this. This probably comes up most often with selectors for methods with no arguments (https://bugs.swift.org/browse/SR-1016).

Guide contributeur

Direction de recherche
Examinez le système de diagnostic du compilateur Swift pour les erreurs #selector. Trouvez le code qui génère le message d'erreur actuel (l'argument de #selector ne fait pas référence à une méthode @objc). Modifiez le pour détecter lorsque l'utilisateur a accidentellement inclus des parenthèses dans la référence au sélecteur et suggérez de les supprimer. Cela implique probablement des modifications dans le vérificateur de type ou la phase Sema.
Stack technique
Aucun
Domaine
compilers
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Swift
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
80