swiftlang/swift

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

Open

#48.543 aperta il 26 set 2017

Vedi su GitHub
 (3 commenti) (0 reazioni) (0 assegnatari)Swift (10.719 fork)batch import
bugcompilerdiagnostics qualitygood first issuetype checker

Metriche repository

Star
 (69.989 star)
Metriche merge PR
 (Merge medio 8g 17h) (510 PR mergiate in 30 g)

Descrizione
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).

Guida contributor

Direzione di ricerca
Esamina il sistema diagnostico del compilatore Swift per gli errori #selector. Trova il codice che genera l'attuale messaggio di errore (l'argomento di #selector non si riferisce a un metodo @objc). Modificalo per rilevare quando l'utente ha accidentalmente incluso le parentesi nel riferimento al selettore e suggerisci di rimuoverle. Ciò probabilmente comporta modifiche al type checker o alla fase Sema.
Tech stack
Nessuno
Dominio
compilers
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Swift
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80