swiftlang/swift

Diagnostic for `static nonmutating` should be improved

Open

#77 835 ouverte le 25 nov. 2024

Voir sur GitHub
 (11 commentaires) (0 réactions) (1 assigné)Swift (10 719 forks)batch import
bugcompilerdeclarationsdiagnostics qualitygood first issuemutatingstatic declarationsswift 6.1type checker

Métriques du dépôt

Stars
 (69 989 stars)
Métriques de merge PR
 (Merge moyen 7j 6h) (556 PRs mergées en 30 j)

Description

Description

No response

Reproduction

struct S {
  nonmutating static func foo() {}
}

430 | 
431 | struct S {
432 |   nonmutating static func foo() {}
    |   `- error: static functions must not be declared mutating [static_functions_not_mutating]
433 | }
434 |

Expected behavior

An example approach to improving this error message is to list both modifiers.

We can also tailor it to call out the modifier in question as it currently does in the case of mutating. The drawback of this option is that it conveys less information about language grammar.s

Environment

Swift version 6.1-dev (26e888e45e6ede4)

Additional information

No response

Guide contributeur

Direction de recherche
Localisez le diagnostic pour 'static functions not mutating' dans le code source du compilateur Swift, probablement dans lib/Sema ou un fichier de diagnostic similaire. Modifiez le message d'erreur pour lister les deux modificateurs 'static' et 'nonmutating' comme suggéré dans l'issue. Consultez la discussion pour obtenir un contexte supplémentaire sur le comportement attendu.
Stack technique
swift
Domaine
developer experience
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
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 programming language
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65